Small fixes

lib/src/codegen/js_codegen.dart

 // Copyright (c) 2015, 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 dev_compiler.src.codegen.js_codegen;
 
 import 'dart:collection' show HashSet;
 import 'dart:io' show Directory, File;
 
 import 'package:analyzer/analyzer.dart' hide ConstantEvaluator;
@@ skipping 1223 matching lines @@
         return js.call('#$op', _visit(expr));
       }
     }
 
     assert(op.lexeme == '++' || op.lexeme == '--');
     return _emitPostfixIncrement(expr, op);
   }
 
   JS.Expression _emitPrefixIncrement(Token op, Expression expr) {
     var one = AstBuilder.integerLiteral(1);
+    one.staticType = rules.provider.intType;
     var increment = AstBuilder.binaryExpression(expr, op.lexeme[0], one);
     return _emitAssignment(expr, increment);
   }
 
   @override
   JS.Expression visitPrefixExpression(PrefixExpression node) {
     return _emitPrefixExpression(node.operator, node.operand);
   }
 
   JS.Expression _emitPrefixExpression(Token op, Expression expr) {
@@ skipping 255 matching lines @@
 
     if (clauses.length == 1) {
       // Special case for a single catch.
       var clause = clauses.single;
       if (clause.exceptionParameter != null) {
         _catchParameter = clause.exceptionParameter.name;
       }
     }
 
     var catchVarDecl = new JS.VariableDeclaration(_catchParameter);
-    var catchBody = _statement(_visitList(clauses));
+    var catchBody = new JS.Block(_visitList(clauses));

[vsm, 2015/03/10 13:34:47]

The js ast code asserts this is a block.

[Jennifer Messerly, 2015/03/10 15:04:08]

funny! we must've only hit the code path for >2 statements, which becomes a
block.
Interestingly this could've been caught by tools too (e.g. C# would catch it)
... catchBody was a Statement, but new Catch takes a Block, which is a subtype
of Statement ...

     _catchParameter = savedCatch;
 
     return new JS.Catch(catchVarDecl, catchBody);
   }
 
   JS.Statement _statement(Iterable stmts) {
     var s = stmts is List ? stmts : new List<JS.Statement>.from(stmts);
     // TODO(jmesserly): empty block singleton?
     if (s.length == 0) return new JS.Block([]);
     if (s.length == 1) return s[0];
@@ skipping 504 matching lines @@
 
     // TODO(jmesserly): in many cases marking the end will be unncessary.
     printer.mark(_location(node.end));
   }
 
   String _getIdentifier(AstNode node) {
     if (node is SimpleIdentifier) return node.name;
     return null;
   }
 }