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

Issue 2391783002: Further cleanups to annotation elements now that #25706 is fixed. (Closed)

Created:
4 years, 2 months ago by Paul Berry
Modified:
4 years, 2 months ago
CC:
reviews_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Further cleanups to annotation elements now that #25706 is fixed. This CL fixes the following issues: - In an annotation like @prefix.ClassName.foo or @prefix.ClassName.foo(), if "foo" couldn't be resolved, the class element would be returned as the annotation element. - ConstantAstCloner wasn't properly cloning annotation elements, leading to weird issues in summary resynthesis tests. - The summary resynthesizer wasn't setting AnnotationImpl.element. R=brianwilkerson@google.com, scheglov@google.com Committed: https://github.com/dart-lang/sdk/commit/7c666425a1297307026fe7361e664cead46a2051

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -15 lines) Patch
M pkg/analyzer/lib/src/dart/ast/ast.dart View 1 chunk +1 line, -1 line 0 comments Download
M pkg/analyzer/lib/src/dart/constant/utilities.dart View 1 chunk +7 lines, -0 lines 0 comments Download
M pkg/analyzer/lib/src/summary/resynthesize.dart View 1 chunk +2 lines, -1 line 0 comments Download
M pkg/analyzer/test/src/summary/resynthesize_common.dart View 2 chunks +1 line, -13 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
Paul Berry
4 years, 2 months ago (2016-10-03 21:46:28 UTC) #2
Brian Wilkerson
lgtm
4 years, 2 months ago (2016-10-03 21:53:20 UTC) #3
scheglov
LGTM
4 years, 2 months ago (2016-10-03 21:55:26 UTC) #4
Paul Berry
4 years, 2 months ago (2016-10-03 22:00:41 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
7c666425a1297307026fe7361e664cead46a2051 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698