Skip to content

Comments

use the 'new' alias syntax#4245

Merged
DmitryOlshansky merged 1 commit intodlang:masterfrom
wilzbach:use_new_alias
Apr 27, 2016
Merged

use the 'new' alias syntax#4245
DmitryOlshansky merged 1 commit intodlang:masterfrom
wilzbach:use_new_alias

Conversation

@wilzbach
Copy link
Contributor

AFAICT we should only use one alias syntax in Phobos (and hopefully deprecate the older soon).
This PR deprecates the old alias type identifier in favor of the new alias identifier = type syntax.

@JackStouffer
Copy link
Contributor

LGTM

I can't believe

  1. That there were still this many uses of the old syntax
  2. That the old syntax still isn't deprecated

@wilzbach
Copy link
Contributor Author

wilzbach commented Apr 27, 2016

That there were still this many uses of the old syntax

There's even a bit more at typecons.d, but here it uses a FuncDeclarator. Is there a good way to replace those occurrences?

@DmitryOlshansky
Copy link
Member

Auto-merge toggled on

@andralex
Copy link
Member

noice

@DmitryOlshansky DmitryOlshansky merged commit 0853d4b into dlang:master Apr 27, 2016
@wilzbach wilzbach deleted the use_new_alias branch June 2, 2016 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants