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

Unified Diff: pkg/analysis_server/lib/src/services/correction/fix_internal.dart

Issue 2547053002: Guard against missing token (issue 27963) (Closed)
Patch Set: Created 4 years 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/analysis_server/lib/src/services/correction/fix_internal.dart
diff --git a/pkg/analysis_server/lib/src/services/correction/fix_internal.dart b/pkg/analysis_server/lib/src/services/correction/fix_internal.dart
index 3e3da6be9d60e47bc3a153d06d7009938ff90f08..aaa21fa30a6033034eab4053a7d54c4d21667bb4 100644
--- a/pkg/analysis_server/lib/src/services/correction/fix_internal.dart
+++ b/pkg/analysis_server/lib/src/services/correction/fix_internal.dart
@@ -489,9 +489,17 @@ class FixProcessor {
AstNode targetNode = targetElement.computeNode();
if (targetNode is FunctionDeclaration) {
FunctionExpression function = targetNode.functionExpression;
- targetOffset = function.parameters.leftParenthesis.end;
+ Token paren = function.parameters.leftParenthesis;
+ if (paren == null) {
+ return;
+ }
+ targetOffset = paren.end;
} else if (targetNode is MethodDeclaration) {
- targetOffset = targetNode.parameters.leftParenthesis.end;
+ Token paren = targetNode.parameters.leftParenthesis;
+ if (paren == null) {
+ return;
+ }
+ targetOffset = paren.end;
} else {
return;
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698