Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Undeclared static setter calls: Compile time error becomes static warning per spec, 10.16 #3093

Closed
gbracha opened this issue May 16, 2012 · 10 comments
Closed

Undeclared static setter calls: Compile time error becomes static warning per spec, 10.16 #3093

gbracha opened this issue May 16, 2012 · 10 comments
Labels
area-analyzer Use area-analyzer for Dart analyzer issues, including the analysis server and code completion. closed-duplicate Closed in favor of an existing report
Milestone
M1

Comments

@gbracha
Copy link
Contributor

gbracha commented May 16, 2012

Spec section 10.16 has changed its handling of static setters (assignment)

It is a static warning if there is no class C in the enclosing lexical scope of the assignment, or if C does not declare, implicitly or explicitly, a setter v.

So, what was once a compile-time error is now a warning.
@danrubel
Copy link

danrubel commented Jun 4, 2012

Added this to the M1 milestone.

@scheglov
Copy link
Contributor

scheglov commented Jun 6, 2012

Set owner to @scheglov.
Added Accepted label.

@scheglov
Copy link
Contributor

scheglov commented Jun 6, 2012

Issue #3087 has been merged into this issue.

@scheglov
Copy link
Contributor

scheglov commented Jun 6, 2012

CL for DartC/Editor is ready, but waiting for VM.

Marked this as being blocked by #3094.

@bwilkerson
Copy link
Member

Added Waiting label.

@scheglov
Copy link
Contributor

Removed the owner.
Marked this as being blocked by #3094.
Unmarked this as being blocked by #3094.

@bwilkerson
Copy link
Member

Added Triaged label.

@scheglov
Copy link
Contributor

Will be solved together with 3084.

Added Duplicate label.
Marked as being merged into #3084.

@iposva-google
Copy link
Contributor

Marked this as blocking #3094.

@kasperl
Copy link

kasperl commented Sep 3, 2012

Marked this as blocking #3095.

@gbracha gbracha added Type-Defect area-analyzer Use area-analyzer for Dart analyzer issues, including the analysis server and code completion. closed-duplicate Closed in favor of an existing report labels Sep 3, 2012
@gbracha gbracha added this to the M1 milestone Sep 3, 2012
@scheglov scheglov mentioned this issue Jun 3, 2015
Closed
@gbracha gbracha mentioned this issue Jun 5, 2015
Closed
dart-bot pushed a commit that referenced this issue Sep 13, 2021
@sigurdm
Roll pub
a704116 
New commits in this version:

git -C third_party/pkg/pub log --oneline cd7a43f2109f7e5eb22e73c7f4e15d25fd57598e..d95c5713dda518ed53ada70e00789e6aadbfbe48
d95c5713 (HEAD, origin/master, origin/HEAD) Remove duplicate global invocation on Windows (#3055)
4c5198df master->main (#3101)
e793fd52 More tool/test.dart tweaks (#3097)
1b228edd Report retracted packages (#3093)
4fedb6c5 Tweak strict dependencies error message for `_validateBenchmarkTestTool` (#3087)
e608ab6e Improve test script (#3091)
abf702c4 Detect potential leaks in `dart pub publish`. (#3049)
9941c1f9 Fix broken simplification of prerelease constraints (#3078)
f0cdfa27 format (#3086)
5a1242c6 Fix unicode terminal detection windows (#2933)
58e2296d Dart format (#3084)
1426601c use incremental compilation in the tool/test.dart script (#3075)
9954f851 Fix a typo (#3062)
44489b31 Use relative import for path.dart (#2959)
77702ab1 Always precompile scripts before running them (#3074)

Change-Id: I913ab9e3b5bc7181d483a2de96ec4ad917028b75
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/213262
Reviewed-by: Jonas Jensen <jonasfj@google.com>
Commit-Queue: Sigurd Meldgaard <sigurdm@google.com>
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area-analyzer Use area-analyzer for Dart analyzer issues, including the analysis server and code completion. closed-duplicate Closed in favor of an existing report
Projects
None yet
Milestone
M1
Development

No branches or pull requests
6 participants
@scheglov @kasperl @danrubel @gbracha @iposva-google @bwilkerson