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

Mike <slavo5150@yahoo.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |slavo5150@yahoo.com

--- Comment #1 from Mike <slavo5150@yahoo.com> ---
A prerequisite to this would be to replace all occurrences of the old syntax in DRuntime and Phobos with the new syntax.

--

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

--- Comment #2 from bearophile_hugs@eml.cc ---
(In reply to Mike from comment #1)
> A prerequisite to this would be to replace all occurrences of the old syntax in DRuntime and Phobos with the new syntax.

Right. And the warning will help find all those occurrences.

--

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

brian-schott@cox.net changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |brian-schott@cox.net

--- Comment #3 from brian-schott@cox.net ---
"dscanner --styleCheck" will find them for you.

https://github.com/Hackerpilot/Dscanner/

--

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

Orvid King <blah38621@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |blah38621@gmail.com

--- Comment #4 from Orvid King <blah38621@gmail.com> ---
The issue with doing this is that there are still quite a few syntax constructs that the new syntax doesn't currently support, including alias this.

--

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

--- Comment #5 from bearophile_hugs@eml.cc ---
(In reply to Orvid King from comment #4)

> The issue with doing this is that there are still quite a few syntax constructs that the new syntax doesn't currently support, including alias this.

Yes, "alias this" is essentially a different syntax, and it will be kept, as said in the first message of this thread.

What other constructs are a problem for the warn against the old alias syntax?

--

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

Andrej Mitrovic <andrej.mitrovich@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |andrej.mitrovich@gmail.com

--- Comment #6 from Andrej Mitrovic <andrej.mitrovich@gmail.com> ---
(In reply to bearophile_hugs from comment #5)
> What other constructs are a problem for the warn against the old alias syntax?

Function pointers. There's an issue where 'alias f = extern(C) void function()'
does not work (it's a filed bug). Until that is fixed we can't plan to go
forward with this warning.

I would also argue that we should start developing a D porting tool that auto-converts the old alias syntax to the new one. It should be trivial to implement and should get rid of the nuisance of manually fixing code.

--

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

yebblies <yebblies@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |yebblies@gmail.com
           Severity|normal                      |enhancement

--- Comment #7 from yebblies <yebblies@gmail.com> ---
This is an enhancement, as it is currently working as designed.

--

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Depends on|                            |12802

--- Comment #8 from Kenji Hara <k.hara.pg@gmail.com> ---
(In reply to Andrej Mitrovic from comment #6)
> (In reply to bearophile_hugs from comment #5)
> > What other constructs are a problem for the warn against the old alias syntax?
> 
> Function pointers. There's an issue where 'alias f = extern(C) void
> function()' does not work (it's a filed bug). Until that is fixed we can't
> plan to go forward with this warning.

I filed it as the issue 12802.

--

https://issues.dlang.org/show_bug.cgi?id=12615
Issue 12615 depends on issue 12802, which changed state.

Issue 12802 Summary: Allow optional 'StorageClasses' for new alias syntax https://issues.dlang.org/show_bug.cgi?id=12802

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

--

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

--- Comment #9 from Sobirari Muhomori <dfj1esp02@sneakemail.com> ---
Probably leave multialias declaration valid, it's quite handy: alias HANDLE BCRYPT_HANDLE, BCRYPT_ALG_HANDLE, BCRYPT_KEY_HANDLE, BCRYPT_HASH_HANDLE, BCRYPT_SECRET_HANDLE;

--