Bug 13502 – [REG2.065] Stray '(' warning not emitted for documentation of enum templates

Status
RESOLVED
Resolution
FIXED
Severity
regression
Priority
P1
Component
dmd
Product
D
Version
D2
Platform
All
OS
All
Creation time
2014-09-20T07:19:00Z
Last change time
2015-02-18T03:37:13Z
Keywords
accepts-invalid, ddoc, pull
Assigned to
nobody
Creator
dlang-bugzilla

Comments

Comment #0 by dlang-bugzilla — 2014-09-20T07:19:25Z
/// ( enum isSomeString(T) = true; With -D -w, this should emit a warning. If you remove "(T)", the warning will be emitted.
Comment #1 by dlang-bugzilla — 2014-09-20T08:11:28Z
Comment #2 by k.hara.pg — 2014-09-20T15:55:04Z
(In reply to Vladimir Panteleev from comment #1) > This is a regression. > > Introduced in https://github.com/D-Programming-Language/dmd/pull/2467 The issue occurs only when the optional template parameters exist. So I think we cannot call it a "regression".
Comment #3 by k.hara.pg — 2014-09-20T15:55:51Z
Comment #4 by dlang-bugzilla — 2014-09-20T16:58:08Z
(In reply to Kenji Hara from comment #2) > The issue occurs only when the optional template parameters exist. No, that was another, earlier change. Pull 2467 allowed writing: enum bool isSomeString(T) = true; but this is not what the problem is with. The line: enum isSomeString(T) = true; worked both before and after that change, however the warning was only emitted before that change. So I think this was a regression after all.
Comment #5 by k.hara.pg — 2014-09-21T02:44:56Z
(In reply to Vladimir Panteleev from comment #4) > (In reply to Kenji Hara from comment #2) > > The issue occurs only when the optional template parameters exist. > > No, that was another, earlier change. > > Pull 2467 allowed writing: > > enum bool isSomeString(T) = true; > > but this is not what the problem is with. > > The line: > > enum isSomeString(T) = true; > > worked both before and after that change, however the warning was only > emitted before that change. > > So I think this was a regression after all. Ok, I confirmed the code was working in 2.064. I revert the status to 'regression'.
Comment #6 by github-bugzilla — 2014-09-21T07:33:30Z
Commits pushed to master at https://github.com/D-Programming-Language/dmd https://github.com/D-Programming-Language/dmd/commit/8eb071e3aab4771304088742f945b8b91a16e97e fix Issue 13502 - Stray '(' warning not emitted for documentation of enum templates https://github.com/D-Programming-Language/dmd/commit/3c5d1cd0365891b0a0bc977ab5b8479cd41d065d Merge pull request #4005 from 9rnsr/fix13502 [REG2.065] Issue 13502 - Stray '(' warning not emitted for documentation of enum templates
Comment #7 by github-bugzilla — 2015-02-18T03:37:13Z
Commits pushed to 2.067 at https://github.com/D-Programming-Language/dmd https://github.com/D-Programming-Language/dmd/commit/8eb071e3aab4771304088742f945b8b91a16e97e fix Issue 13502 - Stray '(' warning not emitted for documentation of enum templates https://github.com/D-Programming-Language/dmd/commit/3c5d1cd0365891b0a0bc977ab5b8479cd41d065d Merge pull request #4005 from 9rnsr/fix13502