Skip to content

Issue 6359 - Pure/@safe-inference should not be affected by __traits(compiles)#3625

Merged
WalterBright merged 4 commits intodlang:masterfrom
9rnsr:fix6359
Jun 6, 2014
Merged

Issue 6359 - Pure/@safe-inference should not be affected by __traits(compiles)#3625
WalterBright merged 4 commits intodlang:masterfrom
9rnsr:fix6359

Conversation

@9rnsr
Copy link
Contributor

@9rnsr 9rnsr commented Jun 5, 2014

@9rnsr 9rnsr closed this Jun 5, 2014
@9rnsr 9rnsr reopened this Jun 5, 2014
src/expression.c Outdated
Copy link
Member

Choose a reason for hiding this comment

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

This should be redone to be:

if (v->ident == Id::gate)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK, done.

WalterBright added a commit that referenced this pull request Jun 6, 2014
Issue 6359 - Pure/@safe-inference should not be affected by __traits(compiles)
@WalterBright WalterBright merged commit e051d11 into dlang:master Jun 6, 2014
@9rnsr 9rnsr deleted the fix6359 branch June 7, 2014 02:20
Copy link
Contributor

Choose a reason for hiding this comment

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

Wow, what a completely self-explanatory function name. And thankfully, there is even a doc comment to explain what is going on!

@CyberShadow
Copy link
Member

This pull request may have introduced a regression:
https://issues.dlang.org/show_bug.cgi?id=17605

@WalterBright
Copy link
Member

The regression is caused by:

https://github.com/dlang/dmd/pull/3625/files#diff-43282ebf5a2de5fdbcb3b5083ddf949dR1294

where SCOPEcompile is turned off without explanation.

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.

4 participants