aboutsummaryrefslogtreecommitdiff
path: root/tests/tcg
diff options
context:
space:
mode:
authorMarkus Armbruster <armbru@redhat.com>2022-11-22 14:49:16 +0100
committerMarkus Armbruster <armbru@redhat.com>2022-12-14 16:19:35 +0100
commit66997c42e02c84481fc162a5b7bd6ad6c643bae2 (patch)
tree889d4e08622f178c74334c3317ccc8a0ca8431c3 /tests/tcg
parent10220d2f96406463fc283a3de6b13fc38f8befff (diff)
downloadqemu-66997c42e02c84481fc162a5b7bd6ad6c643bae2.zip
qemu-66997c42e02c84481fc162a5b7bd6ad6c643bae2.tar.gz
qemu-66997c42e02c84481fc162a5b7bd6ad6c643bae2.tar.bz2
cleanup: Tweak and re-run return_directly.cocci
Tweak the semantic patch to drop redundant parenthesis around the return expression. Coccinelle drops a comment in hw/rdma/vmw/pvrdma_cmd.c; restored manually. Coccinelle messes up vmdk_co_create(), not sure why. Change dropped, will be done manually in the next commit. Line breaks in target/avr/cpu.h and hw/rdma/vmw/pvrdma_cmd.c tidied up manually. Whitespace in tools/virtiofsd/fuse_lowlevel.c tidied up manually. checkpatch.pl complains "return of an errno should typically be -ve" two times for hw/9pfs/9p-synth.c. Preexisting, the patch merely makes it visible to checkpatch.pl. Signed-off-by: Markus Armbruster <armbru@redhat.com> Message-Id: <20221122134917.1217307-2-armbru@redhat.com> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Acked-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Diffstat (limited to 'tests/tcg')
-rw-r--r--tests/tcg/multiarch/sha512.c9
1 files changed, 2 insertions, 7 deletions
diff --git a/tests/tcg/multiarch/sha512.c b/tests/tcg/multiarch/sha512.c
index e172982..9e701bc 100644
--- a/tests/tcg/multiarch/sha512.c
+++ b/tests/tcg/multiarch/sha512.c
@@ -855,8 +855,6 @@ plan_tests(unsigned int tests)
static int
exit_status_(void)
{
- int r;
-
/* If there's no plan, just return the number of failures */
if(no_plan || !have_plan) {
return failures;
@@ -865,15 +863,12 @@ exit_status_(void)
/* Ran too many tests? Return the number of tests that were run
that shouldn't have been */
if(e_tests < test_count) {
- r = test_count - e_tests;
- return r;
+ return test_count - e_tests;
}
/* Return the number of tests that failed + the number of tests
that weren't run */
- r = failures + e_tests - test_count;
-
- return r;
+ return failures + e_tests - test_count;
}
int