Re: [PATCH] tests/qtest: check the return value

2023-11-21 Thread Thomas Huth
On 21/11/2023 09.08, Zhu Jun wrote: These variables "ret" are never referenced in the code, thus add check logic for the "ret" Signed-off-by: Zhu Jun Thanks! Reviewed-by: Thomas Huth I'll queue it for my next pull request.

RE: [PATCH] tests/qtest: check the return value

2023-11-20 Thread Zhang, Chen
> -Original Message- > From: zhujun2 > Sent: Tuesday, November 21, 2023 2:17 PM > To: Zhang, Chen > Cc: lviv...@redhat.com; pbonz...@redhat.com; qemu-devel@nongnu.org; > th...@redhat.com; zhuj...@cmss.chinamobile.com > Subject: [PATCH] tests/qtest: check the return value > > These

Re: [PATCH] tests/qtest: check the return value

2023-11-20 Thread Thomas Huth
Thanks, patch looks better this way, but I'd still like to ask you to fix two small issues: On 21/11/2023 07.16, zhujun2 wrote: These variables "ret" are never referenced in the code, that s/, that/, thus/ add check logic for the "ret" Signed-off-by: zhujun2 Could you *please* fix