Re: [Podofo-users] PoDoFo Patch - Count as reference in page tree

2014-06-15 Thread zyx
On Thu, 2014-06-12 at 07:56 +0200, zyx wrote: > Otherwise the patch looks fine. Hi, I went ahead and committed, slightly changed, Petr's patch as r1640. http://sourceforge.net/p/podofo/code/1640 Bye, zyx -- http://www.litePDF.cz i...@litep

Re: [Podofo-users] PoDoFo Patch - Count as reference in page tree

2014-06-11 Thread zyx
On Wed, 2014-06-11 at 22:28 +0200, Petr Pytelka wrote: > -int GetChildCount( const PdfObject* pNode ) const; > +static int GetChildCount( const PdfObject* pNode ); >   > Reasons to change signature of the method: > 1. GetChildCount does not need any member variables from PdfPagesTree

Re: [Podofo-users] PoDoFo Patch - Count as reference in page tree

2014-06-11 Thread Petr Pytelka
Hi, thank you for integrating the patch. I've tested solution with another set of PDF files and found another places where number of pages is requested using: ... static_cast(this->GetObject()->GetDictionary().GetKeyAsLong( "Count", 0LL )) I've prepared new version of the patch to repl

Re: [Podofo-users] PoDoFo Patch - Count as reference in page tree

2014-06-10 Thread zyx
On Tue, 2014-06-10 at 10:16 +0200, Petr Pytelka wrote: > There is small patch in the attachment which solves the issue. Hi, the patch looks fine, I committed it as r1604: http://sourceforge.net/p/podofo/code/1604 Bye, zyx -- http://www.litePDF.cz

[Podofo-users] PoDoFo Patch - Count as reference in page tree

2014-06-10 Thread Petr Pytelka
Hello, I've received PDF file with broken page table where page count is stored as reference. Opening such file returnes zero pages instead of correct count. There is small patch in the attachment which solves the issue. Any comments are welcomed... Regards, Petr Pytelka Index: src/doc/Pdf