[PATCH xts] Don't sleep(2) at the beginning of Xt test setup

walter harms wharms at bfs.de
Wed Nov 2 14:17:10 UTC 2016



Am 02.11.2016 15:09, schrieb Adam Jackson:
> Regeneration is fast enough these days, we can skip this. This
> eliminates about 40 minutes of wall time from a full xts run.
> 
> Signed-off-by: Adam Jackson <ajax at redhat.com>
> ---
>  xts5/src/libXtTest/avs_init.c | 7 -------
>  1 file changed, 7 deletions(-)
> 
> diff --git a/xts5/src/libXtTest/avs_init.c b/xts5/src/libXtTest/avs_init.c
> index 459f141..6d632d3 100644
> --- a/xts5/src/libXtTest/avs_init.c
> +++ b/xts5/src/libXtTest/avs_init.c
> @@ -130,13 +130,6 @@ int argcount;
>  		tet_result(TET_UNRESOLVED);
>  		exit(0);
>  	}
> -	
> -	/*
> -	 * Wait for the server to reset before trying to connect.
> -	 * (Otherwise a connect attempt could succeed at the beginning
> -	 * of the reset, and the connection broken as part of the reset.)
> -	 */
> -	sleep(2);
>  
>  	for (i = 0; i < (config.reset_delay *2)+1; i++) {
>  		display = XtOpenDisplay(


i would make that optional. i have no idea who may need that
and so (most likely) the user.


just my 2 cents,

re,
 wh


More information about the xorg-devel mailing list