[Devel] [PATCH vz10 02/15] selftests/clone3: remove unused variables in clone3_cap_checkpoint_restore

[Vasileios Almpanis]

Reviewed-by: Vasileios Almpanis <vasileios.almpanis at virtuozzo.com>

On 3/23/26 9:31 PM, Konstantin Khorenko wrote:
>    clone3_cap_checkpoint_restore.c: In function 'call_clone3_set_tid':
>    clone3_cap_checkpoint_restore.c:57:22: warning: unused variable 'tmp' [-Wunused-variable]
>       57 |                 char tmp = 0;
>          |                      ^~~
>    clone3_cap_checkpoint_restore.c:56:21: warning: unused variable 'ret' [-Wunused-variable]
>       56 |                 int ret;
>          |                     ^~~
>    clone3_cap_checkpoint_restore.c: In function 'clone3_cap_checkpoint_restore':
>    clone3_cap_checkpoint_restore.c:138:13: warning: unused variable 'ret' [-Wunused-variable]
>      138 |         int ret = 0;
>          |             ^~~
>
> Remove unused variables 'ret' and 'tmp' to fix -Wunused-variable
> warnings.
>
> https://virtuozzo.atlassian.net/browse/VSTOR-127529
> Signed-off-by: Konstantin Khorenko <khorenko at virtuozzo.com>
>
> Feature: fix selftests
> ---
>   .../testing/selftests/clone3/clone3_cap_checkpoint_restore.c  | 4 ----
>   1 file changed, 4 deletions(-)
>
> diff --git a/tools/testing/selftests/clone3/clone3_cap_checkpoint_restore.c b/tools/testing/selftests/clone3/clone3_cap_checkpoint_restore.c
> index 3c196fa86c99..976e92c259fc 100644
> --- a/tools/testing/selftests/clone3/clone3_cap_checkpoint_restore.c
> +++ b/tools/testing/selftests/clone3/clone3_cap_checkpoint_restore.c
> @@ -53,9 +53,6 @@ static int call_clone3_set_tid(struct __test_metadata *_metadata,
>   	}
>   
>   	if (pid == 0) {
> -		int ret;
> -		char tmp = 0;
> -
>   		TH_LOG("I am the child, my PID is %d (expected %d)", getpid(), set_tid[0]);
>   
>   		if (set_tid[0] != getpid())
> @@ -135,7 +132,6 @@ TEST(clone3_cap_checkpoint_restore)
>   {
>   	pid_t pid;
>   	int status;
> -	int ret = 0;
>   	pid_t set_tid[1];
>   
>   	test_clone3_supported();