https://issues.dlang.org/show_bug.cgi?id=13502

Vladimir Panteleev <thecybershadow@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|minor                       |regression

--- Comment #1 from Vladimir Panteleev <thecybershadow@gmail.com> ---
This is a regression.

Introduced in https://github.com/D-Programming-Language/dmd/pull/2467

--

https://issues.dlang.org/show_bug.cgi?id=13502

Kenji Hara <k.hara.pg@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|regression                  |normal

--- Comment #2 from Kenji Hara <k.hara.pg@gmail.com> ---
(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".

--

https://issues.dlang.org/show_bug.cgi?id=13502

Kenji Hara <k.hara.pg@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |pull

--- Comment #3 from Kenji Hara <k.hara.pg@gmail.com> ---
https://github.com/D-Programming-Language/dmd/pull/4005

--

https://issues.dlang.org/show_bug.cgi?id=13502

--- Comment #4 from Vladimir Panteleev <thecybershadow@gmail.com> ---
(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.

--

https://issues.dlang.org/show_bug.cgi?id=13502

Kenji Hara <k.hara.pg@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Stray '(' warning not       |[REG2.065] Stray '('
                   |emitted for documentation   |warning not emitted for
                   |of enum templates           |documentation of enum
                   |                            |templates
           Severity|normal                      |regression

--- Comment #5 from Kenji Hara <k.hara.pg@gmail.com> ---
(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'.

--

https://issues.dlang.org/show_bug.cgi?id=13502

--- Comment #6 from github-bugzilla@puremagic.com ---
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

--

https://issues.dlang.org/show_bug.cgi?id=13502

github-bugzilla@puremagic.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--

https://issues.dlang.org/show_bug.cgi?id=13502

--- Comment #7 from github-bugzilla@puremagic.com ---
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

--