nem0 / OpenFBX

Lightweight open source FBX importer
MIT License
1.15k stars 135 forks source link

signed long long for time value #24

Closed Neill3d closed 6 years ago

Neill3d commented 6 years ago

Hi,

thank you for your library! I'm using it to open fbx scene in my android application and I hope to add a support for some features from motionbuilder. I've just figured out that in AnimationCurve you are storing values in unsigned long long, which will give unpredictable result when converting time to seconds as real value should be in signed long long, for managing negative time values as well (some animation could have stance pose in -100 frame for example)

nem0 commented 6 years ago

Hi, do you have an example file I can test the issue with?

Neill3d commented 6 years ago

Hi, here is a scene with a pose in -25 frame. testTimeValues.zip

nem0 commented 6 years ago

Thanks, it should work now. closed via fe71524af103731ee595f0c056e5cd4c4b5e9177