andlon / merely3d Goto Github PK

Currently, a lot of code is implemented in header files, which slows down compilation of user code. Most of this code is not performance sensitive, and as such would be better implemented in source files.

When using merely3d as a library loaded through CMake submodules, it would be nice to avoid compiling examples.

merely3d uses Eigen throughout its API. This is OK for our own courses at the RWTH Aachen University, but coupling the merely3d API to Eigen is unnecessary, and unfortunate for anyone who wants to use it without being forced to include Eigen into their project.

In particular, focus on documenting non-obvious or ambiguous interfaces. One example is documenting better how the mesh data for StaticMesh should look like.

Investigate this.

Example:

camera.look_in(Vector3f(0.0, 0.0, 0.0), Vector3f(0.0, 0.0, 1.0));

does not complain, even though obviously the zero vector is not a valid direction.