[Issue 13601] static if (__ctfe) should emit warning

2014-10-22 Thread via Digitalmars-d-bugs 
https://issues.dlang.org/show_bug.cgi?id=13601

--- Comment #7 from Ketmar Dark  ---
sorry for the noise, it seems that i was unintentionally offensive once again
in comment #2. what i really wanted to say is that "my patch is in no way ready
to go to official repository, it's just a crude hack. use it on your own risk
and only if you want to have some diagnostics right away without waiting until
someone will write a proper patch".

i'm sorry.

--

[Issue 13601] static if (__ctfe) should emit warning

2014-10-19 Thread via Digitalmars-d-bugs 
https://issues.dlang.org/show_bug.cgi?id=13601

--- Comment #6 from Martin Nowak  ---
Mmh, this happens because the static if expression is now handled by CTFE
instead of constant folding.
CTFE would need to look at the scope flags to check whether __ctfe can be read.

--

[Issue 13601] static if (__ctfe) should emit warning

2014-10-19 Thread via Digitalmars-d-bugs 
https://issues.dlang.org/show_bug.cgi?id=13601

--- Comment #4 from Martin Nowak  ---
This commit
https://github.com/D-Programming-Language/dmd/commit/021097056744c23231427b71e65bd04abc72d3e3
added a diagnostic regeression, making static if (__ctfe) print two errors.

bug.d(3): Error: variable __ctfe forward referenced
bug.d(3): Error: variable __ctfe cannot be read at compile time

--

[Issue 13601] static if (__ctfe) should emit warning

2014-10-19 Thread via Digitalmars-d-bugs 
https://issues.dlang.org/show_bug.cgi?id=13601

Martin Nowak  changed:

   What|Removed |Added

   Priority|P1  |P3
 CC||c...@dawg.eu
   Severity|enhancement |regression

--- Comment #3 from Martin Nowak  ---
This worked as expected 2.063.2.

cat > bug.d << CODE
void foo()
{
static if (__ctfe) {}
}
CODE

dmd -c bug

bug.d(3): Error: variable __ctfe cannot be read at compile time


--

[Issue 13601] static if (__ctfe) should emit warning

2014-10-19 Thread via Digitalmars-d-bugs 
https://issues.dlang.org/show_bug.cgi?id=13601

--- Comment #5 from Martin Nowak  ---
This commit introduced the regression.
https://github.com/D-Programming-Language/dmd/commit/43a6c87194cae799650249b10a4f7c910081d280

--

[Issue 13601] static if (__ctfe) should emit warning

2014-10-11 Thread via Digitalmars-d-bugs 
https://issues.dlang.org/show_bug.cgi?id=13601

Ketmar Dark  changed:

   What|Removed |Added

 CC||ket...@ketmar.no-ip.org

--- Comment #2 from Ketmar Dark  ---
that's only if the author wants to go to mainline. me not. i don't care anymore
if my code will lang into the oficial branch or not. it's just a patch for
those who interested to patch dmd manually and either trust me or will write
test themselves.

--

[Issue 13601] static if (__ctfe) should emit warning

2014-10-10 Thread via Digitalmars-d-bugs 
https://issues.dlang.org/show_bug.cgi?id=13601

Dicebot  changed:

   What|Removed |Added

 CC||pub...@dicebot.lv

--- Comment #1 from Dicebot  ---
Any DMD changes need relevant test cases added

--