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.

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

Tear-off closures of deferred functions are assumed to be non-null #11137

Closed
peter-ahe-google opened this issue Jun 7, 2013 · 7 comments
Closed

Tear-off closures of deferred functions are assumed to be non-null #11137

peter-ahe-google opened this issue Jun 7, 2013 · 7 comments
Assignees
@peter-ahe-google
Labels
area-web Use area-web for Dart web related issues, including the DDC and dart2js compilers and JS interop. P2 A bug or feature request we're likely to work on type-bug Incorrect behavior (everything from a crash to more subtle misbehavior) web-dart2js

Comments

@peter-ahe-google
Copy link
Contributor

Type inference, tear-off closures of static/top-level functions, and deferred loading doesn't mix.

See readFoo in tests/compiler/dart2js_extra/deferred/deferred_function_test.dart.
@kasperl
Copy link

kasperl commented Jun 10, 2013

Added this to the M6 milestone.
Removed Priority-Unassigned label.
Added Priority-Medium label.

@larsbak
Copy link

larsbak commented Aug 28, 2013

Removed this from the M6 milestone.
Added this to the M7 milestone.

@peter-ahe-google
Copy link
Contributor Author

Marked this as blocking #3940.

@kasperl
Copy link

kasperl commented Sep 25, 2013

Removed this from the M7 milestone.
Added this to the Later milestone.

@kasperl
Copy link

kasperl commented Jul 10, 2014

Removed this from the Later milestone.
Added Oldschool-Milestone-Later label.

@kasperl
Copy link

kasperl commented Aug 4, 2014

Removed Oldschool-Milestone-Later label.

@peter-ahe-google peter-ahe-google self-assigned this Aug 4, 2014
@kevmoo kevmoo added P2 A bug or feature request we're likely to work on type-bug Incorrect behavior (everything from a crash to more subtle misbehavior) and removed triaged labels Feb 29, 2016
@vsmenon vsmenon added the area-web Use area-web for Dart web related issues, including the DDC and dart2js compilers and JS interop. label Jul 20, 2019
@rakudrama
Copy link
Member

I think this is an obsolete issue.

Static method tearoff getters are in the same deferred unit as the method.
There will be a deferred library guard before the access, so the access cannot return null.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@peter-ahe-google peter-ahe-google

Labels
area-web Use area-web for Dart web related issues, including the DDC and dart2js compilers and JS interop. P2 A bug or feature request we're likely to work on type-bug Incorrect behavior (everything from a crash to more subtle misbehavior) web-dart2js
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@kevmoo @rakudrama @vsmenon @kasperl @peter-ahe-google @larsbak