[bug #65585] [gropdf] problems introduced by commit cd9fde325f

[G. Branden Robinson]

Update of bug #65585 (group groff):

  Status:   Fixed => Need Info  

___

Follow-up Comment #8:

Peter reported a regression in the PDF outline for the following document that
I bisected to this commit.


commit 2387949ffc11b753dd91480dcb85be6c22819790 (refs/bisect/bad)
Author: Deri James 
Date:   Fri Apr 12 17:10:49 2024 +0100

Correct linear search introduced in commit cd9fde325f

* tmac/pdf.tmac: Various problems have been rectified, speed
improvements to the linear search, Keith protection against
illegal characters being used in string identifiers.

* contrib/mom/om.tmac: Switch to using Branden's linear search
so that we don't have to maintain two different methods.

Fixes 


Here's the reproducer I used.  (It's Peter's except for the groff version
report.  I was pretty baffled until I remembered I needed to use
$GROFF_BIN_PATH with pdfmom.)


$ cat EXPERIMENTS/peter-cond-thing-gbr.mom 
.TITLE "PDF Outline Test
.PRINTSTYLE TYPESET
.CONDENSE 80
.EXTEND   120
.START
.HEADING 1 "\*[COND]Condensed heading\*[CONDX]"
.PP
Normal text.
This document was generated by groff version \n[.x].\n[.y].\n[.Y].
.br
\*[COND]Condensed text.\*[CONDX]
.HEADING 1 "\*[EXT]Extended heading\*[EXTX]"
.PP
Normal text.
.br
\*[EXT]Extended text.\*[EXTX]




___

Reply to this item at:

  

___
Message sent via Savannah
https://savannah.gnu.org/

[bug #63018] font/devps/ZD: make glyphs accessible via their Unicode spellings

[Deri James]

Follow-up Comment #5, bug #63018 (group groff):

Probably the best way of doing this is by adding to dingbats.map, a suitable
one is attached (to replace the one in font/devps/generate). Also attached is
a replacement ZD file to be placed in fonts/devps. Doing make will propagate
the changes to devpdf and when U-ZD is created it will use the new
dingbats.map.

The attached .trf file includes all the characters in the unicode 2700 code
page (see U2700.pdf). When you run it you will see that some code points are
missing from the font, this probably because the font was created before
unicode and there have been numerous changes.

ZDR is a different kettle of fish! It does not make sense reversing many of
the glyphs, and they would not have unicodes anyway. We currently only have
one entry in reversed-dingbats.map, for \[lh]. For gropdf we have it defined
in pdf.tmac as:-

.char \[lh] \X'pdf: xrev'\[rh]\X'pdf: xrev'


(file #55989, file #55990, file #55991, file #55992)

___

Additional Item Attachment:

File name: ZD Size: 6KiB


File name: dingtst.trfSize: 1KiB


File name: dingbats.map   Size: 2KiB


File name: dingtst.pdfSize: 49KiB



AGPL NOTICE

These attachments are served by Savane. You can download the corresponding
source code of Savane at
https://git.savannah.nongnu.org/cgit/administration/savane.git/snapshot/savane-02a92bc3de37203138822bd2520e89ef55345685.tar.gz


___

Reply to this item at:

  

___
Message sent via Savannah
https://savannah.gnu.org/