Ticket #225: cmdline-subOpts.patch

Source/BackEndClr/SharpGenerator.cobra

@@ -72 +72 @@
         _modules.add(SharpModule(fileName, _verbosity))
 
     def writeSharpRunAllTests(cw as CurlyWriter)
-        runner = .options['test-runner'] to String  # ex: Cobra.Lang.CobraCore.runAllTests, ex: MyProgram.runTests
+        runner = .options.getSubOpt('testifyArgs', 'runner') to String  # ex: Cobra.Lang.CobraCore.runAllTests, ex: MyProgram.runTests
         if runner <> 'nil'
             if runner.endsWith('()'), runner = runner[:-2]
             if runner.startsWith('Cobra.Lang.'), runner = 'CobraLangInternal.' + runner['Cobra.Lang.'.length:]

Source/CommandLine.cobra

@@ -34 +34 @@
     
     get type as String
         require .isUnpacked
-        ensure result in ['accumulator', 'args-list', 'bool', 'int', 'menu', 'set', 'string']
+        ensure result in ['accumulator', 'args-list', 'bool', 'int', 'menu', 'set', 'string', 'multival']
         return _type
 
     get synonyms from var
@@ -77 +77 @@
         if _type == 'main'
             _isMain, _type = true, 'bool'
         if .containsKey('is-main'), _isMain = this['is-main'] to bool
-        assert _type in ['accumulator', 'args-list', 'bool', 'int', 'menu', 'set', 'string']
+        assert _type in ['accumulator', 'args-list', 'bool', 'int', 'menu', 'set', 'string', 'multival']
         if .containsKey('synonyms')
             for syn in this['synonyms']
                 _synonyms.add(syn)
@@ -94 +94 @@
         _unpackPlatforms
         _isUnpacked = true
         assert .type=='menu' implies .choices.count > 0
-        assert .choices.count <> 0 implies .type in ['menu', 'set']
+        assert .choices.count <> 0 implies .type in ['menu', 'set', 'multival']
+        assert .type=='multival' implies .containsKey('subOptions') and .containsKey('choices')
+        assert .type=='multival' implies this['subOptions'].specsList.count == this['choices'].count
+        assert .type=='multival' implies .choices.count > 0
 
     def _unpackPlatforms
         if not .containsKey('platforms')
@@ -103 +106 @@
             assert platform in ['jvm', '.net', 'objc']
         this['platforms'] = Set<of String>(this['platforms'] to List<of String>)
 
+    def defaultIfOn as dynamic
+        if .containsKey('defaultIfOn'), return this['defaultIfOn']
+        if .containsKey('default'), return this['default']
+        return 'on' 
+        
+    def subOptions as List<of OptionSpec>
+        """Return SubOption list ( of OptionSpec) or empty list if no subOptions"""
+        if .containsKey('subOptions')  
+            clos = this['subOptions']
+            assert clos inherits CommandLineOptionSpecs
+            return clos.specsList 
+        return List<of OptionSpec>()
+        # TODO: cache this in unpack and just return cached value from here
 
