[NTG-pdftex] [PATCH] pdftex - Add new \pdfpkscalable primitive

Pali Rohár pali.rohar at gmail.com
Wed May 17 16:40:34 CEST 2017


On Thursday 20 April 2017 14:58:42 Pali Rohár wrote:
> On Saturday 15 April 2017 05:49:23 Pali Rohár wrote:
> > 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{<test.pfb}
> > 
> > and even on old TeX Live 2012 it crashed too:
> > 
> > ../../../texk/web2c/pdftexdir/subfont.c:181: handle_subfont_fm: 
> > Assertion `fm->tfm_name != ((void *)0)' failed.
> > 
> > So it is there for a long time.
> > 
> > > 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?
> 
> Just to note that I sent patch for PK encfile support in Aug 2016...
> So still I would like to hear what is the problem with last version and
> how can I fix it or improve if needed.
> 
> And question about pdftex crash when tfmfile is not specified still
> remind: Should I look at it and propose fix?

PING.

-- 
Pali Rohár
pali.rohar at gmail.com


More information about the ntg-pdftex mailing list