Hi, Jeremias Maerki wrote: > On 10.02.2009 20:26:19 Andreas Delmelle wrote: >> On 10 Feb 2009, at 12:33, Vincent Hennebert wrote: <snip/> >> >> That said, I also can't immediately say how many font-stream types >> there are or should be, but if it's only two classes and it is >> unlikely that others will become necessary, one could start to wonder >> whether it's worth the effort... > > There are two font-stream types at the moment: Type 1 and TTF.
The setupFilterList method is defined or re-defined 5 times in the hierarchy, and the 5 versions are almost identical. Besides the font streams, something’s definitely wrong there. > AbstractPDFStream (or a common AbstractFontStream) without the use of > PDFStream inside its code. At the moment, the TTF data is buffered > twice: once as a byte[] in PDFFactory and once inside the PDFStream's > internal buffer. Doesn't make too much sense. I'll take a closer look > when I have time (and motivation). I have other priorities right now. If > that's a problem I can also revert revision 742766. +1 Thanks, Vincent
