Disable checksum-checking

In sfnt.py, the SFNTReader constructor defaults to checkChecksums=1. I suggest we default to off.

[behdad]

If it’s only the desire to remedy the logic dissonance of the different defaults between TTFont and SFNTReader, then I am not sure this is really worth it, considering this has been like this for more than 20 years and nobody ever complained.

I’m pushing towards deprecating checksums in SFNT and that’s why I opened this as well.

[twardoch]

I think fontTools is now used in some high-production-volume places, kind of as a font “renderer” (unlike years ago where it was used in single fonts by human users), so any performance improvement in the default config might be useful.

On Thu, 10 Sep 2020 at 19:16, Behdad Esfahbod [e-mail] wrote:

If it’s only the desire to remedy the logic dissonance of the different defaults between TTFont and SFNTReader, then I am not sure this is really worth it, considering this has been like this for more than 20 years and nobody ever complained.

I’m pushing towards deprecating checksums in SFNT and that’s why I opened this as well.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/fonttools/fonttools/issues/2057#issuecomment-690531004, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAD6XRBT7AB7GTGSLRO2RQTSFECV5ANCNFSM4Q6XM3NQ .