Fix most warnings and other minor cleanups.

lib/compiler/implementation/operations.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.
 
 interface Operation {
   final SourceString name;
   bool isUserDefinable();
 }
 
 interface UnaryOperation extends Operation {
@@ skipping 217 matching lines @@
       NumConstant leftNum = left;
       NumConstant rightNum = right;
       return new DoubleConstant(leftNum.value + rightNum.value);
     } else if (left.isString() && !right.isObject()) {
       PrimitiveConstant primitiveRight = right;
       DartString rightDartString = primitiveRight.toDartString();
       StringConstant leftString = left;
       if (rightDartString.isEmpty()) {
         return left;
       } else if (leftString.value.isEmpty()) {
-        // TODO(floitsch): There is no right.node. Find a node some other way.
-        return new StringConstant(rightDartString, right.node);
+        return new StringConstant(rightDartString, leftString.node);
       } else {
         DartString concatenated =
             new ConsDartString(leftString.value, rightDartString);
         return new StringConstant(concatenated, leftString.node);
       }
     } else {
       return null;
     }
   }
 }
@@ skipping 64 matching lines @@
   bool isUserDefinable() => false;
   const IdentityOperation();
   Constant fold(Constant left, Constant right) {
     // In order to preserve runtime semantics which says that NaN !== NaN don't
     // constant fold NaN === NaN. Otherwise the output depends on inlined
     // variables and other optimizations.
     if (left.isNaN() && right.isNaN()) return null;
     return new BoolConstant(left == right);
   }
 }