Re: [PATCH] vmstate-static-checker:remove this redundant return

[Laurent Vivier]

Le 28/09/2022 à 11:03, dinglimin a écrit :

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 <dingli...@cmss.chinamobile.com>
---
  scripts/vmstate-static-checker.py | 1 -
  1 file changed, 1 deletion(-)

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():

Applied to my trivial-patches branch.

Thanks,
Laurent