Hi David,

Nice patch. Probably is a good idea remove the dx variable in the function 
getResizeDirection() too, to avoid this compiler warning:

gcc -DHAVE_CONFIG_H -I. -I..   -DLOCALEDIR=\"/usr/local/lib/locale\" -I../wrlib 
-I../WINGs -I/usr/local/include   -DNDEBUG -D_XOPEN_SOURCE=600 -D_GNU_SOURCE  
-g -O2 -std=c11 -Wall -Wextra -Wno-sign-compare -Wno-deprecated 
-Wno-deprecated-declarations -MT moveres.o -MD -MP -MF .deps/moveres.Tpo -c -o 
moveres.o moveres.c
moveres.c: In function ‘getResizeDirection’:
moveres.c:1801:65: warning: unused parameter ‘dx’ [-Wunused-parameter]
 static int getResizeDirection(WWindow * wwin, int x, int y, int dx, int dy, 
int flags)

Cheers,
kix

On Thu, 14 Aug 2014, David Maciejak escribió:

> This patch is fixing compilation issue as variable 'dx'
> is not used in getResizeDirection function
> ---
>  src/moveres.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/moveres.c b/src/moveres.c
> index 3d707d7..6645b22 100644
> --- a/src/moveres.c
> +++ b/src/moveres.c
> @@ -1803,6 +1803,7 @@ static int getResizeDirection(WWindow * wwin,
> int x, int y, int dx, int dy, int
>   int w = wwin->frame->core->width - 1;
>   int cw = wwin->frame->resizebar_corner_width;
>   int dir;
> + (void) dx;
> 
>   /* if not resizing through the resizebar */
>   if (!(flags & RESIZEBAR)) {
> --



-- 
To unsubscribe, send mail to [email protected].

Reply via email to