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

Issue 9052002: Issue 999: NPE from CompileTimeConstantAnalyzer (Closed)

Created:
8 years, 12 months ago by codefu
Modified:
8 years, 11 months ago
Reviewers:
scheglov, zundel
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Patch Set 1 #

Patch Set 2 : Missed formatting #

Patch Set 3 : Adding tests #

Patch Set 4 : Cleanup of ErrorExpectation + static imports #

Messages

Total messages: 4 (0 generated)
codefu
8 years, 12 months ago (2011-12-29 17:08:46 UTC) #1
scheglov
LGTM, but test(s)? I wonder why this worked before. Is it failing only with getter ...
8 years, 12 months ago (2011-12-29 18:01:25 UTC) #2
codefu
On 2011/12/29 18:01:25, scheglov wrote: > LGTM, but test(s)? > > I wonder why this ...
8 years, 12 months ago (2011-12-29 18:15:11 UTC) #3
zundel
8 years, 11 months ago (2012-01-03 15:04:12 UTC) #4
lgtm

Powered by Google App Engine
This is Rietveld 408576698