summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authordinglimin2022-09-28 11:03:12 +0200
committerLaurent Vivier2022-10-22 23:19:11 +0200
commit6191347991225a291ce38f5faf4d6e8181b5561f (patch)
treeb0e588513f293dc19c51e66d270372d918acf463
parenttests/qtest: vhost-user-test: Fix [-Werror=format-overflow=] build warning (diff)
downloadqemu-6191347991225a291ce38f5faf4d6e8181b5561f.tar.gz
qemu-6191347991225a291ce38f5faf4d6e8181b5561f.tar.xz
qemu-6191347991225a291ce38f5faf4d6e8181b5561f.zip
vmstate-static-checker:remove this redundant return
Jump statements, such as return and continue let you change the default flow of program execution, but jump statements that direct the control flow to the original direction are just a waste of keystrokes. Signed-off-by: dinglimin <dinglimin@cmss.chinamobile.com> Reviewed-by: John Snow <jsnow@redhat.com> Message-Id: <20220928090312.2537-1-dinglimin@cmss.chinamobile.com> Signed-off-by: Laurent Vivier <laurent@vivier.eu>
-rwxr-xr-xscripts/vmstate-static-checker.py1
1 files changed, 0 insertions, 1 deletions
diff --git a/scripts/vmstate-static-checker.py b/scripts/vmstate-static-checker.py
index b369388360..dfeee8231a 100755
--- a/scripts/vmstate-static-checker.py
+++ b/scripts/vmstate-static-checker.py
@@ -367,7 +367,6 @@ def check_machine_type(s, d):
if s["Name"] != d["Name"]:
print("Warning: checking incompatible machine types:", end=' ')
print("\"" + s["Name"] + "\", \"" + d["Name"] + "\"")
- return
def main():