[Bug 219608] print/freetype2 produces incorrect line spacing for some fonts

[bugzilla-noreply at freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=219608

--- Comment #3 from lightside <lightside at gmx.com> ---
Created attachment 183227
  --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=183227&action=edit
Some files/patch-src_truetype_ttdriver.c patch for test (for v2.8)

I attached some patch for test, which uses previous approach as in following
commit:
http://git.savannah.gnu.org/cgit/freetype/freetype2.git/commit/?id=b0962ac34e66052ccfee7996e5468f30d4bd5a72
This reverts changes for src/truetype/ttdriver.c file in following commit:
http://git.savannah.gnu.org/cgit/freetype/freetype2.git/commit/?id=bcc74f4dafee25ea89f1d3144646cba7e30f9908

with some changes for 2.8 version to fix following error:
-8<--
In file included from
<..>/freetype2/work/freetype-2.8/src/truetype/truetype.c:22:
<..>/freetype2/work/freetype-2.8/src/truetype/ttdriver.c:360:28: error: 
      assigning to 'FT_Size_Metrics' (aka 'struct FT_Size_Metrics_') from
incompatible type 'FT_Size_Metrics *'
      (aka 'struct FT_Size_Metrics_ *'); dereference with *
      ttsize->root.metrics = ttsize->metrics;
                           ^ ~~~~~~~~~~~~~~~
                             *
1 error generated.
-->8-
e.g. "ttsize->root.metrics = *ttsize->metrics;".
For example, copy patch to print/freetype2/files directory and rebuild port for
test.

Not sure if this is right approach, but possible to make it optional (for
EXTRA_PATCHES) as an alternative to AF_CONFIG_OPTION_TT_SIZE_METRICS define
(TT_SIZE_METRICS port's option):
http://git.savannah.gnu.org/cgit/freetype/freetype2.git/tree/src/autofit/afloader.c?id=VER-2-8#n255
before possible fix available in new version (and/or some changes for affected
applications).

-- 
You are receiving this mail because:
You are the assignee for the bug.