Multiple non Javascript-native instance variable declarations per-line don't work #99

Closed
cappuccino opened this Issue May 3, 2009 · 6 comments

Comments

Projects
None yet
4 participants
@ghost

ghost commented May 3, 2009

Cappuccino appears to have a problem with having instance variables declared in a class where the type is non Javascriopt-native and you specify multiple variable declarations per line. For example:

@@@
@implementation NavigationBar : CPView
{
int _type, _fieldPadding;
id _backButton, _forwardButton;
}
.
.
.
@@@

will result in a ’_backButton not defined’ error the first time _backButton is referenced at runtime. There are no problems accessing either _type or _fieldPadding, which are int’s. It also doesn’t appear to matter whether the offending variables are declared as ’id’s or some other non Javascript-native type.

Changing this to:

@@@
@implementation NavigationBar : CPView
{
int _type, _fieldPadding;
id _backButton;
id _forwardButton;
}
.
.
.
@@@

alleviates the problem.

original LH ticket

This ticket has 0 attachment(s).

@ghost

ghost commented May 3, 2009

Multiple non Javascript-native instance variable declarations per-line don’t work

This is actually due to us not supporting the comma notation for multiple declarations of the same time. It’s surprising that any of them worked.

by Francisco Tolmasky

Member

boucher commented May 5, 2009

I'm guessing the int one didn't actually work, and the failure case just wasn't detected.

Contributor

aparajita commented Apr 10, 2012

Hopefully this can be added to Objective-J 2.

cappbot commented May 9, 2012

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

Contributor

aparajita commented Feb 16, 2013

Not a priority.

#wont-fix

cappbot commented Feb 16, 2013

Milestone: 1.0. Labels: #wont-fix, Objective-J, feature. What's next? A reviewer or core team member has decided against acting upon this issue.

cappbot closed this Feb 16, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment