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

Issue 11415291: Issue 7128. Don't show problems in dart:core (Closed)

Created:
8 years ago by scheglov
Modified:
8 years ago
Reviewers:
Brian Wilkerson
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Issue 7128. Don't show problems in dart:core R=brianwilkerson@google.com BUG= Committed: https://code.google.com/p/dart/source/detail?r=15701

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -0 lines) Patch
M compiler/java/com/google/dart/compiler/DartCompilerMainContext.java View 2 chunks +7 lines, -0 lines 2 comments Download

Messages

Total messages: 3 (0 generated)
scheglov
8 years ago (2012-12-04 18:44:59 UTC) #1
Brian Wilkerson
LGTM https://codereview.chromium.org/11415291/diff/1/compiler/java/com/google/dart/compiler/DartCompilerMainContext.java File compiler/java/com/google/dart/compiler/DartCompilerMainContext.java (right): https://codereview.chromium.org/11415291/diff/1/compiler/java/com/google/dart/compiler/DartCompilerMainContext.java#newcode61 compiler/java/com/google/dart/compiler/DartCompilerMainContext.java:61: if (Elements.isCoreLibrarySource(event.getSource())) { Hm. I thought we had ...
8 years ago (2012-12-04 19:14:14 UTC) #2
scheglov
8 years ago (2012-12-04 19:42:15 UTC) #3
https://codereview.chromium.org/11415291/diff/1/compiler/java/com/google/dart...
File compiler/java/com/google/dart/compiler/DartCompilerMainContext.java
(right):

https://codereview.chromium.org/11415291/diff/1/compiler/java/com/google/dart...
compiler/java/com/google/dart/compiler/DartCompilerMainContext.java:61: if
(Elements.isCoreLibrarySource(event.getSource())) {
On 2012/12/04 19:14:15, Brian Wilkerson wrote:
> Hm. I thought we had a request from the core library team to *not* hide these
> errors. Am I mis-remembering? If not, perhaps we need a command-line flag to
> control it (defaulting, I would guess, to hide such errors).

We show the same problems like "Expected constant expression of type num, got
int", when we analyze from Editor and from comment line. So, there are no much
reasons to show them anywhere.

Powered by Google App Engine
This is Rietveld 408576698