Thank you very much for the bufixes!
As for your e-mail question: it is best to send fixes to
[EMAIL PROTECTED]

This is the mailing list address.
There are 3 people on the mailing list that are able
to do CVS updates.

Bruno

Quoting "Tkalcec, Danijel" <[EMAIL PROTECTED]>:

> Good day!
> 
> First, I would like to tell you that iText is a really great library for
> creating PDF documents. Since we needed a way to dinamically generate PDF
> documents for our (also dinamically generated) HTML sites, and we couldn't
> find a solution that would suit our needs, we decided to use iText as a PDF
> generator and write our own "HTML to PDF converter". While doing so, I
> found
> and fixed 2 major bugs in iText. Sorry if this is the wrong address to send
> you bug fixes for the iText library, but I don't know any other ways.
> 
> 1. GIF import: <com.lowagie.text.pdf.PdfImage>
> - Larger GIFs or GIFs with more colours didn't import. The problem is a 12
> BIT GIF Table limit, which the GIF importer ignored. To fix this BUG (so
> that any GIF can be imported), you just need to insert a line in PdfImage
> constructor (in my version, this is line 488). 
> 
> This is the code chunk you would have to change ...
> -------------
>                         if (code == 256) {
>                             codelength = 9;
>                             tablelength = 257;
>                         }
>                         if (code == 257) {
>                             break;
>                         }
>                         if (tablelength == (1 << codelength))
>                         {
> /* insert this line  */   if (codelength<12) // bugfix by DT
>                             codelength++;
>                         }
> -------------------------------
> 
> 2. Table generation: <com.lowagie.text.pdf.PdfDocument>
> - If you use the "hasToFitPageCells" flag and have a cell that doesn't fit
> on an empty page, the Table generator will play stupid and generate
> thousands of empty pages while trying to fit this cell on a page. A simple
> bugfix would be to check the "hasToFitPageCells" flag only if we already
> placed a cell on this page. To do this, you just need to insert a line to
> PdfDocument.add(Element element). In my version, this is line 1531, but
> here
> is the code chunk where it goes ...
> --------------------------------
>                         // loop over the cells
>                         boolean cellsShown = false;
>                         int currentRownumber = 0;
>                         for (ListIterator iterator = cells.listIterator(); 
>                                           iterator.hasNext() &&
> !tableHasToFit; ) {
>                             cell = (PdfCell) iterator.next();
> 
> /* insert this line */      if (cellsShown) // bugfix by DT
>                             if (cell.rownumber() != currentRownumber && 
>                                                     !cell.isHeader() &&
> cellsHaveToFit) {
>                                 currentRownumber = cell.rownumber();
>                                 int cellCount = 0;
>                                 boolean cellsFit = true;
> --------------------------------
> 
> Is there a way I could communicate with you concerning iText (and send you
> bugfixes, if I should find any more bugs) that you would prefer rather that
> using the <[EMAIL PROTECTED]> E-mail address?
> 
> Best regards,
> 
> Danijel Tkalcec
> Systems Engineer
> 
> IMAGIN AG
> Kurmainzer Stra�e 20
> D - 65817 Eppstein
> Tel: +49 06198 586-241
> Fax: +49 06198 586-222
> E-mail: [EMAIL PROTECTED]
> 
> 





-------------------------------------------------------
This sf.net email is sponsored by:
With Great Power, Comes Great Responsibility 
Learn to use your power at OSDN's High Performance Computing Channel
http://hpc.devchannel.org/
_______________________________________________
iText-questions mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/itext-questions

Reply via email to