[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