Skip to content

Refactor RustcTargetData to do lazy loading of targets information #9049

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

Closed

Conversation

Ekleog
Copy link
Contributor

@Ekleog Ekleog commented Jan 6, 2021

I gave up midway through this refactoring, it looks way too big for what it's supposed to do.

More discussion on what to actually do will probably be available at #9030 (comment)

@rust-highfive
Copy link

r? @alexcrichton

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 6, 2021
@Ekleog Ekleog marked this pull request as draft January 6, 2021 01:55
@alexcrichton
Copy link
Member

Without diving into the depths of this refactoring, which unfortunately I don't have time for right now, I can't really help much personally taking this to a full-working state.

@Ekleog
Copy link
Contributor Author

Ekleog commented Jan 14, 2021

Got it, thank you for both this and your comment on the other PR! I'll close this and go with the “add something that allows adding kinds as they appear” route, as it looks like it can create a much better API than sparkling IO access and fallibility literally everywhere :)

@Ekleog Ekleog closed this Jan 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants