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

Fix nested cfg_if if-statement not to hang #6251

Closed
wants to merge 1 commit into from
Closed

Fix nested cfg_if if-statement not to hang #6251

wants to merge 1 commit into from

Conversation

youknowone
Copy link

Fix #4442

@ytmimi
Copy link
Contributor

ytmimi commented Jul 25, 2024

Thanks, but I'm going to close this in favor of #5341

@ytmimi ytmimi closed this Jul 25, 2024
@youknowone
Copy link
Author

youknowone commented Jul 26, 2024 •
edited
Loading

@ytmimi I am glad there is a working patch. But the patch is working in process for more than 2 years.
Unless it has plan to be merged in short time, could you consider to merge this one first to prevent the infinite memory consumption at the moment and later overwrite it with #5341 once it is done?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
pr-not-reviewed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

rusfmt hangs when using cfg-if macro
3 participants
@youknowone @ytmimi @rustbot