We need to update internal copy of zlib On Thu, Feb 27, 2020 at 6:59 AM Moazin Khatri <[email protected]> wrote: >> >> > There are 3 ways to solve this problem: >> > 1. In devel/ftoption.h:226 enable the macro #define >> > FT_CONFIG_OPTION_SYSTEM_ZLIB >> > 2. Revert my patch altogether. But then gzip encoded glyphs from >> > ot-svg fonts won't work. (Don't know the impact on WOFF2 fonts) >> > 3. Update the internal version of zlib to the newest version. >> >> Solution 3 is the way to go, of course (and thanks for your >> willingness to work on it :-). However, as a temporary measure, maybe >> you could add some #ifdef stuff until you are done with updating the >> internal zlib? `FT_CONFIG_OPTION_SYSTEM_ZLIB' is a user option, and >> it has to work both switched on and off. > > > You mean something like this? > > #ifdef FT_CONFIG_OPTION_SYSTEM_ZLIB > err = inflateInit2( &stream, MAX_WBITS|32 ); > #else > err = inflateInit2( &stream, MAX_WBITS ); > #endif > > Basically, support the gzip header only if a system installation of > zlib is being used.
-- Alexei A. Podtelezhnikov, PhD
