Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

[OBJJ] Parser doesn't detect double instance variable declaration #362

Closed
p3consulting opened this Issue · 10 comments

5 participants

@p3consulting

No description provided.

@p3consulting

@implementation MyClass : CPObject {
CPString var1 ;

CPNumber var1 ;
}

doesn't generate error or warning.

@stevegeek

yeah the parser doesn't do semantic checks on the code, only syntax checks so this would be missed.

@cappbot
Collaborator

Milestone: 1.0. Labels: #accepted, #needs-patch, Objective-J, bug. What's next? This issue needs a volunteer to write and submit code to address it.

@ahankinson

This seems to still be an issue.

Perhaps @mrcarlberg can comment on this?

@mrcarlberg
Collaborator

Yes, the new compiler does not issue a warning for this. I have this and a hand full other warnings on my todo list. I will implement those when I get some time to spare.

Thanks for letting me know about this issue.

@ahankinson

Possibly related: #512

@ahankinson

@mrcarlberg Was this solved by your recent pull request?

@mrcarlberg
Collaborator

No, this is not fixed yet.

@mrcarlberg
Collaborator

+#fixed
by #2326

@mrcarlberg mrcarlberg closed this
@cappbot
Collaborator

Milestone: 1.0. Labels: #fixed, Objective-J, bug. What's next? This issue is considered successfully resolved.

@cappbot cappbot added #fixed and removed #accepted #needs-patch labels
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.