Skip to content

Issue 13225 - [ICE] Access violation on invalid mixin template instantiation#3840

Merged
WalterBright merged 3 commits intodlang:masterfrom
9rnsr:fix13225
Aug 6, 2014
Merged

Issue 13225 - [ICE] Access violation on invalid mixin template instantiation#3840
WalterBright merged 3 commits intodlang:masterfrom
9rnsr:fix13225

Conversation

@9rnsr
Copy link
Contributor

@9rnsr 9rnsr commented Jul 31, 2014

@yebblies
Copy link
Contributor

I thought FuncDeclarations could never be typed as Terror?

@dnadlinger
Copy link
Contributor

@yebblies: That's what I thought as well, at least I got that impression a while back.

@9rnsr
Copy link
Contributor Author

9rnsr commented Aug 1, 2014

It was changed in #2850.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quotes around the identifier in the diagnostic would help.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, but it should go to in separated PR.

@9rnsr
Copy link
Contributor Author

9rnsr commented Aug 5, 2014

Also fixes regression 13245.

@AndrewEdwards AndrewEdwards added this to the 2.066 milestone Aug 5, 2014
WalterBright added a commit that referenced this pull request Aug 6, 2014
Issue 13225 - [ICE] Access violation on invalid mixin template instantiation
@WalterBright WalterBright merged commit d0d95cd into dlang:master Aug 6, 2014
9rnsr pushed a commit to 9rnsr/dmd that referenced this pull request Aug 6, 2014
Issue 13225 - [ICE] Access violation on invalid mixin template instantiation
@9rnsr 9rnsr deleted the fix13225 branch August 6, 2014 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants