Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement DIP42 - parameterized enum and alias declaration syntax #2368

Merged
merged 1 commit into from Jul 26, 2013

Conversation

@9rnsr
Copy link
Member

commented Jul 20, 2013

http://wiki.dlang.org/DIP42

I think alias declaration should also support additional template parameters for consistency.

The declarations allSatisfy and anySatisfy in the test case are not workable, because issue 9073 (#1325) is not yet fixed.

@9rnsr

This comment has been minimized.

Copy link
Member Author

commented Jul 20, 2013

Note that, I don't support template constraint combination.

enum ident(T) if (constraint) = ...;  // not supported

This just comes from my sense of balance - DIP42 is merely syntactic sugar support. So increase its complexity would decrease cost/benefit ratio.

@WalterBright

This comment has been minimized.

Copy link
Member

commented Jul 21, 2013

@andralex please weigh in on this!

@andralex

This comment has been minimized.

Copy link
Member

commented Jul 26, 2013

LGTM. Congrats @9rnsr!

@andralex

This comment has been minimized.

Copy link
Member

commented Jul 26, 2013

We should task the documentation writeup.

andralex added a commit that referenced this pull request Jul 26, 2013
Merge pull request #2368 from 9rnsr/impl_dip42
Implement DIP42 - parameterized enum and alias declaration syntax

@andralex andralex merged commit 4d06ae4 into dlang:master Jul 26, 2013

1 check was pending

default Pass: 8, Pending: 2
Details
@9rnsr

This comment has been minimized.

Copy link
Member Author

commented Aug 12, 2013

We should task the documentation writeup.

I opened new pull request #2467 to extend DIP42 feature. Please read the summary in it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.