[poppler] [RFC] poppler_form_field_get_font_size is broken
Albert Astals Cid
aacid at kde.org
Mon Mar 26 22:24:07 UTC 2018
El dilluns, 26 de març de 2018, a les 22:14:34 CEST, Adam Reichold va
escriure:
> Hello,
>
> Am 26.03.2018 um 19:36 schrieb Albert Astals Cid:
> > So i was looking at Annot and realized we have a fontSize member that is
> > initialized to 0 and just used to return in glib
> > poppler_form_field_get_font_size
> >
> > The attached patch removes the member variable and makes
> > poppler_form_field_get_font_size return 0. This is obviously not the
> > correct fix, but maybe it's better to commit this (maybe adding a TODO to
> > the glib code) than having some code that does nothing (until someone can
> > actually fix it properly?)
>
> I think the change is reasonable and the TODO would help. Better to have
> it explicitly broken than to follow a mirage of partial implementations.
Ok, if noone disagrees, i'll commit it (with the TODO) in ~1 or 2 weeks.
Cheers,
Albert
>
> Best regards, Adam.
>
> > Cheers,
> >
> > Albert
> >
> > _______________________________________________
> > poppler mailing list
> > poppler at lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/poppler
>
> _______________________________________________
> poppler mailing list
> poppler at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/poppler
More information about the poppler
mailing list