[vlc-devel] [PATCH 1/1] Freetype: support Android's new fonts.xml
filip at atch.se
Sat Mar 11 21:42:36 CET 2017
Erhm, I have no idea why my brain decided to sprinkle *"not"* in
places where it certainly does not belong.
Sorry about the confusion (if any).
On 2017-03-11 21:34, Filip Roséen wrote:
> The above check of `*psz_val` is redundant as `atoi` will return `0`
> on an empty (or ill-formed) string.
> > + else if( !strcasecmp( "style", psz_attr ) && psz_val && *psz_val )
> > + if( !strcasecmp( "italic", psz_val ) )
> > + b_italic = true;
> Same thing about `*psz_val` in the above, `strcasecmp` will not return
> `0` for an empty string.
Should be read as *"`strcasecmp` **will** return `0` for an empty string"*.
> > + }
> > +
> > + if( i_weight == 700 )
> > + b_bold = true;
> As a font can be either *bold* or *not-bold* through `NewFont`, it
> might make sense to check whether `i_weight` is
> *greater-than-or-equal* to 700 (as `701` would not result in a
> *non-bold* font).
Last part should be read as *"as `701` **would** result in a
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the vlc-devel