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

Issue 9186015: Issue 933: Fix for resolving type of Function in field. (Closed)

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

Description

Issue 933: Fix for resolving type of Function in field. http://code.google.com/p/dart/issues/detail?id=933 R=zundel@google.com BUG= TEST= Committed: https://code.google.com/p/dart/source/detail?r=3410

Patch Set 1 #

Total comments: 1

Patch Set 2 : Rollback TypeTestCase change #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+42 lines, -2 lines) Patch
M compiler/java/com/google/dart/compiler/type/TypeAnalyzer.java View 1 chunk +5 lines, -0 lines 0 comments Download
M compiler/javatests/com/google/dart/compiler/type/TypeAnalyzerCompilerTest.java View 2 chunks +20 lines, -2 lines 0 comments Download
M tests/language/language-leg.status View 1 chunk +1 line, -0 lines 0 comments Download
A tests/language/src/FunctionFieldTest.dart View 1 chunk +16 lines, -0 lines 2 comments Download

Messages

Total messages: 6 (0 generated)
scheglov
8 years, 11 months ago (2012-01-11 20:29:47 UTC) #1
zundel
http://codereview.chromium.org/9186015/diff/1/compiler/javatests/com/google/dart/compiler/type/TypeTestCase.java File compiler/javatests/com/google/dart/compiler/type/TypeTestCase.java (right): http://codereview.chromium.org/9186015/diff/1/compiler/javatests/com/google/dart/compiler/type/TypeTestCase.java#newcode100 compiler/javatests/com/google/dart/compiler/type/TypeTestCase.java:100: assertEquals(message, expectedTypeErrors, 0); even before your change, the way ...
8 years, 11 months ago (2012-01-11 21:34:58 UTC) #2
zundel
lgtm
8 years, 11 months ago (2012-01-18 18:07:45 UTC) #3
regis
https://chromiumcodereview.appspot.com/9186015/diff/3003/tests/language/src/FunctionFieldTest.dart File tests/language/src/FunctionFieldTest.dart (right): https://chromiumcodereview.appspot.com/9186015/diff/3003/tests/language/src/FunctionFieldTest.dart#newcode4 tests/language/src/FunctionFieldTest.dart:4: // VMOptions=--fatal-type-errors --enable_type_checks The VM does not support --fatal-type-errors.
8 years, 10 months ago (2012-02-23 19:42:09 UTC) #4
scheglov
> tests/language/src/FunctionFieldTest.dart:4: // VMOptions=--fatal-type-errors > --enable_type_checks > The VM does not support --fatal-type-errors. How can ...
8 years, 10 months ago (2012-02-24 14:46:52 UTC) #5
zundel
8 years, 10 months ago (2012-02-24 15:38:57 UTC) #6
https://chromiumcodereview.appspot.com/9186015/diff/3003/tests/language/src/F...
File tests/language/src/FunctionFieldTest.dart (right):

https://chromiumcodereview.appspot.com/9186015/diff/3003/tests/language/src/F...
tests/language/src/FunctionFieldTest.dart:4: // VMOptions=--fatal-type-errors
--enable_type_checks
On 2012/02/23 19:42:09, regis wrote:
> The VM does not support --fatal-type-errors.

VMOptions is kind of a misnomer in that they are just cmdline flags passed to
whatever tool is being invoked.  The tools should be set to ignore unrecognized
options.

Powered by Google App Engine
This is Rietveld 408576698