[Ffmpeg-devel] [PATCH] drawtext.c: 01 fix compiler warnings

Michael Niedermayer michaelni
Sun Sep 24 01:00:13 CEST 2006


Hi

On Sat, Sep 23, 2006 at 01:16:12PM -0300, Gustavo Sverzut Barbieri wrote:
> On 9/12/06, Michael Niedermayer <michaelni at gmx.at> wrote:
> >Hi
> >
> >On Sun, Sep 10, 2006 at 02:00:02PM -0300, Gustavo Sverzut Barbieri wrote:
> >> This is the first patch of a series of incremental patches to drawtext.c.
> >>
> >> it fix the compiler warnings found in SVN, it's basically signess of
> >> datatypes.
> >[...]
> >> -  unsigned char *text = ci->text;
> >> +  char *text = ci->text;
> >[...]
> >> +          unsigned char previous = text[i - 1];
> >>            FT_Get_Kerning( ci->face,
> >> -                          ci->glyphs_index[ text[i-1] ],
> >> +                          ci->glyphs_index[previous],
> >
> >this is ugly IMHO
> 
> so, any other objection? I use "previous" to avoid a cast in order to
> avoid compiler warnings.

iam fine with this patch except the unsigned char previous thing

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In the past you could go to a library and read, borrow or copy any book
Today you'd get arrested for mere telling someone where the library is




More information about the ffmpeg-devel mailing list