Loading of castle-anim-frames is significantly slower in CGE 6.3

[michaliskambi]

Loading of castle-anim-frames is significantly slower in CGE 6.3

[eugeneloza]

No problem :)
But very useful to know, as I'm trying to work with animations loading/processing at the moment. Thanks!

[michaliskambi]

I fixed the speed loss. And then I discovered that there was no speed loss... so the fix is useless. So the fix is not committed to the CGE master branch (as it makes code a bit uglier, and since it's not needed, it's better to not have it).

Sorry for the noise, looks like there was no need to notify you about this, and there was no need for me to do anything... Well, this wasn't my most productive moment :) Off to sleep now :)

More details:

After I made a fix I tested what the actual speed difference was (lost in e240cc9 and gained again in fab51cb ). And there is no speed difference. Neither there is a difference in memory usage.

Of course, I should have tested this earlier, but I just assumed that e240cc9 was definitely something losing speed or increasing memory usage. It turns out it doesn't matter. The custom "epsilon" in castle-anim-frames is not actually used to gain anything anymore.

The fix is lying for now in https://github.com/castle-engine/castle-engine/compare/use-castle-anim-frames-epsilon branch , but it seems that there's no need to merge it ever to master, so eventually I'll just remove this branch.

[eugeneloza]

Great! Thanks for notification!