[PATCH xts 2/3] xts5: Fix missing variable for format specifier

walter harms wharms at bfs.de
Sat Oct 29 20:45:00 UTC 2016



Am 29.10.2016 05:37, schrieb Rhys Kidd:
> XtRemoveEventHandler.c:458:3: warning: format '%d' expects a matching 'int' argument [-Wformat=]
>    sprintf(ebuf, "ERROR: Error message handler was invoked %d times");
>    ^
> XtRemoveEventHandler.c:463:3: warning: format '%d' expects a matching 'int' argument [-Wformat=]
>    sprintf(ebuf, "ERROR: Warning message handler was invoked %d times");
>    ^
> XtRemoveEventHandler.c:540:3: warning: format '%d' expects a matching 'int' argument [-Wformat=]
>    sprintf(ebuf, "ERROR: Error message handler was invoked %d times");
>    ^
> XtRemoveEventHandler.c:545:3: warning: format '%d' expects a matching 'int' argument [-Wformat=]
>    sprintf(ebuf, "ERROR: Warning message handler was invoked %d times");
>    ^
> 
> Signed-off-by: Rhys Kidd <rhyskidd at gmail.com>
> ---
>  xts5/Xt9/XtRemoveEventHandler.m | 16 ++++++++--------
>  1 file changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/xts5/Xt9/XtRemoveEventHandler.m b/xts5/Xt9/XtRemoveEventHandler.m
> index 1851ef9..6e6cbbe 100644
> --- a/xts5/Xt9/XtRemoveEventHandler.m
> +++ b/xts5/Xt9/XtRemoveEventHandler.m
> @@ -425,13 +425,13 @@ int invoked = 0;
>  	XtAppMainLoop(app_ctext);
>  	LKROF(pid2, AVSXTTIMEOUT-2);
>  	tet_infoline("TEST: Error message was not emitted");
> -	if (avs_get_event(3) != 0) {
> -		sprintf(ebuf, "ERROR: Error message handler was invoked %d times");
> +	if (invoked = avs_get_event(3) != 0) {
> +		sprintf(ebuf, "ERROR: Error message handler was invoked %d times", invoked);
>  		tet_infoline(ebuf);
>  		tet_result(TET_FAIL);
>  	}

IMHO this is to complicated. what is wrong with:

invoked = avs_get_event(3);
if (invoked != 0) {
		sprintf(ebuf, "ERROR: Error message handler was invoked %d times", invoked);
  		tet_infoline(ebuf);
 		tet_result(TET_FAIL);
 }



> -	if (avs_get_event(2) != 0) {
> -		sprintf(ebuf, "ERROR: Warning message handler was invoked %d times");
> +	if (invoked = avs_get_event(2) != 0) {
> +		sprintf(ebuf, "ERROR: Warning message handler was invoked %d times", invoked);
>  		tet_infoline(ebuf);
>  		tet_result(TET_FAIL);
>  	}


btw: this is the same msg als with avs_get_event(3). I have no clue what the differenz is
     but maybe it should be reflected in the errmsg.

just my 2 cents,

re,
 wh


> @@ -491,13 +491,13 @@ int invoked = 0;
>  		tet_result(TET_FAIL);
>  	}
>  	tet_infoline("TEST: Error message was not emitted");
> -	if (avs_get_event(3) != 0) {
> -		sprintf(ebuf, "ERROR: Error message handler was invoked %d times");
> +	if (invoked = avs_get_event(3) != 0) {
> +		sprintf(ebuf, "ERROR: Error message handler was invoked %d times", invoked);
>  		tet_infoline(ebuf);
>  		tet_result(TET_FAIL);
>  	}
> -	if (avs_get_event(2) != 0) {
> -		sprintf(ebuf, "ERROR: Warning message handler was invoked %d times");
> +	if (invoked = avs_get_event(2) != 0) {
> +		sprintf(ebuf, "ERROR: Warning message handler was invoked %d times", invoked);
>  		tet_infoline(ebuf);
>  		tet_result(TET_FAIL);
>  	}


More information about the xorg-devel mailing list