[MPlayer-dev-eng] [PATCH]Fix FFmpeg warning

Carl Eugen Hoyos cehoyos at ag.or.at
Sat Mar 10 07:55:30 CET 2012



Diego Biurrun <diego <at> biurrun.de> writes:

> > > > -static struct vf_priv_s {
> > > > +struct vf_priv_s {
> > > >          unsigned char *buf;
> > > >          int brightness;
> > > >          int contrast;
> > > > -} const vf_priv_dflt = {
> > > > +} static const vf_priv_dflt = {
> > 
> > > This looks wrong.
> > 
> > Please elaborate.
> 
> gcc can complain about static not being the first keyword in declarations
> in -Wextra mode. 

True, but we do not use -Wextra
Do you plan to add it?

> So I wonder what the warning you are trying to fix here
> is in the first place.  Just splitting the struct and the variable
> declaration could be simpler in any case.

Please elaborate what is "wrong" about above patch.

Carl Eugen



More information about the MPlayer-dev-eng mailing list