Chromium Code Reviews| Index: sdk/lib/_internal/compiler/implementation/resolution/members.dart |
| diff --git a/sdk/lib/_internal/compiler/implementation/resolution/members.dart b/sdk/lib/_internal/compiler/implementation/resolution/members.dart |
| index 9bb7d1035768587e5b36312b35c8e2832ce4a6eb..a69d69f0625180d9e5ed47efdb4e5051dd195f57 100644 |
| --- a/sdk/lib/_internal/compiler/implementation/resolution/members.dart |
| +++ b/sdk/lib/_internal/compiler/implementation/resolution/members.dart |
| @@ -600,13 +600,16 @@ class ResolverTask extends CompilerTask { |
| if (value == null) return; |
| if (!(isUserDefinableOperator(value) || identical(value, 'unary-'))) return; |
| + bool isMinus = false; |
| int requiredParameterCount; |
| MessageKind messageKind; |
| FunctionSignature signature = function.computeSignature(compiler); |
| if (identical(value, 'unary-')) { |
| + isMinus = true; |
| messageKind = MessageKind.MINUS_OPERATOR_BAD_ARITY; |
| requiredParameterCount = 0; |
| } else if (isMinusOperator(value)) { |
| + isMinus = true; |
| messageKind = MessageKind.MINUS_OPERATOR_BAD_ARITY; |
| requiredParameterCount = 1; |
| } else if (isUnaryOperator(value)) { |
| @@ -622,17 +625,19 @@ class ResolverTask extends CompilerTask { |
| compiler.internalErrorOnElement(function, |
| 'Unexpected user defined operator $value'); |
| } |
| - checkArity(function, requiredParameterCount, messageKind); |
| + checkArity(function, requiredParameterCount, messageKind, isMinus); |
| } |
| void checkArity(FunctionElement function, |
| - int requiredParameterCount, MessageKind messageKind) { |
| + int requiredParameterCount, MessageKind messageKind, |
| + bool isMinus) { |
| FunctionExpression node = function.parseNode(compiler); |
| FunctionSignature signature = function.computeSignature(compiler); |
| if (signature.requiredParameterCount != requiredParameterCount) { |
| Node errorNode = node; |
| if (node.parameters != null) { |
| - if (signature.requiredParameterCount < requiredParameterCount) { |
| + if (isMinus || // Point to the parameter list is case of operator -. |
|
ngeoffray
2013/01/10 14:20:52
is -> in
|
| + signature.requiredParameterCount < requiredParameterCount) { |
| errorNode = node.parameters; |
| } else { |
| errorNode = node.parameters.nodes.skip(requiredParameterCount).head; |