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

Issue 1686613002: Use different SDKs based on option settings (Closed)

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

Description

We need to perform analysis differently based on some option settings (such as strong mode), but there is currently a single SDK that is shared across all contexts no matter which option values are being used. This causes us to inconsistently analyze different portions of the SDK depending on which outer context was the first to request the analysis. This is at least a first step toward solving this problem by allowing multiple SDKs to be created, one for each unique set of options. R=paulberry@google.com, scheglov@google.com Committed: https://github.com/dart-lang/sdk/commit/bdd0d637fed7b4bfc4004e08e9f9c8428c3e8022

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+323 lines, -63 lines) Patch
M pkg/analysis_server/lib/src/analysis_server.dart View 10 chunks +45 lines, -17 lines 0 comments Download
M pkg/analysis_server/lib/src/context_manager.dart View 5 chunks +38 lines, -14 lines 0 comments Download
M pkg/analysis_server/lib/src/server/driver.dart View 2 chunks +19 lines, -7 lines 0 comments Download
M pkg/analysis_server/lib/src/socket_server.dart View 4 chunks +9 lines, -1 line 0 comments Download
M pkg/analysis_server/test/analysis/set_priority_files_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M pkg/analysis_server/test/analysis_abstract.dart View 1 chunk +1 line, -1 line 0 comments Download
M pkg/analysis_server/test/analysis_server_test.dart View 3 chunks +13 lines, -9 lines 0 comments Download
M pkg/analysis_server/test/context_manager_test.dart View 2 chunks +6 lines, -1 line 0 comments Download
M pkg/analysis_server/test/domain_analysis_test.dart View 2 chunks +2 lines, -2 lines 0 comments Download
M pkg/analysis_server/test/domain_diagnostic_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M pkg/analysis_server/test/domain_execution_test.dart View 4 chunks +8 lines, -6 lines 0 comments Download
M pkg/analysis_server/test/domain_server_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M pkg/analysis_server/test/socket_server_test.dart View 2 chunks +5 lines, -1 line 0 comments Download
M pkg/analyzer/lib/src/generated/engine.dart View 3 chunks +44 lines, -0 lines 0 comments Download
M pkg/analyzer/lib/src/generated/sdk.dart View 2 chunks +55 lines, -1 line 1 comment Download
M pkg/analyzer/lib/src/generated/utilities_general.dart View 1 chunk +18 lines, -0 lines 0 comments Download
M pkg/analyzer/lib/src/task/options.dart View 3 chunks +57 lines, -0 lines 0 comments Download

Messages

Total messages: 8 (4 generated)
Brian Wilkerson
4 years, 10 months ago (2016-02-09 17:30:24 UTC) #3
Paul Berry
lgtm https://codereview.chromium.org/1686613002/diff/1/pkg/analyzer/lib/src/generated/sdk.dart File pkg/analyzer/lib/src/generated/sdk.dart (right): https://codereview.chromium.org/1686613002/diff/1/pkg/analyzer/lib/src/generated/sdk.dart#newcode88 pkg/analyzer/lib/src/generated/sdk.dart:88: * Manages the DartSdk's that Incomplete comment
4 years, 10 months ago (2016-02-09 18:21:18 UTC) #5
scheglov
LGTM
4 years, 10 months ago (2016-02-09 18:36:12 UTC) #6
Brian Wilkerson
4 years, 10 months ago (2016-02-09 19:05:40 UTC) #8
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
bdd0d637fed7b4bfc4004e08e9f9c8428c3e8022 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698