swordlegend / recastnavigation

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

dtAssert & rcAssert to go via context? #114

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
I'd be tempted to make your assert macros call a method on "rcBuildContext" or 
similar. Every company I've worked at has had their own funky implementation of 
assert...and I think it makes sense for recast to be able to channel its 
asserts via a custom implementation.

Just a thought :) (and sorry for so much API feedback all of a sudden!)

Original issue reported on code.google.com by armstron...@gmail.com on 24 Aug 2010 at 10:29

GoogleCodeExporter commented 9 years ago
I would love to use the Pow2 [1] assert, but everyone is rolling their own 
anyways.

I decided to have a simple include file whose sole purpose is to just define 
assert, so that you can include what ever your platform should use. I added 
comment about that in the header files.

[1] http://cnicholson.net/2009/02/stupid-c-tricks-adventures-in-assert/

Original comment by memono...@gmail.com on 24 Aug 2010 at 5:59