← Back to index | Original Bugzilla link

Bug 9018 – __traits(compiles, ...) is true on second check for same incompilable code

Status
RESOLVED
Resolution
FIXED
Severity
normal
Priority
P2
Component
dmd
Product
D
Version
D2
Platform
All
OS
All
Creation time
2012-11-13T12:22:00Z
Last change time
2013-01-12T20:32:10Z
Keywords
pull, rejects-valid
Assigned to
nobody
Creator
verylonglogin.reg

Comments

Comment #0 by verylonglogin.reg — 2012-11-13T12:22:49Z
--- template Inst(alias Template, T) { alias Template!T Inst; } template Template(T) { enum Template = T; } static assert(!__traits(compiles, Inst!(Template, int))); // Assert passes static assert(!__traits(compiles, Inst!(Template, int))); // Assert fails ---
Comment #1 by k.hara.pg — 2012-11-15T09:17:32Z
https://github.com/D-Programming-Language/dmd/pull/1294
Comment #2 by github-bugzilla — 2012-11-16T01:48:31Z
Commits pushed to master at https://github.com/D-Programming-Language/dmd https://github.com/D-Programming-Language/dmd/commit/f60d9238e261c756caded36012f0415710a0b63a fix Issue 9018 - __traits(compiles, ...) is true on second check for same incompilable code https://github.com/D-Programming-Language/dmd/commit/f968f3c29d10d537d1f9f039151eed6df70d5fe5 Merge pull request #1294 from 9rnsr/fix9018 Issue 9018 - __traits(compiles, ...) is true on second check for same incompilable code
Comment #3 by yebblies — 2013-01-12T20:10:22Z
Fixed?
Comment #4 by andrej.mitrovich — 2013-01-12T20:11:09Z
(In reply to comment #3) > Fixed? Shouldn't closing be automatic? I don't understand why it doesn't happen anymore..
Comment #5 by yebblies — 2013-01-12T20:20:10Z
(In reply to comment #4) > (In reply to comment #3) > > Fixed? > > Shouldn't closing be automatic? I don't understand why it doesn't happen > anymore.. It was never automatic. The commits are not always complete fixes, so it probably shouldn't be automatic.
Comment #6 by andrej.mitrovich — 2013-01-12T20:24:03Z
(In reply to comment #5) > It was never automatic. The commits are not always complete fixes, so it probably shouldn't be automatic. That makes sense. I've misinterpreted what it meant to add 'fix issue ####' in the commit message.
Comment #7 by andrej.mitrovich — 2013-01-12T20:32:10Z
Seems to be fixed in 2.061 release.