search

BaldrDash / recastnavigation

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

Use of simple structs instead of float*, int*, short* #159

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Enhancement request:

Working to make adjustments or simply understand the Recast and Detour code can 
be a bit hard in places where there is the need to treat [var]* as a vector 
array.  It would be useful to have simple structs declared which have x,y and z 
members.  This shouldn't add any memory overhead.  Also if the need is still 
present to pass the data via [var]* the data array could be cast to [struct]* 
to make usage of the data more straight forward and help prevent errors.

Original issue reported on code.google.com by shawnlha...@gmail.com on 25 Jan 2011 at 4:14

GoogleCodeExporter commented 9 years ago 
This would mean quite a huge rewrite. I'm hesitant to do such huge change to 
the API, but if there is something I could do gradually to make the code more 
understandable, I'm all for it.

Original comment by memono...@gmail.com on 3 Feb 2011 at 10:17

GoogleCodeExporter commented 9 years ago 
I believe this is something that could be done gradually by simply continuing 
to pass along float* but before usage cast them to Vector*.  This would allow 
you to change one function at a time.

Original comment by shawnlha...@gmail.com on 3 Feb 2011 at 3:30

GoogleCodeExporter commented 9 years ago

Original comment by memono...@gmail.com on 16 Sep 2013 at 6:54