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

Issue 10829431: Eliminate imported_into pointers in Library (Closed)

Created:
8 years, 4 months ago by hausner
Modified:
8 years, 4 months ago
Reviewers:
siva
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Eliminate imported_into pointers in Library The imported_into pointers are no longer necessary with the new library scoping rules. Committed: https://code.google.com/p/dart/source/detail?r=10991

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -232 lines) Patch
M runtime/vm/object.h View 3 chunks +0 lines, -17 lines 0 comments Download
M runtime/vm/object.cc View 8 chunks +1 line, -211 lines 0 comments Download
M runtime/vm/raw_object.h View 2 chunks +0 lines, -2 lines 0 comments Download
M runtime/vm/raw_object_snapshot.cc View 2 chunks +0 lines, -2 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
hausner
8 years, 4 months ago (2012-08-20 17:34:11 UTC) #1
siva
lgtm
8 years, 4 months ago (2012-08-20 17:38:38 UTC) #2
hausner
8 years, 4 months ago (2012-08-20 17:53:36 UTC) #3
Thank you. The impact on isolate size is insignificant, about 1/4 KB:

Before:

$ dart --trace-isolates ~/tmp/d.dart
[+] Starting isolate:
	isolate:    d.dart/main-7112
Size of isolate snapshot = 779522
New space (0k of 32768k) Old space (1079k of 1280k) Code space (0k of 0k)


After:

$ dart --trace-isolates ~/tmp/d.dart 
[+] Starting isolate:
	isolate:    d.dart/main-7112
Size of isolate snapshot = 779215
New space (0k of 32768k) Old space (1079k of 1280k) Code space (0k of 0k)

Powered by Google App Engine
This is Rietveld 408576698