On Wed, 2015-08-26 at 15:59 +0000, Matthew Brincke wrote:
> What do you think?

        Hi,
I do not know the exact rationale for the code being changed by your
patch. I think the best option is to have a test .pdf file(s) for it
and kind of a unit test, proving the change being right. The mix of
numKids and numDirectKids also makes things more complicated.

> +                          ); // TODO: perhaps log reference too?

I'd prefer to not add any new TODO, just decide and do it (or don't do
it).
        Bye,
        zyx

-- 
http://www.litePDF.cz                                 i...@litepdf.cz



------------------------------------------------------------------------------
_______________________________________________
Podofo-users mailing list
Podofo-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/podofo-users

Reply via email to