tdf#113448 fix type1 font subsetter
commit6907cbd0f8e198a0f1810b1a07f552a47c9da660
authorThorsten Behrens <Thorsten.Behrens@CIB.de>
Wed, 13 Mar 2019 23:36:37 +0000 (14 00:36 +0100)
committerThorsten Behrens <Thorsten.Behrens@CIB.de>
Sat, 16 Mar 2019 22:41:33 +0000 (16 23:41 +0100)
treeff857be15860163df2aac5c9d8e028a21d0da435
parentb4d5e95b3c9454ead87dd2333799dcae41ac7970
tdf#113448 fix type1 font subsetter

This fixes verapdf A-1 validation error '6.2.11.5-1 glyph width in
dict and font file inconsistent'

Previous code was writing hard-coded '1000' as glyph width for any
type1 font subsetting - since actual type2 width info only became
available during subsequent convertOneTypeOp() parsing.

Catch was, that loop sometimes already modifies the output buffer
(whenever glyph path info was given), so we fix that by first padding
out 5 bytes for the width (size of integers are sadly variable), then
parsing Type2 glyph code, and in the end putting in the actual width
value we then know.

Can't put hsbw type1 op last, since standard requires that to be first
(and can only be given _once_)...

Could be re-done nicer by buffering Type2 glyph data, then writing it.
Left as an exercise for the reader.

Change-Id: I64ffaa32ded2f0a7c06311d1e0426cf358308a0a
Reviewed-on: https://gerrit.libreoffice.org/69293
Tested-by: Jenkins
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
vcl/source/fontsubset/cff.cxx