search

Roxside / jbox2d

Automatically exported from code.google.com/p/jbox2d
BSD 2-Clause "Simplified" License
0 stars 0 forks source link

PolygonShape.set doesnt copy vertices #6

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What version of the product are you using? On what operating system?
2.1.2 alpha

Please provide any additional information below.
File: PolygonShape.java
Method: void set(final Vec2[] vertices, final int count)
Line: 174

It should be "m_vertices[i].set(vertices[i])" instead of "m_vertices[i] = 
vertices[i]" to make an hard copy. Otherwise the vertices could be changed 
outside of this class.

Original issue reported on code.google.com by zel1...@gmail.com on 19 May 2011 at 5:34

GoogleCodeExporter commented 9 years ago 
cool, thanks, just changed

Original comment by toucansa...@gmail.com on 28 May 2011 at 5:55

GoogleCodeExporter commented 9 years ago 
done, commited.  thanks!

Original comment by toucansa...@gmail.com on 28 May 2011 at 5:58