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

Issue 10857029: Revert "Sharing of sources for corelib date implementation between VM and" (Closed)

Created:
8 years, 4 months ago by Mads Ager (google)
Modified:
8 years, 4 months ago
Reviewers:
Anders Johnsen
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Revert "Sharing of sources for corelib date implementation between VM and" Revert "Fix create_sdk." Revert "Fix bad merge. 'unified' directory is gone." We need to update dartc to not check that final fields are initialized for 'external' constructors (it does not have the body and initializers for those). R=ajohnsen@google.com BUG= Committed: https://code.google.com/p/dart/source/detail?r=10797

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+202 lines, -556 lines) Patch
M corelib/src/implementation/corelib_impl_sources.gypi View 1 chunk +0 lines, -1 line 0 comments Download
D corelib/src/implementation/date.dart View 1 chunk +0 lines, -184 lines 0 comments Download
A + corelib/unified/core/date.dart View 0 chunks +-1 lines, --1 lines 0 comments Download
A + corelib/unified/core/implementation/date.dart View 4 chunks +5 lines, -8 lines 0 comments Download
M lib/compiler/implementation/lib/core.dart View 1 chunk +1 line, -1 line 0 comments Download
M lib/compiler/implementation/lib/coreimpl.dart View 1 chunk +1 line, -1 line 0 comments Download
M lib/compiler/implementation/lib/coreimpl_patch.dart View 1 chunk +0 lines, -7 lines 0 comments Download
A + runtime/lib/date.dart View 9 chunks +192 lines, -27 lines 0 comments Download
D runtime/lib/date_patch.dart View 1 chunk +0 lines, -327 lines 0 comments Download
M runtime/lib/lib_impl_sources.gypi View 1 chunk +1 line, -1 line 0 comments Download
M tools/create_sdk.py View 1 chunk +3 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Mads Ager (google)
8 years, 4 months ago (2012-08-16 07:39:40 UTC) #1
Anders Johnsen
8 years, 4 months ago (2012-08-16 07:40:06 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698