On Monday 18 December 2017 15:46:13 Hans Hagen wrote:
For pdftex patches, I specially prepared different enc files to test that pdftex with my patches does not crash (either print fatal warning or produce PDF) and that I always get PDF file rendered in same way as enc files must not affect how PDF file is rendered. And I observed problem when enc file contain one glyph name more times, therefore I added that code which deals with duplicates.
well, if you apply that patch, then at least let pdftex complain very loud on the console and in the log that the resulting pdf is probably bugged (even with the patch) due to conflicts in the encoding
In my patch is pdftex_warn() which cause warning on console. Resulting PDF is not buggy as duplicate glyph names are replaced by "a<num>" (same glyph name as before my patch).
(i don't mind that much about a patch because i don't use pdftex so i'm unlikely to be a victim of such a bad font .. so in the end it's karl who has to agree)
Ok, so it is up to Karl now. Changing pdftex_warn() to fatal error is trivial if is better to disallow usage of such broken enc files. -- Pali Rohár pali.rohar@gmail.com