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

Issue 1743653002: Stop prefixing the library name to type names when reporting a type error with (Closed)

Created:
4 years, 10 months ago by regis
Modified:
4 years, 10 months ago
Reviewers:
srdjan
CC:
reviews_dartlang.org, vm-dev_dartlang.org
Base URL:
git@github.com:dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Stop prefixing the library name to type names when reporting a type error with identical names. The library name is not always helpful, and the prefix confuses users when the types are function types. Instead, enumerate all appearing types with their URIs. This should help debug issue #27229258. R=srdjan@google.com Committed: https://github.com/dart-lang/sdk/commit/a8d71a461d82a54a5d9a2a7feac4faa74981aafc

Patch Set 1 #

Total comments: 2

Patch Set 2 : enumerate URIs #

Unified diffs Side-by-side diffs Delta from patch set Stats (+146 lines, -34 lines) Patch
M runtime/vm/code_generator.cc View 1 2 chunks +7 lines, -27 lines 0 comments Download
M runtime/vm/object.h View 1 10 chunks +21 lines, -6 lines 0 comments Download
M runtime/vm/object.cc View 1 8 chunks +114 lines, -0 lines 0 comments Download
M runtime/vm/symbols.h View 1 2 chunks +4 lines, -1 line 0 comments Download

Messages

Total messages: 8 (3 generated)
regis
4 years, 10 months ago (2016-02-26 17:32:24 UTC) #2
srdjan
https://codereview.chromium.org/1743653002/diff/1/runtime/vm/code_generator.cc File runtime/vm/code_generator.cc (right): https://codereview.chromium.org/1743653002/diff/1/runtime/vm/code_generator.cc#newcode528 runtime/vm/code_generator.cc:528: String& dst_type_name = String::Handle(dst_type.UserVisibleName()); const String https://codereview.chromium.org/1743653002/diff/1/runtime/vm/code_generator.cc#newcode533 runtime/vm/code_generator.cc:533: } ...
4 years, 10 months ago (2016-02-26 17:54:16 UTC) #3
regis
PTAL
4 years, 10 months ago (2016-02-26 22:18:00 UTC) #5
srdjan
lgtm
4 years, 10 months ago (2016-02-26 22:54:20 UTC) #6
regis
4 years, 10 months ago (2016-02-26 23:12:15 UTC) #8
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
a8d71a461d82a54a5d9a2a7feac4faa74981aafc (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698