On Saturday 15 April 2017 01:18:44 Karl Berry wrote:
I'm sorry, but I am not comfortable with releasing these changes for the Type 3 fonts + encoding + Unicode. I realize the features now work in your cases, but I feel much more testing and understanding is needed, before inflicting changes to these low-level routines on the world. I can't undertake this for TL17.
The proximate cause is that, in some further testing, I discovered a case that crashed for me now, but did not crash before: \pdfmapline{<}
That is because in mapfile.c check_fm_entry() is missing check if
tfm_name is not NULL.
Such thing is possible to hit also without my patches... And is not
related to my patches.
Try:
\pdfmapline{
Also, it's unclear to when the new warning "invalid entry for `%s': encoding file for type3 missing" is intended to be given. I could not find any input that caused it.
That was added by Akira.
I'm sure that it would not be hard to remedy these particular things, one way or another, but the general principle remains -- I'm simply too uneasy to release these changes, which have grown and grown. As I said before, the primary criterion for pdftex nowadays must be stability.
Unfortunately, in that testing, I also discovered other erroneous mapline cases that were already unhandled (before your patches). I know I have not found them all yet. Sigh.
If Thanh or Martin wants to include these changes in the upcoming release, and can help with testing/code review, then that is fine. Otherwise, it will have to wait for another time.
For the present, I have reverted the changes in the TL repository. (Hopefully correctly.)
At least you have written the code now, so can compile your own pdftex and use it for your own work ... --sorry, karl.
So what is needed for inclusion? More testing? And do we have a case where pdftex behave incorrect (or crash) which is related to those toUnicode patches for PK fonts? Because I have not found any problem in current code yet and above crash (which you found) is there for a long time. Do you want a fix for above crash? -- Pali Rohár pali.rohar@gmail.com