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

Issue 9421001: Issue 1606/1108. The 'native' keyword is supposed to only be allowed in core libraries. (Closed)

Created:
8 years, 10 months ago by scheglov
Modified:
8 years, 10 months ago
Reviewers:
codefu, zundel
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Patch Set 1 #

Messages

Total messages: 6 (0 generated)
scheglov
8 years, 10 months ago (2012-02-17 16:27:33 UTC) #1
codefu
What's the time line on removing dartc's backend? If its fully removed, we shouldn't seen ...
8 years, 10 months ago (2012-02-17 16:37:33 UTC) #2
scheglov
On 2012/02/17 16:37:33, codefu wrote: > What's the time line on removing dartc's backend? If ...
8 years, 10 months ago (2012-02-17 17:08:08 UTC) #3
codefu
On 2012/02/17 17:08:08, scheglov wrote: > On 2012/02/17 16:37:33, codefu wrote: > > What's the ...
8 years, 10 months ago (2012-02-17 17:38:32 UTC) #4
scheglov
On 2012/02/17 17:38:32, codefu wrote: > On 2012/02/17 17:08:08, scheglov wrote: > > On 2012/02/17 ...
8 years, 10 months ago (2012-02-17 17:52:14 UTC) #5
codefu
8 years, 10 months ago (2012-02-17 17:53:05 UTC) #6
On 2012/02/17 17:52:14, scheglov wrote:
> On 2012/02/17 17:38:32, codefu wrote:
> > On 2012/02/17 17:08:08, scheglov wrote:
> > > On 2012/02/17 16:37:33, codefu wrote:
> > > > What's the time line on removing dartc's backend?  If its fully removed,
> we
> > > > shouldn't seen the 'native' keyword at all.
> > > 
> > > March 1-st.
> > > 
> > > We still parse/analyze corelib itself, so we still may see "native".
> > 
> > What is the point in corelib having any natives?  The implementations should
> all
> > be empty shells.
> 
> Well, for now corelibs for DartC and Frog import implementation libraries.
> When we will see different situation, we will able to remove "native" support
at
> all.

LGTM

Powered by Google App Engine
This is Rietveld 408576698