thcmedia / papervision3d

Automatically exported from code.google.com/p/papervision3d
0 stars 0 forks source link

Pixels.removePixel3D removes all pixels - fix attached #102

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Add some pixels to a Pixels object
2. Remove one pixel, using removePixel3D

What is the expected output? What do you see instead?
Expect to see the one pixel removed.
Instead, all pixels are removed.

What version of the product are you using? On what operating system?
2.0 beta Latest from SVN on 16 September 2008.

Please provide any additional information below.
There are two brackets in the wrong place. The removePixel3D method should be

public function removePixel3D(pixel:Pixel3D):void
{
    pixel.instance = null;
    pixels.splice(pixels.indexOf(pixel),1);
    vertices.splice(vertices.indexOf(pixel.vertex3D),1);
}

Original issue reported on code.google.com by bigroom....@googlemail.com on 16 Sep 2008 at 7:04

Attachments:

GoogleCodeExporter commented 8 years ago
Andy, can you confirm ? 

Original comment by r.hauw...@gmail.com on 17 Oct 2008 at 1:53

GoogleCodeExporter commented 8 years ago
Thank you for reporting. Implemented.

Original comment by r.hauw...@gmail.com on 17 Nov 2008 at 12:40

GoogleCodeExporter commented 8 years ago

Original comment by r.hauw...@gmail.com on 17 Nov 2008 at 12:41