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

Issue 19763007: Use library prefixes in scope of the mixin class when compiling a functions of (Closed)

Created:
7 years, 5 months ago by regis
Modified:
7 years, 5 months ago
Reviewers:
hausner, Ivan Posva
CC:
reviews_dartlang.org, hausner
Visibility:
Public.

Description

Use library prefixes in scope of the mixin class when compiling a functions of a mixin application (issue 11891). Committed: https://code.google.com/p/dart/source/detail?r=25233

Patch Set 1 #

Total comments: 1

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+36 lines, -1 line) Patch
M runtime/vm/parser.cc View 1 1 chunk +8 lines, -1 line 0 comments Download
A tests/language/mixin_prefix_lib.dart View 1 chunk +11 lines, -0 lines 0 comments Download
A tests/language/mixin_prefix_test.dart View 1 chunk +17 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
regis
7 years, 5 months ago (2013-07-18 23:47:00 UTC) #1
hausner
DBC: I wonder why we look up the library prefix through the class and not ...
7 years, 5 months ago (2013-07-19 05:21:58 UTC) #2
regis
On 2013/07/19 05:21:58, hausner wrote: > DBC: I wonder why we look up the library ...
7 years, 5 months ago (2013-07-19 18:08:00 UTC) #3
regis
Committed patchset #2 manually as r25233 (presubmit successful).
7 years, 5 months ago (2013-07-19 18:08:22 UTC) #4
Ivan Posva
LGTMwC -ip https://codereview.chromium.org/19763007/diff/1/runtime/vm/parser.cc File runtime/vm/parser.cc (right): https://codereview.chromium.org/19763007/diff/1/runtime/vm/parser.cc#newcode2801 runtime/vm/parser.cc:2801: if (current_class().mixin() == Type::null()) { Please add ...
7 years, 5 months ago (2013-07-19 19:00:25 UTC) #5
regis
7 years, 5 months ago (2013-07-19 20:48:52 UTC) #6
Message was sent while issue was closed.
On 2013/07/19 19:00:25, Ivan Posva wrote:
> LGTMwC -ip
> 
> https://codereview.chromium.org/19763007/diff/1/runtime/vm/parser.cc
> File runtime/vm/parser.cc (right):
> 
>
https://codereview.chromium.org/19763007/diff/1/runtime/vm/parser.cc#newcode2801
> runtime/vm/parser.cc:2801: if (current_class().mixin() == Type::null()) {
> Please add a TODO to figure out why we are not using the librar() to lookup
here
> and other places below.

Thanks. It was submitted with a TODO.

Powered by Google App Engine
This is Rietveld 408576698