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

Issue 2912843004: Fix an exception caused by incomplete input (issue 29687) (Closed)

Created:
3 years, 6 months ago by Brian Wilkerson
Modified:
3 years, 6 months ago
Reviewers:
scheglov
CC:
reviews_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Fix an exception caused by incomplete input (issue 29687) R=scheglov@google.com Committed: https://github.com/dart-lang/sdk/commit/852edf38d9f77dbea13d990f333e8b85b003b755

Patch Set 1 #

Patch Set 2 : Added test #

Total comments: 4
Unified diffs Side-by-side diffs Delta from patch set Stats (+25 lines, -1 line) Patch
M pkg/analyzer/lib/src/summary/summarize_const_expr.dart View 1 1 chunk +3 lines, -0 lines 2 comments Download
M pkg/analyzer/test/src/summary/resynthesize_common.dart View 1 2 chunks +22 lines, -1 line 2 comments Download

Messages

Total messages: 7 (2 generated)
Brian Wilkerson
I'm not sure where / how to write a test for this, but did verify ...
3 years, 6 months ago (2017-05-30 16:39:48 UTC) #2
Brian Wilkerson
Thanks! I added a test, and fixed a bug in checked mode. PTAL.
3 years, 6 months ago (2017-05-30 17:22:20 UTC) #3
scheglov
LGTM https://codereview.chromium.org/2912843004/diff/20001/pkg/analyzer/lib/src/summary/summarize_const_expr.dart File pkg/analyzer/lib/src/summary/summarize_const_expr.dart (right): https://codereview.chromium.org/2912843004/diff/20001/pkg/analyzer/lib/src/summary/summarize_const_expr.dart#newcode290 pkg/analyzer/lib/src/summary/summarize_const_expr.dart:290: value = 0; This could be also. value ...
3 years, 6 months ago (2017-05-30 17:25:18 UTC) #4
Brian Wilkerson
https://codereview.chromium.org/2912843004/diff/20001/pkg/analyzer/lib/src/summary/summarize_const_expr.dart File pkg/analyzer/lib/src/summary/summarize_const_expr.dart (right): https://codereview.chromium.org/2912843004/diff/20001/pkg/analyzer/lib/src/summary/summarize_const_expr.dart#newcode290 pkg/analyzer/lib/src/summary/summarize_const_expr.dart:290: value = 0; Done https://codereview.chromium.org/2912843004/diff/20001/pkg/analyzer/test/src/summary/resynthesize_common.dart File pkg/analyzer/test/src/summary/resynthesize_common.dart (right): https://codereview.chromium.org/2912843004/diff/20001/pkg/analyzer/test/src/summary/resynthesize_common.dart#newcode3312 ...
3 years, 6 months ago (2017-05-30 17:38:45 UTC) #5
Brian Wilkerson
3 years, 6 months ago (2017-05-30 17:39:06 UTC) #7
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
852edf38d9f77dbea13d990f333e8b85b003b755 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698