More String + elimination in tests

tools/testing/dart/test_options.dart

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #library("test_options_parser");
 
 #import("dart:io");
 #import("dart:builtin");
 #import("drt_updater.dart");
 
@@ skipping 75 matching lines @@
               '''Where the tests should be run.
     vm: Run Dart code on the standalone dart vm.
 
     d8: Run JavaScript from the command line using v8.
 
     drt: Run Dart or JavaScript in the headless version of Chrome,
          DumpRenderTree.
 
     dartium: Run Dart or JavaScript in Dartium.
 
-    [ff | chrome | safari | ie | opera]: Run JavaScript in the specified 
+    [ff | chrome | safari | ie | opera]: Run JavaScript in the specified
          browser.
 
     none: No runtime, compile only (for example, used for dartc static analysis
           tests).''',
               ['-r', '--runtime'],
               ['vm', 'd8', 'drt', 'dartium', 'ff', 'firefox', 'chrome',
               'safari', 'ie', 'opera', 'none'],
               'vm'),
           new _TestOptionSpecification(
               'arch',
@@ skipping 300 matching lines @@
                             'none']).indexOf(config['runtime']) >= 0;
           break;
         case 'dartc':
           isValid = config['runtime'] == 'none';
           break;
         case 'none':
           isValid = (const ['vm', 'drt',
                             'dartium']).indexOf(config['runtime']) >= 0;
       }
     if (!isValid) {
-      print("Warning: combination of ${config['compiler']} and " +
+      print("Warning: combination of ${config['compiler']} and "
           "${config['runtime']} is invalid. Skipping this combination.");
     }
     if (config['runtime'] == 'ie' &&
         Platform.operatingSystem != 'windows') {
       isValid = false;
-      print("Warning cannot run Internet Explorer on non-Windows operating" +
+      print("Warning cannot run Internet Explorer on non-Windows operating"
           " system.");
     }
     if (config['shard'] < 1 || config['shard'] > config['shards']) {
       isValid = false;
-      print("Error: shard index is ${config['shard']} out of " + 
+      print("Error: shard index is ${config['shard']} out of "
             "${config['shards']} shards");
     }
     if (config['runtime'] == 'dartium' && config['compiler'] == 'none' &&
         config['checked']) {
       // TODO(vsm): Set the DART_FLAGS environment appropriately when
       // invoking Selenium to support checked mode.  It's not clear
       // the current selenium API supports this.
       isValid = false;
       print("Warning: checked mode is not yet supported for dartium tests.");
     }
@@ skipping 54 matching lines @@
         var slashLocation = pattern.indexOf('/');
         if (slashLocation != -1) {
           suite = pattern.substring(0, slashLocation);
           pattern = pattern.substring(slashLocation + 1);
           pattern = pattern.replaceAll('*', '.*');
           pattern = pattern.replaceAll('/', '.*');
         } else {
           pattern = ".*";
         }
         if (selectorMap.containsKey(suite)) {
-          print("Error: '$suite/$pattern'.  Only one test selection" +
+          print("Error: '$suite/$pattern'.  Only one test selection"
                 " pattern is allowed to start with '$suite/'");
           exit(1);
         }
         selectorMap[suite] = new RegExp(pattern);
       }
       configuration['selectors'] = selectorMap;
     }
 
     // Expand the architectures.
     if (configuration['arch'].contains(',')) {
@@ skipping 126 matching lines @@
         return option;
       }
     }
     print('Unknown test option $name');
     exit(1);
   }
 
 
   List<_TestOptionSpecification> _options;
 }