consider a reusable context for iup_optimize_deltas

[cmyr]

In cases where a font has a large number of variation locations, iup_optimize_deltas is doing a lot of redundant allocation. In particular we clone the point list for each location when this could be shared, and we also allocate a number of hashsets that could be reused.

This feels low priority/is not backed by any profiling, but there is likely some room for improvement here.

[rsheeter]

Id really like to see this pop out on a profile or some sort of measurement before making the code more complicated for it.

…

On Tue, Sep 19, 2023, 10:51 AM Colin Rofls ***@***.***> wrote: In cases where a font has a large number of variation locations, iup_optimize_deltas is doing a lot of redundant allocation. In particular we clone the point list for each location when this could be shared, and we also allocate a number of hashsets that could be reused. This feels low priority/is not backed by any profiling, but there is likely some room for improvement here. — Reply to this email directly, view it on GitHub <#618>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABRKXABMDSRVNSNEHO7EHHTX3HLRFANCNFSM6AAAAAA46VYPRE> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>