[Bf-committers] C++ in Blenders Kernel lib (Nav mesh)

Campbell Barton ideasman42 at gmail.com
Sat Sep 17 09:59:34 CEST 2011


Recent navmesh commit adds C++ into blenkernel -
source/blender/blenkernel/intern/navmesh_conversion.cpp

Until now we didn't have C++ in core libs, so not sure if this is
acceptable? - or what we should do now its in.

The file is only 450 LOC and doesn't use C++ classes or functionality
which is tricky to convert, the main thing holding it back from going
to C is Recast.h which has initializers inline with structs.

While I'm not up on C++, to me there seems to be 2 options.
1) Make Recast.h C compatible, convert navmesh_conversion.cpp to C code.
2) move navmesh_conversion.cpp into its own lib and have a C API (for
such a small file this is overkill but at least means no mixing
C/C++).

#2 is most straightforward, we can have intern/navmesh and arrange
much the same was as ghost.

Any comments?, anyone want to put their hands up to do this? :)

... or do we just not bother and have C++ in blenkernel?

-- 
- Campbell


More information about the Bf-committers mailing list