[PATCH 5/7] debug output format fix in ephyrProcessMouseMotion()

Peter Hutterer peter.hutterer at who-t.net
Sun Aug 16 18:21:23 PDT 2015


On Thu, Aug 13, 2015 at 01:11:35PM +0100, Jon TURNEY wrote:
> xorg/xserver/hw/kdrive/ephyr/ephyr.c:979:9: error: format ‘%d’ expects argument of type ‘int’, but argument 4 has type ‘ScreenPtr’ [-Werror=format=]
> 
> This looks like a genuine bug, and ephyrCursorScreen->myNum was meant here
> rather than ephyrCursorScreen
> 
> Signed-off-by: Jon TURNEY <jon.turney at dronecode.org.uk>
> ---
>  hw/kdrive/ephyr/ephyr.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/hw/kdrive/ephyr/ephyr.c b/hw/kdrive/ephyr/ephyr.c
> index 164ebdc..22710a6 100644
> --- a/hw/kdrive/ephyr/ephyr.c
> +++ b/hw/kdrive/ephyr/ephyr.c
> @@ -978,7 +978,7 @@ ephyrProcessMouseMotion(xcb_generic_event_t *xev)
>      if (ephyrCursorScreen != screen->pScreen) {
>          EPHYR_LOG("warping mouse cursor. "
>                    "cur_screen%d, motion_screen:%d\n",

looks like a typo too, might as well insert a ":" here

Cheers,
   Peter

> -                  ephyrCursorScreen, screen->pScreen->myNum);
> +                  ephyrCursorScreen->myNum, screen->pScreen->myNum);
>          ephyrWarpCursor(inputInfo.pointer, screen->pScreen,
>                          motion->event_x, motion->event_y);
>      }
> -- 
> 2.4.5
> 
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel
> 


More information about the xorg-devel mailing list