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

Side by Side Diff: runtime/vm/resolver.h

Issue 19662003: Refactor resolution code in the vm to properly handle ambiguity errors. (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 7 years, 5 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 | Annotate | Revision Log
« no previous file with comments | « runtime/vm/parser_test.cc ('k') | runtime/vm/resolver.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2011, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2011, 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 #ifndef VM_RESOLVER_H_ 5 #ifndef VM_RESOLVER_H_
6 #define VM_RESOLVER_H_ 6 #define VM_RESOLVER_H_
7 7
8 #include "vm/allocation.h" 8 #include "vm/allocation.h"
9 9
10 namespace dart { 10 namespace dart {
(...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
46 // either application library or core library if no application library 46 // either application library or core library if no application library
47 // exists. Passing negative num_arguments means that the function 47 // exists. Passing negative num_arguments means that the function
48 // will be resolved by name only. 48 // will be resolved by name only.
49 // Otherwise null is returned if the number or names of arguments are not 49 // Otherwise null is returned if the number or names of arguments are not
50 // valid for the resolved function. 50 // valid for the resolved function.
51 static RawFunction* ResolveStatic(const Library& library, 51 static RawFunction* ResolveStatic(const Library& library,
52 const String& cls_name, 52 const String& cls_name,
53 const String& function_name, 53 const String& function_name,
54 intptr_t num_arguments, 54 intptr_t num_arguments,
55 const Array& argument_names, 55 const Array& argument_names,
56 StaticResolveType resolve_type); 56 StaticResolveType resolve_type,
57 String* ambiguity_error_msg);
57 58
58 // Resolve specified dart static function. 59 // Resolve specified dart static function.
59 static RawFunction* ResolveStaticByName(const Class& cls, 60 static RawFunction* ResolveStaticByName(const Class& cls,
60 const String& function_name, 61 const String& function_name,
61 StaticResolveType resolve_type); 62 StaticResolveType resolve_type);
62 63
63 // Resolve specified dart static function with specified arity. 64 // Resolve specified dart static function with specified arity.
64 static RawFunction* ResolveStatic(const Class& cls, 65 static RawFunction* ResolveStatic(const Class& cls,
65 const String& function_name, 66 const String& function_name,
66 intptr_t num_arguments, 67 intptr_t num_arguments,
67 const Array& argument_names, 68 const Array& argument_names,
68 StaticResolveType resolve_type); 69 StaticResolveType resolve_type);
69 }; 70 };
70 71
71 } // namespace dart 72 } // namespace dart
72 73
73 #endif // VM_RESOLVER_H_ 74 #endif // VM_RESOLVER_H_
OLDNEW
« no previous file with comments | « runtime/vm/parser_test.cc ('k') | runtime/vm/resolver.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698