+    def subOptionValue(name as String, keyList as vari String) as dynamic
+        """
+        Find subOptions spec with given name; using keys from keyList return the value for 
+        the first existing key in spec. 
+        Return empty string if no subOptions or no matching keys found.
+        """
+        for subOpt in .subOptions
+            if subOpt['name'] == name  
+                for key in keyList
+                    if subOpt.containsKey(key), return subOpt[key]
+                break   
+        return ''   
+        
+    def subOptionSpec(name as String) as OptionSpec
+        """Return subOption for name or fail"""
+        require .containsKey('subOptions')  
+        for subOpt in .subOptions
+            if subOpt['name'] == name  
+                return subOpt
+        assert subOpt, name
+        return  OptionSpec()
+        
 class CommandLineOptionSpecs
     """
     Returns the command line option specifications, via .specsList, as a List of OptionSpec.
@@ -114 +152 @@
 
     cue init
         base.init
-        for rawSpec in _rawCommandLineOptionSpecs
+        __initInternal(_rawCommandLineOptionSpecs)
+            
+    cue init(rawOptionSpecs) 
+        base.init
+        __initInternal(rawOptionSpecs)
+        
+    def __initInternal(rawOptionSpecs) 
+        assert rawOptionSpecs.count
+        for rawSpec in rawOptionSpecs
             spec = OptionSpec()
             if rawSpec inherits Dictionary<of String, Object>
-                for key in rawSpec.keys, spec[key] = rawSpec[key]
+                for key in rawSpec.keys
+                    spec[key] = if(key == 'subOptions', CommandLineOptionSpecs(rawSpec[key]), rawSpec[key])
             else if rawSpec inherits Dictionary<of String, String>
                 for key in rawSpec.keys, spec[key] = rawSpec[key]
             else
                 throw FallThroughException(rawSpec.getType)
             spec.unpack
             _specs.add(spec)
-
+                
     def specsList as List<of OptionSpec>
         """
         Returns the option specs in the order they were declared.
         """
         return _specs
 
+    # Spec for each cobra compiler commandline option - List of Dictionaries
+    # TODO: doc the types
+    #type: main
+    #type: string
+    #type: int
+    #type: bool
+    #type: set
+    #type: menu
+    #
+    # type: multival supports a single cmdline option supporting multiple subOptions as comma separated name[=value] pairs
+    #   e.g. -multiValueDefault  -multival:key0,key1=15,key2=fred 
+    #   Uses standard entries 
+    #   (e.g. 'default' for a default value (commaSep list of pairs) if the cmdline option not given)
+    #   On the cmdline you can specify just the option (without any subOptions - to enable a default set)
+    #       this uses the value of entry 'defaultIfOn' (or 'default' if non existent.)
+    #   Must have a  'choices' entry which is a list of the recognised subOption names AND 
+    #   Must have a 'subOptions' entry whose contents is a list of Dictionaries, one for each subOption supported
+    #       Each subOption dict must have at least entry for 'name' but more commonly also 
+    #       'type', 'description' (for help display) and possibly entries for defaults 
+    #       'defaultIfKeyOnly' and  'default'
+    #       On cmdline can give just a subOption( without '=value'). If want to support this and its different from 
+    #           a value for 'default' (below) provide an entry for 'defaultIfKeyOnly' giving the value for this case.
+    #       If want a subOption value set if the option is enabled regardless of the subOption being given
+    #           then provide a 'default' entry.
+    #   
     var _rawCommandLineOptionSpecs = [
         {
             'name': 'about',
@@ -332 +404 @@
             'args': 'TYPENAME',
         },
         {
+            'name': 'metrics',
+            'type': 'multival',
+            'args': r'[loc[=value],][mcc[=value]...]',
+            'description': 'Enable metrics calculations; name specific metrics to be done and any threshold values as comma separated name=value pairs.',
+            'choices': ['loc', 'mcc'],  #allowable subOption keys
+            'example': ['loc=80,mcc=10', 'mcc,loc=50', 'mcc=12', 'mcc,loc'],
+            #'default': 'loc=100,mcc=25',       # default if not specify name at all (none)
+            'defaultIfOn': 'loc=100,mcc=25',    # default (what subOpt values enabled) if only specify name
+            'subOptions' : [
+                {
+                    'name': 'loc',
+                    'type': 'int',
+                    'description': 'Show methodname and number of codelines if greater than threshold value.', 
+                    'defaultIfKeyOnly': '100',  # value to set if only specify subOpt name
+                },  
+                {
+                    'name': 'mcc',
+                    'type': 'int',
+                    'description': 'Show method name and McCabe Complexity Calculation value if omplexity more than threshold. Reasonable values are in range 12-30',
+                    'defaultIfKeyOnly': '25', # value to set if only specify subOpt name
+                },
+            ],
+        },
+        {
             'name': 'namespace',
             'synonyms': ['name-space', 'ns'],
             'type': 'string',
@@ -429 +525 @@
             'type': 'main',
         },
         {
-            'name': 'test-runner',
-            'type': 'string',
-            'description': 'Specify the method to invoke to run the unit tests. The method must be "shared". Typically the method will make use of classes in Cobra.Lang.Test to set up and initiate the test run.',
-            'default': 'Cobra.Lang.CobraCore.runAllTests',
-            'args': 'QUALIFIED-METHOD-NAME|nil',
-        },
-        {
             'name': 'testify',
-            'description': '...',
+            'description': 'Run Cobra compiler test suite',
             'type': 'main',
             'developer-only': true,
         },
         {
-            'name': 'testify-results',
-            'description': 'The filename to write the testify results to. Progress is still written to console.',
-            'type': 'string',
-            'default': 'r-testify',
+            'name': 'testifyArgs',
+            'type': 'multival',
+            'description': 'Set names and values for controlling testify as a list of comma separated name=value pairs.',
+            'choices': ['resultsFile', 'runner', 'numThreads'],  #allowable subOption keys
+            'example': ['numThreads=4,resultsFile=testResult', 'runner=Core.myTestRunner', 'numThreads=2'],
             'developer-only': true,
+            'default': 'resultsFile=r-testify,runner=Cobra.Lang.CobraCore.runAllTests,numThreads=1',
+            'subOptions' : [
+                {
+                    'name': 'resultsFile',
+                    'type': 'string',
+                    'description': 'Filename to write the testify results to. Progress is still written to console.',
+                    'default': 'r-testify',        # value to default to if not specify name at all
+                    #'defaultIfKeyOnly': 'r-testify', # value to set if only specify subOpt name
+                },  
+                {
+                    'name': 'runner',
+                    'type': 'string',
+                    'description': 'method to invoke to run the unit tests. The method must be "shared". Typically the method will make use of classes in Cobra.Lang.Test to setup and initiate the test run.',
+                    'default': 'Cobra.Lang.CobraCore.runAllTests',
+                },
+                {
+                    'name': 'numThreads',
+                    'type': 'int',
+                    'description': 'Number of threads to run the testify testsuite in.',
+                    'default': '1',
+                },
+            ],
         },
         {
-            'name': 'testify-threads',
-            'description': '...',
-            'type': 'int',
-            'developer-only': true,
-        },
-        {
             'name': 'timeit',
             'description': 'Gives the total duration of running cobra (including the target program, if it is to be run). This is "wall time", not "cpu time".',
             # although this option is implied by 'testify', the description does not say so, since 'testify' is a hidden option
@@ -488 +594 @@
             'type': 'main',
         },
     ]
+    
 
-
 class CommandLine
     """
     The main options that control the command line's behavior are:
