[MPlayer-dev-eng] ctype.h functions on win32

Rich Felker dalias at aerifal.cx
Sun Oct 14 15:11:59 CEST 2007


On Sun, Oct 14, 2007 at 02:46:27PM +0200, Aurelien Jacobs wrote:
> On Sun, 14 Oct 2007 01:01:18 -0400
> Rich Felker <dalias at aerifal.cx> wrote:
> 
> > On Sun, Oct 14, 2007 at 06:33:25AM +0200, Gianluigi Tiesi wrote:
> > > pls try attached patches (splitted mp/lavc/lavf)
> > > I leaved as is all ints that come from fgetc since
> > > it's supposed to be ok
> > > changed chars to unsigned chars where needed
> > > (and also in other similar functions for conformance)
> > > cast to uint8_t/uchar(in input.c uint8_t not defined)
> > > when the source of the value it's not so clear
> > > (at least for me of course :D)
> > > 
> > >  static int
> > > -vobsub_parse_timestamp(vobsub_t *vob, const char *line)
> > > +vobsub_parse_timestamp(vobsub_t *vob, const unsigned char *line)
> > >  {
> > >      // timestamp: HH:MM:SS.mmm, filepos: 0nnnnnnnnn
> > > -    const char *p;
> > > +    unsigned const char *p;
> > >      int h, m, s, ms;
> > >      off_t filepos;
> > >      while (isspace(*line))
> > 
> > Looks ok.
> 
> This one looks inconsistent ("unsigned const" vs. "const unsigned").

These types are the same as long as they're both valid.

Rich



More information about the MPlayer-dev-eng mailing list