Multiply defined functions at link time due to omission of inline

[csc14gg]

The proposed fix:

inline_functions.patch

Mark all functions inline
1. Corrects multiply defined symbol linker errors when
including tiny_bvh.h in more than one CPP file.
2. Replace static inline with just inline for the
functions declared that way, since static has no
semantic meaning with inline also declared.

Would you like me to open a pull request for this? Thanks!

[jbikker]

Could it be that you included tinybvh with TINYBVH_IMPLEMENTATION defined in several files? You should include it with that define active only in one c/cpp.

[csc14gg]

No, the functions my patch changes are not in the implementation section guarded by TINYBVH_IMPLEMENTATION.

[jbikker]

Oh I see what's going on. I specifically setup an example to test inclusion in multiple files, but that test also overrides the math classes... I am going to apply your patch, thanks. :)

[jbikker]

In fact, could you submit this as a PR in the dev branch?

[csc14gg]

Will do sure!

[csc14gg]

#102