[poppler] Read some more info from the Font Descriptor

Albert Astals Cid aacid at kde.org
Mon Jul 4 09:42:39 PDT 2005


I'm calmed.

Isn't it obvious why i want to use an enum? The use you want to give to the=
=20
API is irrelevant. Would you suggest storing the name of weedays using a in=
t=20
instead of using an enum? This is the same situation, FontStretch has 9=20
possible values and FontWeight has other 9 possible values, using an int fo=
r=20
a variable that has 9 possible values is simply bad design.

But if it is needed so that you accept my patch tell me which int values yo=
u=20
want me to give to the stretch variable as I'm not smart enough to know how=
=20
to represent "Condensed" using an int.

Albert

A Dilluns 04 Juliol 2005 03:24, v=C3=A0reu escriure:
> Albert Astals Cid wrote:
> > Ok, so the master wants ints for those both attributes, i assume you wa=
nt
> > me to store 100, 200, 300, 400, 500, 600, 700, 800 and 900 for FontWeig=
ht
> > but which values should i give to FontStretch?
> >
> > Albert
>
> Albert, could you please calm down and explain, in technical terms, why
> you think my suggestion is bad and how using enums is better?  Maybe
> we're just envisioning different uses for this API; you haven't
> described how you plan to use this, so maybe I'm just missing the bigger
> picture.
>
> thanks,
> Kristian

		
______________________________________________ 
Renovamos el Correo Yahoo! 
Nuevos servicios, más seguridad 
http://correo.yahoo.es


More information about the poppler mailing list