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

Issue 8497064: Fix method resolution to warn about things that don't exist on the declared type. (Closed)

Created:
9 years, 1 month ago by Jennifer Messerly
Modified:
9 years, 1 month ago
Reviewers:
jimhug
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Fix method resolution to warn about things that don't exist on the declared type. Not sure how/when this got broken, but we were only warning if no name in the whole world matched. The fix for this is in "value.dart" Also cleaned up a lot of the compiler code to keep it warning free. There are some warnings in leg, but I left those as I'm not sure what should be done. Committed: https://code.google.com/p/dart/source/detail?r=1383

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+521 lines, -411 lines) Patch
M frog/frogsh View 94 chunks +282 lines, -235 lines 0 comments Download
M frog/gen.dart View 6 chunks +54 lines, -45 lines 0 comments Download
M frog/lib/corelib.dart View 1 chunk +1 line, -1 line 0 comments Download
M frog/library.dart View 3 chunks +3 lines, -3 lines 0 comments Download
M frog/member.dart View 13 chunks +48 lines, -30 lines 0 comments Download
M frog/tree.dart View 1 chunk +1 line, -0 lines 0 comments Download
M frog/type.dart View 10 chunks +65 lines, -44 lines 0 comments Download
M frog/utils.dart View 1 chunk +1 line, -1 line 0 comments Download
M frog/value.dart View 6 chunks +52 lines, -38 lines 0 comments Download
M frog/world.dart View 1 chunk +14 lines, -14 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
Jennifer Messerly
9 years, 1 month ago (2011-11-10 05:43:32 UTC) #1
tbr

Powered by Google App Engine
This is Rietveld 408576698