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

Issue 944493002: Change equality comparison for FileBasedSource to use URI. (Closed)

Created:
5 years, 10 months ago by Paul Berry
Modified:
5 years, 10 months ago
Reviewers:
scheglov
CC:
reviews_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Change equality comparison for FileBasedSource to use URI. Previously, we considered two FileBasedSources to be equal if they referred to the same file. As a result, if a given file was accessible via multiple URI's, only one cache entry would be created for the file and we would only resolve it once; the elements created by resolution would be nondeterministic, they would refer to the file via whatever source happened to be used to analyze it first. Worse yet, incremental analysis might cause some of the elements in the file to later be replaced by elements that would refer to the file via a different source, resulting in confusing errors. BUG=dartbug.com/22030 R=scheglov@google.com Committed: https://code.google.com/p/dart/source/detail?r=43888

Patch Set 1 #

Patch Set 2 : Reformat and sort code. #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+27 lines, -1 line) Patch
M pkg/analyzer/lib/src/generated/source_io.dart View 1 chunk +1 line, -1 line 2 comments Download
M pkg/analyzer/test/generated/engine_test.dart View 1 2 chunks +26 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (1 generated)
Paul Berry
5 years, 10 months ago (2015-02-19 18:51:25 UTC) #2
scheglov
LGTM https://codereview.chromium.org/944493002/diff/20001/pkg/analyzer/lib/src/generated/source_io.dart File pkg/analyzer/lib/src/generated/source_io.dart (right): https://codereview.chromium.org/944493002/diff/20001/pkg/analyzer/lib/src/generated/source_io.dart#newcode200 pkg/analyzer/lib/src/generated/source_io.dart:200: object != null && object is FileBasedSource && ...
5 years, 10 months ago (2015-02-19 18:54:47 UTC) #3
Paul Berry
https://codereview.chromium.org/944493002/diff/20001/pkg/analyzer/lib/src/generated/source_io.dart File pkg/analyzer/lib/src/generated/source_io.dart (right): https://codereview.chromium.org/944493002/diff/20001/pkg/analyzer/lib/src/generated/source_io.dart#newcode200 pkg/analyzer/lib/src/generated/source_io.dart:200: object != null && object is FileBasedSource && uri ...
5 years, 10 months ago (2015-02-19 19:15:32 UTC) #4
Paul Berry
5 years, 10 months ago (2015-02-19 19:15:58 UTC) #5
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as 43888 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698