Two functions implemented in cpPolyShape.h require C99

Discuss any Chipmunk bugs here.
Post Reply
Michael Buckley
Posts: 46
Joined: Tue Aug 21, 2007 10:30 am
Contact:

Two functions implemented in cpPolyShape.h require C99

Post by Michael Buckley »

I just noticed when I was compiling a program, and I forgot to give -std=c99 to gcc, that cpPolyShape.h contains two functions (cpPolyShapeValueOnAxis and cpPolyShapeContainsVert) which contain for loops in which the integer i is used but not previously declared. That is OK in C99, but won't compile under older standards. It's not terribly important, but a lot of what I'm writing doesn't need any C99 features, so I don't usually pass -std=c99 to gcc. It would be nice to be able to include the headers without needing specify C99.
User avatar
slembcke
Site Admin
Posts: 4166
Joined: Tue Aug 14, 2007 7:13 pm
Contact:

Re: Two functions implemented in cpPolyShape.h require C99

Post by slembcke »

Someone else pointed that out to me already actually. It's fixed in subversion and will be part of the next release.
Can't sleep... Chipmunks will eat me...
Check out our latest projects! -> http://howlingmoonsoftware.com/wordpress/
Michael Buckley
Posts: 46
Joined: Tue Aug 21, 2007 10:30 am
Contact:

Re: Two functions implemented in cpPolyShape.h require C99

Post by Michael Buckley »

I did check the SVN repository through the Aerosol Google Code page. Should I be checking a different repository before submitting bugs?
User avatar
slembcke
Site Admin
Posts: 4166
Joined: Tue Aug 14, 2007 7:13 pm
Contact:

Re: Two functions implemented in cpPolyShape.h require C99

Post by slembcke »

:O You sir are correct. Not in trunk or the other branch I was working in recently. I crossed it off my list and everything. Maybe I did change it then reverted the source? Bah. Honestly I don't know of anyone more absent minded.

It's done now anyway.
Can't sleep... Chipmunks will eat me...
Check out our latest projects! -> http://howlingmoonsoftware.com/wordpress/
Post Reply

Who is online

Users browsing this forum: No registered users and 6 guests