seastar-json2code: use dict.get() when checking allowMultiple

simpler this way. it also silences following warning: ``` visually indented line with same indent as next logical line [E129] ``` from pycodestyle, see also https://peps.python.org/pep-0008/#indentation. Signed-off-by: Kefu Chai <[email protected]>
tchaikov · Feb 26, 2024 · 3f534c7 · 3f534c7
1 parent cb78f9e
commit 3f534c7
Showing 1 changed file with 6 additions and 7 deletions.
diff --git a/scripts/seastar-json2code.py b/scripts/seastar-json2code.py
@@ -294,14 +294,13 @@ def add_operation(hfile, ccfile, path, oper):
         else:
             fprint(ccfile, "\n,")
         if is_url:
-            fprint(ccfile, '{', '"/', path_param , '", path_description::url_component_type::FIXED_STRING', '}')
+            component_type = 'FIXED_STRING'
+        elif get_parameter_by_name(oper, path_param).get("allowMultiple",
+                                                         False):
+            component_type = 'PARAM_UNTIL_END_OF_PATH'
         else:
-            path_param_type = get_parameter_by_name(oper, path_param)
-            if ("allowMultiple" in path_param_type and
-                path_param_type["allowMultiple"]):
-                fprint(ccfile, '{', '"', path_param , '", path_description::url_component_type::PARAM_UNTIL_END_OF_PATH', '}')
-            else:
-                fprint(ccfile, '{', '"', path_param , '", path_description::url_component_type::PARAM', '}')
+            component_type = 'PARAM'
+        fprint(ccfile, f'{{"/{path_param}", path_description::url_component_type::{component_type}}}')
     fprint(ccfile, '}')
     fprint(ccfile, ',{')
     enum_definitions = ""