On Fri, 2020-02-21 at 19:53 +0100, Michal Sudolsky wrote:
> Maybe I would personally did not add new parameter but instead tried
> to fix it in way that value passed to SetIgnoreBrokenObjects() is not
> overwritten in Init() and make true the default.

        Hi,
thinking of it, I like the idea. Just have the PdfMemDocument and
PdfParser hold the preference and use it, instead of passing it around
as extra parameters. Having it default to true is also a good idea.

The PdfMemDocument's 'load' constructors will still need to be changed,
but otherwise PdfMemDocument can pass its value to its PdfParser
instance when needed.

Eventually, it could be a global (static) property of the PdfParser, to
make things even simpler. Would there be a need to use PdfParser from
multiple threads with different setting at the same time? I guess not.
This would make the patch much smaller.

Ideas?
        Thanks and bye,
        zyx



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

Reply via email to