Skip to content

Use scope instead of std.typecons.scoped #656

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
Merged
WebFreak001 merged 1 commit into from
Oct 10, 2021
Merged

Use scope instead of std.typecons.scoped #656

WebFreak001 merged 1 commit into from
Oct 10, 2021

Conversation

@nordlow
Copy link
Contributor

@nordlow nordlow commented Oct 9, 2021

No description provided.

@WebFreak001
Copy link
Member

WebFreak001 commented Oct 9, 2021

wasn't there something about that you should use scoped!() instead of scope for some reason?

@nordlow
Copy link
Contributor Author

nordlow commented Oct 10, 2021 •
edited
Loading

wasn't there something about that you should use scoped!() instead of scope for some reason?

I see no reason to use scoped in variable declarations as scope is a builtin feature that behaves that same as scoped but without reliance on template bloat and std.typecons (Phobos) and activates escape analysis on declaration when -dip1000 is used.

@WebFreak001 WebFreak001 merged commit 5aba1b1 into dlang-community:master Oct 10, 2021
@nordlow nordlow deleted the use-scope branch October 19, 2021 19:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@WebFreak001 WebFreak001 WebFreak001 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@nordlow @WebFreak001