Don't report 'no such method' if class defines 'noSuchMethod'

Don't report 'no such method' if class defines 'noSuchMethod'


R=brianwilkerson@google.com
BUG=

Committed: https://code.google.com/p/dart/source/detail?r=12121

[Brian Wilkerson, 2012-09-10 13:43:30]

LGTM as long as this is still spec compliant. I don't have access to the spec at
the moment, but I don't remember it saying that the error should not be reported
under these conditions. Perhaps we should add a preference to suppress this
error under these conditions and default it to the specified behavior.

[scheglov, 2012-09-10 14:52:38]

PTAL

[Brian Wilkerson, 2012-09-10 15:22:56]

LGTM

https://chromiumcodereview.appspot.com/10915171/diff/4001/editor/tools/plugin...
File
editor/tools/plugins/com.google.dart.tools.core/src/com/google/dart/tools/core/utilities/compiler/DartCompilerUtilities.java
(right):

https://chromiumcodereview.appspot.com/10915171/diff/4001/editor/tools/plugin...
editor/tools/plugins/com.google.dart.tools.core/src/com/google/dart/tools/core/utilities/compiler/DartCompilerUtilities.java:486:
false);
Unless I'm confused, this is the inverse of the default for the compiler
settings. Was that intentional (command line reports errors, editor doesn't)?

[scheglov, 2012-09-10 15:40:05]

https://chromiumcodereview.appspot.com/10915171/diff/4001/editor/tools/plugin...
File
editor/tools/plugins/com.google.dart.tools.core/src/com/google/dart/tools/core/utilities/compiler/DartCompilerUtilities.java
(right):

https://chromiumcodereview.appspot.com/10915171/diff/4001/editor/tools/plugin...
editor/tools/plugins/com.google.dart.tools.core/src/com/google/dart/tools/core/utilities/compiler/DartCompilerUtilities.java:486:
false);
On 2012/09/10 15:22:56, Brian Wilkerson wrote:
> Unless I'm confused, this is the inverse of the default for the compiler
> settings. Was that intentional (command line reports errors, editor doesn't)?

Yes, this is correct.
Do according Spec for command line, do what is logical (as it seems everyone
think) in Editor.