[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