[MPlayer-dev-eng] [PATCH] trivial changes to suppress compiler warnings

[Diego Biurrun]

On Fri, Oct 23, 2009 at 08:43:22PM +0200, Reimar Döffinger wrote:
> On Fri, Oct 23, 2009 at 07:48:34PM +0200, Diego Biurrun wrote:
> > On Thu, Oct 22, 2009 at 03:25:19PM +0200, Reimar Döffinger wrote:
> > > On Thu, Oct 22, 2009 at 02:00:30PM +0200, Diego Biurrun wrote:
> > > > --- libvo/x11_common.c	(revision 29789)
> > > > +++ libvo/x11_common.c	(working copy)
> > > > @@ -1341,7 +1341,7 @@
> > > >  
> > > >  void vo_x11_fullscreen(void)
> > > >  {
> > > > -    int x, y, w, h;
> > > > +    int x=0, y=0, w=0, h=0;
> > > 
> > > The whole code of this function should probably be written in a better
> > > way, but here specifically I'd suggest just moving the
> > >             x = vo_old_x;
> > >             y = vo_old_y;
> > >             w = vo_old_width;
> > >             h = vo_old_height;
> > > part up here.
> > 
> > Like $attached?
> 
> Yes, looks fine to me (of course an extra check can't hurt).

Applied.

Diego