opened
chrisdembia
username_0
The changes I made here are based on discussions in https://github.com/opensim-org/opensim-core/issues/592.

I chose to remove the template class typedefs necessary, but I left the ones that I didn't need to change (with perhaps 1 or 2 exceptions). Also, I chose *not* to remove any typedefs for Mat.h; instead I put the typedefs in the public section. I justify this because Vec.h has similar typedefs that are public. By making only the necessary changes, I hope to have minimized the chance of introducing a bug.

Also, for computation within enums, I chose to use `#ifndef SWIG` rather than resorting to a `constexpr` function in order to make sure the C++ code remains clear and that I don't introduce a bug.

I've tested OpenSim wrapping against these changes, and both the python and java wrapping compile (and the python tests and testContext both pass). You can see my changes to OpenSim here: https://github.com/opensim-org/opensim-core/tree/swig-simtk
Changes to allow swig 3.0.5 to process certain Simbody headers.
issue
created
chrisdembia
username_0
@aymanhab
comment
created
sherm1
username_1
This all looks good, Chris -- thanks! Merging.
comment
created
chrisdembia
username_0
Thank you Sherm.
comment