Skip to content

Conversation

@zhaijialong
Copy link
Contributor

@zhaijialong
Copy link
Contributor Author

@dingpinglv @pandamicro take a look please

@pandamicro
Copy link
Contributor

Which test case can test its functionality ?

@pandamicro
Copy link
Contributor

No problem with the code logic. Need to test the functionality

@zhaijialong
Copy link
Contributor Author

Test added

@pandamicro
Copy link
Contributor

Thanks, I will test it on Mac and iOS

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe this should be the assert for b.y

@zhaijialong
Copy link
Contributor Author

Fixed, it will not crash now.

@pandamicro
Copy link
Contributor

Tested on Mac and iOS, good to merge

dingpinglv added a commit that referenced this pull request Nov 7, 2014
add missing properties of cp.SegmentShape and cp.PolyShape
@dingpinglv dingpinglv merged commit e6c7c7f into cocos2d:develop Nov 7, 2014
@zhaijialong zhaijialong deleted the shape branch November 7, 2014 03:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants