[yocto] [PATCH 0/8] Refactoring of remotetools plugin

Zhang, Jessica jessica.zhang at intel.com
Wed May 1 12:11:17 PDT 2013


Hi Ioana,

Overall looks good with few comment/question that I provide separately.

Thanks,
Jessica

-----Original Message-----
From: yocto-bounces at yoctoproject.org [mailto:yocto-bounces at yoctoproject.org] On Behalf Of Ioana Grigoropol
Sent: Tuesday, April 30, 2013 7:36 AM
To: yocto at yoctoproject.org
Subject: [yocto] [PATCH 0/8] Refactoring of remotetools plugin

- refactor all remote tools plugin to have single points of control for remote utilities plugin
- remove all duplicate code

Ioana Grigoropol (8):
  Remove UstLegacyModel since it is no longer supported
  Refactor Model(s) to display customized messages
  Unify preProcess for all commands models
  Add default behaviour for commands for post processing
  Remove duplicate code when getting remote data from powertop and ust2
  Refactor RemoteApplication implementation
  Refactor powertop, oprofile and ust2 models to remote duplicate code
  Refactor Latencytop and Perf Handler to remove duplicate code

 .../yocto/sdk/remotetools/actions/BaseModel.java   |  114 +++++++---
 .../sdk/remotetools/actions/IBaseConstants.java    |    3 +
 .../sdk/remotetools/actions/LatencytopHandler.java |   22 +-
 .../sdk/remotetools/actions/OprofileModel.java     |   79 ++-----
 .../yocto/sdk/remotetools/actions/PerfHandler.java |   22 +-
 .../sdk/remotetools/actions/PowertopModel.java     |   91 ++------
 .../sdk/remotetools/actions/SystemtapModel.java    |   34 ++-
 .../sdk/remotetools/actions/TerminalHandler.java   |   16 +-
 .../yocto/sdk/remotetools/actions/Ust2Model.java   |  132 +++--------
 .../sdk/remotetools/actions/UstModelLegacy.java    |  238 --------------------
 .../sdk/remotetools/remote/RemoteApplication.java  |  155 -------------
 .../sdk/remotetools/remote/RemoteShellExec.java    |  149 ++++++++++++
 12 files changed, 346 insertions(+), 709 deletions(-)  delete mode 100644 plugins/org.yocto.sdk.remotetools/src/org/yocto/sdk/remotetools/actions/UstModelLegacy.java
 delete mode 100644 plugins/org.yocto.sdk.remotetools/src/org/yocto/sdk/remotetools/remote/RemoteApplication.java
 create mode 100644 plugins/org.yocto.sdk.remotetools/src/org/yocto/sdk/remotetools/remote/RemoteShellExec.java

--
1.7.9.5

_______________________________________________
yocto mailing list
yocto at yoctoproject.org
https://lists.yoctoproject.org/listinfo/yocto



More information about the yocto mailing list