@@ -589 +695 @@
             dest = _htmlWriter to TextWriter
         else
             dest = Console.out
-        if _htmlWriter
-            stylePath = Path.combine(Path.getDirectoryName(CobraCore.exePath), 'styles-output-html.css')
-            _htmlWriter.writeHtml('<html><head><link href="file://[stylePath]" rel=stylesheet type="text/css"></head><body>[_htmlWriter.newLine]')
+            
+        _runPreamble
         print to dest
             paths = _pathList to !
             options = _options
@@ -609 +714 @@
                 print 'Paths:'
                 for path in paths
                     print '    [path]'
+
             if options.boolValue('testify')
                 .doTestify(paths)
             else if options.boolValue('run')
@@ -635 +741 @@
                 .doHelp
             else
                 .doRun(paths)
+            _runPostamble
+                
+    def _runPreamble        
         if _htmlWriter
+            stylePath = Path.combine(Path.getDirectoryName(CobraCore.exePath), 'styles-output-html.css')
+            _htmlWriter.writeHtml('<html><head><link href="file://[stylePath]" rel=stylesheet type="text/css"></head><body>[_htmlWriter.newLine]')
+
+    def _runPostamble
+        if _htmlWriter
             _htmlWriter.writeHtml('</body></html>[_htmlWriter.newLine]')
 
     def doHighlight(paths as List<of String>) as Compiler
@@ -679 +793 @@
             # Each phase of the compiler may throw an exception to stop compilation.
             # Before doing so, it prints its errors.
             assert c.errors.count>0
-            if _options.containsKey('editor')
-                spec = _options['editor'] to String?
-            else
-                spec = Environment.getEnvironmentVariable('COBRA_EDITOR')
-            if spec and spec <> ''
-                if spec.indexOf('FILE')==-1
-                    .error('Missing FILE from editor spec.')
-                if spec.indexOf('LINE')==-1
-                    .error('Missing LINE from editor spec.')
-                i = spec.indexOf('_')
-                if i == -1
-                    i = spec.indexOf(' ')
-                    if i == -1
-                        .error('Missing underscore or space from editor spec.')
-                exeName = spec.substring(0, i)
-                args = spec.substring(i+1)
-                for error in c.errors
-                    if error.isError and error.hasSourceSite
-                        if error.fileName.trim <> ''
-                            # trace error.fileName, error.lineNum
-                            args = args.replace('FILE', error.fileName)
-                            args = args.replace('LINE', error.lineNum.toString)
-                            p = System.Diagnostics.Process()
-                            p.startInfo.fileName = exeName
-                            p.startInfo.arguments = args
-                            p.startInfo.useShellExecute = false
-                            if _verbosity >= 3
-                                print 'Running: [p.startInfo.fileName] [p.startInfo.arguments]'
-                            try
-                                p.start
-                                p.waitForExit  # TODO: is this really needed?
-                            catch exc as Exception
-                                print 'Cannot invoke editor:'
-                                print '    Command: [p.startInfo.fileName] [p.startInfo.arguments]'
-                                print '    Exception: [exc]'
-                            break
+            editorSpec = if(_options.containsKey('editor'), _options['editor'] to String?, Environment.getEnvironmentVariable('COBRA_EDITOR'))
+            if editorSpec and editorSpec.length
+                _editorOnErrorLines(c, editorSpec to !)
+    
         CobraMain.linesCompiled = c.linesCompiled 
         CobraMain.nodesCompiled = c.nodesCompiled 
         CobraMain.tokensCompiled = c.tokensCompiled 
         return c
 
