Skip to content

Disallow extern enum(type) #6899

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.

Sign up for GitHub

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

Jump to bottom
Closed
tadeokondrak opened this issue Oct 31, 2020 · 2 comments
Closed

Disallow extern enum(type) #6899

tadeokondrak opened this issue Oct 31, 2020 · 2 comments
Labels
proposal This issue suggests modifications. If it also has the "accepted" label then it is planned.
Milestone
0.8.0

Comments

@tadeokondrak
Copy link
Contributor

This, as far as I know, is just treated like the type in the parentheses.
#6427 allowed normal enums doing this, and so I think this syntax is redundant.
@Vexu
Copy link
Member

Vexu commented Oct 31, 2020

It's not redundant at the moment since we might need the enum to have members with values that don't fit in a c_int and we don't currently select the type based on the tag values like C does.

@Vexu Vexu added the proposal This issue suggests modifications. If it also has the "accepted" label then it is planned. label Oct 31, 2020
@Vexu Vexu added this to the 0.8.0 milestone Oct 31, 2020
@andrewrk andrewrk modified the milestones: 0.8.0, 0.9.0 May 19, 2021
@andrewrk
Copy link
Member

Instead, we're getting rid of extern enums altogether: #2115 (comment)

@andrewrk andrewrk modified the milestones: 0.9.0, 0.8.0 May 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
proposal This issue suggests modifications. If it also has the "accepted" label then it is planned.
Projects
None yet
Milestone
0.8.0
Development

No branches or pull requests
3 participants
@andrewrk @tadeokondrak @Vexu