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

Reply via email to