search

BaldrDash / recastnavigation

Automatically exported from code.google.com/p/recastnavigation
zlib License
0 stars 0 forks source link

rcIntArray still using new/delete #95

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
I noticed you'd missed the new/delete in rcIntArray.

It also doesn't have a valid copy constructor or assignment operator.

Original issue reported on code.google.com by cameron....@gmail.com on 9 Jul 2010 at 11:33

Attachments:

GoogleCodeExporter commented 9 years ago 
Thanks for pointing that out. Let me know if you have others. It is so easy to 
get blind when repretitively changing a lot of things.

I found one more in debug utils in duReadCompactHeightfield(). Not quite sure 
which alloc to use for the display list thing, maybe move I'll move it out from 
there. Looking into that.

I also found one more delete [] in removeTile(), and interestingly the navmes 
builder was using sizeof(new unsigned char) there too.

All fixes in R181.

Original comment by memono...@gmail.com on 10 Jul 2010 at 9:06