[poppler] Second opinion for patch

Carlos Garcia Campos carlosgc at gnome.org
Thu Nov 17 16:48:24 UTC 2016


Albert Astals Cid <aacid at kde.org> writes:

> Can someone have a look at https://bugs.freedesktop.org/show_bug.cgi?id=98732
>
> I'd like a second opinion on the patch, it fixes the problem on that broken 
> file for me but before commiting i'd like to know i'm not missing something.

I guess the file is broken because "Last" is required field in outline
dictionary. In that case I would keep using Last if present, and simply
remove the early return in OutlineItem::readItemList() otherwise.

> Cheers,
>   Albert
> _______________________________________________
> poppler mailing list
> poppler at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/poppler

-- 
Carlos Garcia Campos
PGP key: http://pgp.mit.edu:11371/pks/lookup?op=get&search=0x523E6462
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 162 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/poppler/attachments/20161117/0baa4aa3/attachment.sig>


More information about the poppler mailing list