+    def _editorOnErrorLines(c as Compiler, editorSpec as String)
+        if editorSpec.indexOf('FILE')==-1,  .error('Missing FILE from editor spec.')
+        if editorSpec.indexOf('LINE')==-1,  .error('Missing LINE from editor spec.')
+        i = editorSpec.indexOf('_')
+        if i == -1
+            i = editorSpec.indexOf(' ')
+            if i == -1, .error('Missing underscore or space from editor spec.')
+
+        editorExe = editorSpec.substring(0, i)
+        args = editorSpec.substring(i+1)
+        for error in c.errors
+            if error.isError and error.hasSourceSite
+                if error.fileName.trim <> ''
+                    # trace error.fileName, error.lineNum
+                    args = args.replace('FILE', error.fileName)
+                    args = args.replace('LINE', error.lineNum.toString)
+                    p = System.Diagnostics.Process()
+                    p.startInfo.fileName = editorExe
+                    p.startInfo.arguments = args
+                    p.startInfo.useShellExecute = false
+                    if _verbosity >= 3
+                        print 'Running: [p.startInfo.fileName] [p.startInfo.arguments]'
+                    try
+                        p.start
+                        p.waitForExit  # TODO: is this really needed?
+                    catch exc as Exception
+                        print 'Cannot invoke editor:'
+                        print '    Command: [p.startInfo.fileName] [p.startInfo.arguments]'
+                        print '    Exception: [exc]'
+                    break
+        
     def doDocument(paths as List<of String>) as Compiler
         comp = .doCompile(paths, false, false, do(c as Compiler)=c.lastPhase inherits BindInterfacePhase)
         GenerateHtmlDocVisitor(do(module)=module inherits CobraModule).gen(comp)
@@ -799 +912 @@
     
         exeFileName as String? = nil
         runArgs = .options.getStringList('run-args')
-        # TODO: what's this?
+        # TODO: exeArgs as arglist of Strings exe-name exe-arg-0 exe-arg-1 ... cf argv list
         # exeArgs = .options.getDefaultLOStr('exe-args')
         # if exeArgs.count
         #   exeFileName = exeArgs[0]
@@ -838 +951 @@
         print '      -test ....... Run the unit tests of a library.'
         print '      -document ... Document the program (partial compilation). Also, -doc'
         print '      -highlight .. Syntax highlight the program in HTML.'
+        if Utils.isDevMachine
+            print '      -testify .... Run the compiler testsuite (cwd must be cobra Source dir).'
         print ''
         print '  cobra <options> <command>'
         print '    * standalone commands are:'
@@ -877 +992 @@
                     sep = ', '
                 print
             s = spec.description
-            while s.length
-                if s.length < width
-                    print '[leftMarginStr][s]'
-                    s = ''
-                else
-                    # TODO: bug in here for narrow widths. try "width = 20" to reproduce
-                    j = width + 1
-                    if j >= s.length, j = s.length - 1
-                    while j > 0 and s[j] <> ' ', j -= 1
-                    if j
-                        sub = s.substring(0, j)
-                        s = if(s.length, s.substring(j+1), '')
-                        print '[leftMarginStr][sub]'
+            if s.length
+                _printWrapped(s, width, leftMarginStr) 
+            for subOptionSpec in spec.subOptions 
+                assert subOptionSpec.name.length
+                if subOptionSpec.containsKey('description')
+                    _printWrapped('"[subOptionSpec.name]" = [subOptionSpec.description]' , width, leftMarginStr + '  ') 
+                if subOptionSpec.hasDefault
+                    soDefault = subOptionSpec['default']
+                    print '[leftMarginStr]    (default is [soDefault])'
             if spec.containsKey('example')
                 if spec['example'] inherits System.Collections.IList
                     first = true
