[MPlayer-dev-eng] [PATCH] another batch of compilation warnings fixes (GUI too)

Dominik Mierzejewski dominik at rangers.eu.org
Thu Oct 10 01:58:13 CEST 2002


On Thursday, 10 October 2002, Arpi wrote:
> Hi,
[...] 
> no, it's much simpler: just add const to devname decl.

Of course. :-)

> > Oh, and I've attached some patches to fix other warnings and one thing I
> > thought could be done better:
> > Gui/mplayer/gtk/fs.c:
> > -   if ( ( name[i] >='a' )&&( name[i] <= 'z' ) ) { tmp[c++]='['; tmp[c++]=name[i]; tmp[c++]=name[i] - 32; tmp[c++]=']'; }
> > +   if ( isalpha(name[i]) ) { tmp[c++]='['; tmp[c++]=name[i]; tmp[c++]=name[i] - 32; tmp[c++]=']'; }
> 
> is ctype.h standard on all mplayer-supported playforms?

Good question. I haven't thought about it. But I'm sure it's in the ANSI C
standard, so it should be included with all ANSI compliant compilers and
since we require gcc (which is ANSI C compliant), I think the answer is
yes.

> > - int             i;
> > +/* int             i;*/
> unused? unsure?

Unused.
 
-- 
MPlayer RPMs maintainer: http://www.piorunek.pl/~dominik/linux/pkgs/mplayer/
"The Universe doesn't give you any points for doing things that are easy."
        -- Sheridan to Garibaldi in Babylon 5:"The Geometry of Shadows"



More information about the MPlayer-dev-eng mailing list