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

Unified Diff: runtime/vm/parser.cc

Issue 2640853005: Make Analyzer, VM and dart2js accept URI strings as part-of library identifier. (Closed)
Patch Set: Status fixes Created 3 years, 9 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « pkg/compiler/lib/src/elements/modelx.dart ('k') | tests/language/language_kernel.status » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/parser.cc
diff --git a/runtime/vm/parser.cc b/runtime/vm/parser.cc
index a9666a3e64ddef5934720280b23a13c002acb88e..210e9a6642b58f6bec4fa48902dbb57e8eca3b5b 100644
--- a/runtime/vm/parser.cc
+++ b/runtime/vm/parser.cc
@@ -6409,12 +6409,16 @@ void Parser::ParsePartHeader() {
ReportError("'part of' expected");
}
ConsumeToken();
- // The VM is not required to check that the library name matches the
- // name of the current library, so we ignore it.
- ExpectIdentifier("library name expected");
- while (CurrentToken() == Token::kPERIOD) {
- ConsumeToken();
- ExpectIdentifier("malformed library name");
+ // The VM is not required to check that the library name or URI matches the
+ // name or URI of the current library, so we ignore them.
+ if (CurrentToken() == Token::kSTRING) {
+ ParseStringLiteral(false);
+ } else {
+ ExpectIdentifier("library name expected");
+ while (CurrentToken() == Token::kPERIOD) {
+ ConsumeToken();
+ ExpectIdentifier("malformed library name");
+ }
}
ExpectSemicolon();
}
« no previous file with comments | « pkg/compiler/lib/src/elements/modelx.dart ('k') | tests/language/language_kernel.status » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698