@@ -903 +1014 @@
             if spec.containsKey('eg') # verbatim example line
                 print '[leftMarginStr]e.g. [spec["eg"]]'
 
+    def _printWrapped(s as String, width as int, leftMarginStr as String)
+        while s.length
+            if s.length < width
+                print '[leftMarginStr][s]'
+                s =''
+            else    
+                # TODO: bug in here for narrow widths. try "width = 20" to reproduce
+                j = width + 1
+                if j >= s.length, j = s.length - 1
+                while j > 0 and s[j] <> ' ', j -= 1
+                if j
+                    sub = s.substring(0, j)
+                    s = if(s.length, s.substring(j+1), '')
+                    print '[leftMarginStr][sub]'
+
     def doAbout
         # CC: multiline string
         print
@@ -1257 +1383 @@
             
     def _addInDefaults(valueDict as Dictionary<of String, Object>)
         for spec in _optionSpecs
+            if spec.containsKey('subOptions') and valueDict.containsKey(spec.name) 
+                # set defaults for unspecified subOptions with defaults if any subOption is set
+                _addInSubOptionDefaults(spec, valueDict)
+
             if not valueDict.containsKey(spec.name) and spec.hasDefault
                 defaultValue = _interpretValue(spec.default, spec) to !
                 if .verbosity
                     print 'Setting option "[spec.name]" to default value [defaultValue].'
                 valueDict[spec.name] = defaultValue
