Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(261)

Side by Side Diff: pkg/compiler/lib/src/universe/call_structure.dart

Issue 2647043002: Fix hints reported by analyzer. (Closed)
Patch Set: Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2015, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2015, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 library dart2js.call_structure; 5 library dart2js.call_structure;
6 6
7 import '../common.dart';
8 import '../common/names.dart' show Names; 7 import '../common/names.dart' show Names;
9 import '../elements/types.dart' show FunctionType; 8 import '../elements/types.dart' show FunctionType;
10 import '../util/util.dart'; 9 import '../util/util.dart';
11 import 'selector.dart' show Selector; 10 import 'selector.dart' show Selector;
12 11
13 /// The structure of the arguments at a call-site. 12 /// The structure of the arguments at a call-site.
14 // TODO(johnniwinther): Should these be cached? 13 // TODO(johnniwinther): Should these be cached?
15 // TODO(johnniwinther): Should isGetter/isSetter be part of the call structure 14 // TODO(johnniwinther): Should isGetter/isSetter be part of the call structure
16 // instead of the selector? 15 // instead of the selector?
17 class CallStructure { 16 class CallStructure {
(...skipping 134 matching lines...) Expand 10 before | Expand all | Expand 10 after
152 return first.compareTo(second); 151 return first.compareTo(second);
153 }); 152 });
154 return _orderedNamedArguments; 153 return _orderedNamedArguments;
155 } 154 }
156 155
157 @override 156 @override
158 String structureToString() { 157 String structureToString() {
159 return 'arity=$argumentCount, named=[${namedArguments.join(', ')}]'; 158 return 'arity=$argumentCount, named=[${namedArguments.join(', ')}]';
160 } 159 }
161 } 160 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698