[MPlayer-cvslog] r24170 - trunk/command.c

[Reimar Döffinger]

Hello,
On Sat, Aug 25, 2007 at 02:47:42PM +0200, diego wrote:
> Author: diego
> Date: Sat Aug 25 14:47:41 2007
> New Revision: 24170
> 
> Log:
> Fix unused variable warning when USE_DVDNAV is not defined.
> 
> 
> Modified:
>    trunk/command.c
> 
> Modified: trunk/command.c
> ==============================================================================
> --- trunk/command.c	(original)
> +++ trunk/command.c	Sat Aug 25 14:47:41 2007
> @@ -2736,12 +2736,13 @@ int run_command(MPContext * mpctx, mp_cm
>  	    break;
>  
>  	case MP_CMD_SET_MOUSE_POS:{
> -		int button = -1, pointer_x, pointer_y;
> +		int pointer_x, pointer_y;
>  		double dx, dy;
>  		pointer_x = cmd->args[0].v.i;
>  		pointer_y = cmd->args[1].v.i;
>  		rescale_input_coordinates(pointer_x, pointer_y, &dx, &dy);
>  #ifdef USE_DVDNAV
> +		int button = -1;
>  		if (mpctx->stream->type == STREAMTYPE_DVDNAV
>  		    && dx > 0.0 && dy > 0.0) {
>  		    pointer_x = (int) (dx * (double) sh_video->disp_w);

IMO you should move this into the if, it is not used outside.

Greetings,
Reimar Döffinger