-
+                
+    def _addInSubOptionDefaults(spec as OptionSpec, valueDict as Dictionary<of String, Object>)
+        ov = valueDict[spec.name] to OptionValues # that whats been explicitly set
+        for subOptSpec in spec.subOptions
+            if subOptSpec.hasDefault
+                if not ov.containsKey(subOptSpec.name) # value not set already
+                    defaultValue = _interpretValue(subOptSpec['default'], subOptSpec) to !
+                    # Should we force subOptions to always have a default value ??
+                    if .verbosity
+                        print 'Setting option "[spec.name]:[subOptSpec.name]" to default value [defaultValue].'
+                    ov[subOptSpec.name] = defaultValue
+                                    
+                        
     def _unpackOptions(valueDict as Dictionary<of String, Object>, fileList as List<of String>) 
         """
         Unpack certain options (verbosity and timeit) into specific class fields, 
@@ -1381 +1523 @@
                     else
                         valueSet.add(choice)
                 value = valueSet
+            on 'multival' # comma separated  key=value pairs  - key1[=value],key2[=value],...
+                if valueStr =='on', valueStr =  spec.defaultIfOn
+                pairs = List<of String>(valueStr.split(c','))
+                assert pairs.count > 0
+                subValueOpts = OptionValues()
+                for kvp in pairs
+                    kve = List<of String>(kvp.trim.split(c'='))
+                    assert kve.count > 0
+                    key = kve[0]
+                    if not key in spec.choices and key <> 'on'
+                        print 'Unknown subOption key "[key]" in option "[spec.name]"'
+                        continue
+                    valStr = if(kve.count > 1, kve[1], spec.subOptionValue(key, 'defaultIfKeyOnly', 'default') to String)
+                    iValue = _interpretValue(valStr, spec.subOptionSpec(key))   
+                    subValueOpts.setKV(key, iValue)
+                value = subValueOpts    
             else
                 throw FallThroughException(spec.type)
         return value
@@ -1428 +1586 @@
 
 
 class OptionValues inherits Dictionary<of String, Object>
-
+    
     var _isSpecified = Dictionary<of String, bool>()  # CC: could just be a Set
 
     cue init
@@ -1460 +1618 @@
 
     def get(key as String) as dynamic?
         return this[key]
-
+        
+    def setKV(key as String, value as Object)
+        this[key] = value
+        .didSpecify(key)
+        
     def getDefault(key as String, default as dynamic?) as dynamic?
         if .containsKey(key)
             return this[key]
@@ -1477 +1639 @@
         else
             return List<of String>()
 
+    def getSubOpt(majorKey as String, minorKey as String, default as dynamic) as dynamic
+        """
+        Return subOption value for the given option and subOption (for a multi key=value Option)
+        SubOptions are stored themselves as a set of OptionValues.
+        If no major or minor key value return default.  
+        """
+        if .containsKey(majorKey)
+            so = .get(majorKey) to OptionValues
+            if  so.containsKey(minorKey)
+                return so[minorKey] 
+        return default
+                
+    def getSubOpt(majorKey as String, minorKey as String ) as dynamic
+        """
+        Return subOption value for the given option and subOption (multivalued option,  multi key=value).
+        Both the Option and subOption must exist otherwise an assert exception is incurred
+        """
+        assert .containsKey(majorKey)
+        so = .get(majorKey) to OptionValues
+        assert  so.containsKey(minorKey)
+        return so[minorKey] 
+
     # CC: def getDefault<of T>(key as String, value as T) as T ...
 
     def setValue(key as String) as Set<of String>

Source/TestifyRunner.cobra

@@ -58 +58 @@
         paths = _pathList
         if paths.count == 0
             paths = .cobraTestPaths + [Path.getFullPath(Path.combine('..', 'HowTo'))]
-        numThreads = .options.getDefault('testify-threads', 1) to int
+        #numThreads = .options.getDefault('testify-threads', 1) to int
+        numThreads = .options.getSubOpt('testifyArgs', 'numThreads', 1) to int
         if numThreads > 1
             .runThreaded(numThreads, paths)
         else
@@ -77 +78 @@
 
         args = CobraCore.commandLineArgs
         _subCobraExe = args[0]
-        _subCommandLineArgs = for arg in args[1:] where arg.startsWith('-') and not '-testify-threads:' in arg
+        #_subCommandLineArgs = for arg in args[1:] where arg.startsWith('-') and not '-testify-threads:' in arg
+        _subCommandLineArgs = for arg in args[1:] where arg.startsWith('-') and not '-testifyArgs:' in arg
 
         _statusWriter.writeLine('Queueing for threads:')
         for path in paths
@@ -102 +104 @@
             concat.append(sep)
             concat.append(File.readAllText(fileName))
             sep = sepNl
-        File.writeAllText(.options['testify-results'] to String, concat.toString)
+        File.writeAllText(.options.getSubOpt('testifyArgs','resultsFile') to String, concat.toString)
         for fileName in _subResultsFileNames
             try
                 File.delete(fileName)
@@ -110 +112 @@
                 _statusWriter.writeLine('warning: Cannot delete "[fileName]" due to: [exc]')
 
     def _printTotals
-        resultsFileName = .options['testify-results'] to String
+        resultsFileName = .options.getSubOpt('testifyArgs', 'resultsFile') to String
         using resultsWriter = File.appendText(resultsFileName)
             __printTotals(resultsWriter to !)
         __printTotals(_statusWriter to !)
@@ -133 +135 @@
                 pathIndex = _subDirQueue.count
                 resultsFileName = 'r-testify-[pathIndex]'
                 lock _subResultsFileNames, _subResultsFileNames[pathIndex] = resultsFileName
-                args = _subCommandLineArgs + ['-testify-results:[resultsFileName]', '-testify-threads:1', '"[path]"']
-                lock _statusWriter, _statusWriter.writeLine('Thread [tid] start: [args.join(" ")]')
+                args = _subCommandLineArgs + ['-testifyArgs:resultsFile=[resultsFileName],numThreads=1', '"[path]"']
+                lock _statusWriter, _statusWriter.writeLine('Thread [tid] start: [_subCobraExe] [args.join(" ")]')
             p = Process()
             p.startInfo.useShellExecute = false
             p.startInfo.redirectStandardOutput = true
@@ -177 +179 @@
         _statusWriter = IndentedWriter(AutoFlushWriter(Console.out))
         _statusWriter.indentString = '    '
         try
-            resultsFileName = .options['testify-results'] to String
+            resultsFileName = .options.getSubOpt('testifyArgs', 'resultsFile') to String
             using resultsWriter = File.createText(resultsFileName)
                 _resultsWriter = IndentedWriter(AutoFlushWriter(resultsWriter))
                 print to _resultsWriter to !

Developer/IntermediateReleaseNotes.text

@@ -94 +94 @@
 
 * Added -verbosity-ref option to output information about resolving references to libraries.
 
-
+* Added support for specifying a cmdline option with subOptions and modified  
+    -test-runner, -testify-Threads, -testify-results to use it becoming a new multival option 
+    -testifyArgs with subOptions runner,results and numThreads.
+    e.g. -testifyArgs:results=tResults,numThreads=4,runner=Core.Runner
+         -testifyArgs:numThreads=4
+        
 ================================================================================
 Samples etc.
 ================================================================================