fixes #202, check IndexExpression

lib/src/checker/checker.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.checker.checker;
 
 import 'package:analyzer/analyzer.dart';
 import 'package:analyzer/src/generated/ast.dart';
 import 'package:analyzer/src/generated/element.dart';
 import 'package:analyzer/src/generated/scanner.dart' show Token, TokenType;
@@ skipping 701 matching lines @@
         // Dynamic invocation
         // TODO(vsm): Move this logic to the resolver?
         if (op.type != TokenType.EQ_EQ && op.type != TokenType.BANG_EQ) {
           _recordDynamicInvoke(node);
         }
       } else {
         var element = node.staticElement;
         // Method invocation.
         if (element is MethodElement) {
           var type = element.type as FunctionType;
-          assert(type.normalParameterTypes.length == 1);
-          node.rightOperand =
-              checkArgument(node.rightOperand, type.normalParameterTypes[0]);
+          // Analyzer should enforce number of parameter types, but check in
+          // case we have erroneous input.
+          if (type.normalParameterTypes.isNotEmpty) {
+            node.rightOperand =
+                checkArgument(node.rightOperand, type.normalParameterTypes[0]);
+          }
         } else {
           // TODO(vsm): Assert that the analyzer found an error here?
         }
       }
     } else {
       // Non-method operator.
       switch (op.type) {
         case TokenType.AMPERSAND_AMPERSAND:
         case TokenType.BAR_BAR:
           var boolType = _rules.provider.boolType;
           node.leftOperand = checkArgument(node.leftOperand, boolType);
           node.rightOperand = checkArgument(node.rightOperand, boolType);
           break;
         case TokenType.BANG_EQ:
           break;
         default:
           assert(false);
       }
     }
     node.visitChildren(this);
   }
 
+  @override
+  void visitIndexExpression(IndexExpression node) {
+    if (_rules.isDynamicTarget(node.target)) {
+      _recordDynamicInvoke(node);
+    } else {
+      var element = node.staticElement;
+      if (element is MethodElement) {
+        var type = element.type as FunctionType;
+        // Analyzer should enforce number of parameter types, but check in
+        // case we have erroneous input.
+        if (type.normalParameterTypes.isNotEmpty) {
+          node.index = checkArgument(node.index, type.normalParameterTypes[0]);
+        }
+      } else {
+        // TODO(vsm): Assert that the analyzer found an error here?
+      }
+    }
+    node.visitChildren(this);
+  }
+
   DartType getType(TypeName name) {
     return (name == null) ? _rules.provider.dynamicType : name.type;
   }
 
   Expression checkAssignment(Expression expr, DartType type) {
     if (expr is ParenthesizedExpression) {
       expr.expression = checkAssignment(expr.expression, type);
     } else {
       final staticInfo = _rules.checkAssignment(expr, type, _constantContext);
       _recordMessage(staticInfo);
@@ skipping 83 matching lines @@
   void _recordDynamicInvoke(AstNode node) {
     _reporter.log(new DynamicInvoke(_rules, node));
   }
 
   void _recordMessage(StaticInfo info) {
     if (info == null) return;
     if (info.level >= logger.Level.SEVERE) _failure = true;
     _reporter.log(info);
   }
 }