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

Issue 2110693002: Remove "_finished" assertion checks from summary codegen. (Closed)

Created:
4 years, 5 months ago by Paul Berry
Modified:
4 years, 5 months ago
Reviewers:
Brian Wilkerson
CC:
reviews_dartlang.org
Base URL:
git@github.com:dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Remove "_finished" assertion checks from summary codegen. These assertions were helpful back when the client manually managed offsets. Now that offsets are managed automatically by the toBuffer() logic, they don't carry any benefit. Removing these assertions will pave the way for allowing a summary to be created, serialized, modified, and then serialized again, which we will need to do to efficiently generate both "full" and "semantic" summaries. R=brianwilkerson@google.com Committed: https://github.com/dart-lang/sdk/commit/efe5a31421f04e5c9250779f7666f71fdd9ffbf3

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -372 lines) Patch
M pkg/analyzer/lib/src/summary/format.dart View 299 chunks +0 lines, -367 lines 0 comments Download
M pkg/analyzer/tool/summary/generate.dart View 3 chunks +0 lines, -5 lines 0 comments Download

Messages

Total messages: 5 (2 generated)
Paul Berry
4 years, 5 months ago (2016-06-28 22:30:34 UTC) #2
Brian Wilkerson
lgtm
4 years, 5 months ago (2016-06-28 23:20:28 UTC) #3
Paul Berry
4 years, 5 months ago (2016-06-28 23:23:01 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
efe5a31421f04e5c9250779f7666f71fdd9ffbf3 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698