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 Issue 23205 - Can't declare mixin template inside a function #14240

Merged
merged 1 commit into from
Jun 23, 2022
Merged

Fix Issue 23205 - Can't declare mixin template inside a function #14240

merged 1 commit into from
Jun 23, 2022

Conversation

BorisCarvajal
Copy link
Member

No description provided.

@dlang-bot
Copy link
Contributor

Thanks for your pull request, @BorisCarvajal!

Bugzilla references

Auto-close Bugzilla Severity Description
✓ 23205 normal Can't declare mixin template inside a function

Testing this PR locally

If you don't have a local development environment setup, you can use Digger to test this PR:

dub run digger -- build "master + dmd#14240"

@dlang-bot dlang-bot merged commit 1e43256 into dlang:master Jun 23, 2022
tim-dlang added a commit to tim-dlang/dlang.org that referenced this pull request May 23, 2023
@tim-dlang
Move Template(Mixin)Declaration to Declaration in grammar
96b59ee 
This allows to declare them inside functions.
For mixin templates this is supported by DMD since dlang/dmd#14240
Declaring normal templates in functions is also supported by DMD.
@tim-dlang tim-dlang mentioned this pull request May 23, 2023
Merged
dkorpel pushed a commit to dlang/dlang.org that referenced this pull request Jun 26, 2023
@tim-dlang
Move Template(Mixin)Declaration to Declaration in grammar (#3614)
b705f73 
This allows to declare them inside functions.
For mixin templates this is supported by DMD since dlang/dmd#14240
Declaring normal templates in functions is also supported by DMD.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thewilsonator thewilsonator thewilsonator approved these changes

@maxhaton maxhaton maxhaton approved these changes

Assignees
No one assigned
Labels
Merge:auto-merge Severity:Bug Fix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@BorisCarvajal @dlang-bot @maxhaton @thewilsonator