[ptp-dev] [perf] Generalizing performance framework tools

[Beth Tibbitts <tibbitts@xxxxxxxxxx>]

Wyatt,
Some  observations in trying to use the performance tools framework to
launch my external tools.
(1 )IPerformanceLaunchConfigurationConstants probably needs to not be in an
'internal' package, since my client code is using it
(2) We need a general way to get user-specified values out at the end,
without knowing the translation that you do on the keys.
   To get an option value out of the launch configuration:
           e.g. in the XML that specifies the info that the UI is built
from, i have <togoption label="Report" optname="-repfile"   ...
              and later, in my postprocessing code (my derivation of
AbstractPerformanceDataManager)  i do ILaunchConfiguration.getAttribute
("-REPFILE_ARGUMENT_SAVED", "defaultValue")
             to get the value the user entered.
      I propose putting in a utility method that does this translation for
you, so the client code doesn't have to know that translation process.
      I have tried putting it AbstractPerformanceDataManager as
                    getConfigAttribute(ILaunchConfiguration config, String
key, String defaultValue)
      but it seems there ought to be a more general place to put it so that
you
      can use it when you stash that value as well, in
ToolMaker.finishToolOption()
     Do you have a suggestion for where it should go?  Make a general
ToolUtility class there perhaps?
(3) Should we put a constants class in the toolopts plugin instead of
continuing to add to the IPerformanceLaunchConfigurationConstants in the
main org.eclipse.ptp.perf plugin,
        to store constants used in that plugin?

I'm happy to make the changes if you agree or have other suggestions...


..Beth

Beth Tibbitts  (859) 243-4981  (TL 545-4981)
High Productivity Tools / Parallel Tools  http://eclipse.org/ptp
IBM T.J.Watson Research Center
Mailing Address:  IBM Corp., 745 West New Circle Road, Lexington, KY 40511