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

Unified Diff: tool/patch_sdk.dart

Issue 962213003: fix analysis messages (Closed) Base URL: git@github.com:dart-lang/dev_compiler.git@master
Patch Set: Created 5 years, 10 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
« test/checker/inferred_type_test.dart ('K') | « test/codegen_test.dart ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tool/patch_sdk.dart
diff --git a/tool/patch_sdk.dart b/tool/patch_sdk.dart
index 74ecb404634e48b1d0ed2ff0028390908d400b17..8540ed427fd412be68c1f94685bd04be606f2b79 100755
--- a/tool/patch_sdk.dart
+++ b/tool/patch_sdk.dart
@@ -108,7 +108,6 @@ List<String> _patchLibrary(List<String> partsContents, String patchContents) {
var patchFinder = new PatchFinder.parseAndVisit(patchContents);
// Merge `external` declarations with the corresponding `@patch` code.
- bool first = true;
for (var partContent in partsContents) {
var partEdits = new StringEditBuffer(partContent);
var partUnit = parseCompilationUnit(partContent);
@@ -138,7 +137,6 @@ class PatchApplier extends GeneralizingAstVisitor {
// TODO(jmesserly): remove duplicate imports
var directivePos = unit.directives.last.end;
for (var directive in patch.unit.directives) {
- var uri = directive.uri.stringValue;
var code = patch.contents.substring(directive.offset, directive.end);
edits.insert(directivePos, '\n' + code);
}
« test/checker/inferred_type_test.dart ('K') | « test/codegen_test.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698