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

Dart2js lazy loading only checks for DeferredLibrary in the main script #10586

Closed
alan-knight opened this issue May 10, 2013 · 8 comments
Closed

Dart2js lazy loading only checks for DeferredLibrary in the main script #10586

alan-knight opened this issue May 10, 2013 · 8 comments
Labels
closed-obsolete Closed as the reported issue is no longer relevant web-dart2js

Comments

@alan-knight
Copy link
Contributor

If I have a library which in turn wants to load libraries lazily, it's ignored.

My specific use case is with code generated for message catalogs, where I'd like to be able to lazily load only the one(s) in use.
@peter-ahe-google
Copy link
Contributor

The current behavior is intentional, but we can discuss if it the right behavior. I think you have a strong case against the current behavior.

The motivation for the current behavior is that the author of an application should be in control of what is loaded lazily. Only looking for annotations in the main script is one way to put the author in control.

Perhaps a better way to put the author in control is to have an annotation you can use in the main script to override if other libraries are lazy.

@kasperl
Copy link

kasperl commented May 23, 2013

Added this to the Later milestone.

@kasperl
Copy link

kasperl commented May 23, 2013

Added TriageForM5 label.

@kasperl
Copy link

kasperl commented May 28, 2013

Removed TriageForM5 label.

@sethladd
Copy link
Contributor

cc @sigurdm.

@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.

@floitschG
Copy link
Contributor

This limitation has been removed.
Please reopen if you still see issues.

Added AssumedStale label.

@alan-knight alan-knight added Type-Defect web-dart2js closed-obsolete Closed as the reported issue is no longer relevant labels Sep 8, 2014
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
closed-obsolete Closed as the reported issue is no longer relevant web-dart2js
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@sethladd @kasperl @alan-knight @peter-ahe-google @floitschG