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

Jump to bottom

camlidl and enum #2817

Closed
vicuna opened this issue Jun 27, 2001 · 2 comments
Closed

camlidl and enum #2817

vicuna opened this issue Jun 27, 2001 · 2 comments
Labels
bug camlidl

Comments

@vicuna
Copy link

vicuna commented Jun 27, 2001

Original bug ID: 413
Reporter: administrator
Status: closed
Resolution: fixed
Priority: normal
Severity: minor
Category: -for CamlIDL use https://github.com/xavierleroy/camlidl/issues

Bug description

Full_Name: Dmitry Bely
Version: 3.01, camlidl cvs 27.06.01
OS: Windows NT 4.0
Submission from: d009.p3.col.ru (195.210.132.9)

enum declaclaration does not allow extra comma after the last member:

enum enum1 {
E11 = 0,
E12 = 0 /* OK */
};

enum enum2 {
E21 = 0,
E22 = 0, /* syntax error */
};

It would be nice just to ignore such extra commas, because Microsoft's .idl
files are full of them (don't know if IDL specs allow them or not. Probably they
do because such commas are legal from C point of view).
@vicuna
Copy link
Author

vicuna commented Jun 29, 2001

Comment author: administrator

enum declaclaration does not allow extra comma after the last member:
It would be nice just to ignore such extra commas, because
Microsoft's .idl files are full of them (don't know if IDL specs
allow them or not. Probably they do because such commas are legal
from C point of view).

My K&R second edition doesn't allow the trailing comma, and nor does
the MIDL spec, but I agree it's common usage, convenient, and easy to
implement. I'll implement it.

  • Xavier Leroy

@vicuna
Copy link
Author

vicuna commented Jun 29, 2001

Comment author: administrator

Allowed optional trailing comma. -XL, 2001-06-29

@vicuna vicuna closed this as completed Jul 30, 2001
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug camlidl
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@vicuna