Skip to content

Commit c4258d8

Browse files
walacglemco
authored andcommitted
rv/rvgen: remove unnecessary semicolons
Remove unnecessary semicolons from Python code in the rvgen tool. Python does not require semicolons to terminate statements, and their presence goes against PEP 8 style guidelines. These semicolons were likely added out of habit from C-style languages. This cleanup improves consistency with Python coding standards and aligns with the recent improvements to remove other Python anti-patterns from the codebase. Signed-off-by: Wander Lairson Costa <wander@redhat.com> Reviewed-by: Nam Cao <namcao@linutronix.de> Reviewed-by: Gabriele Monaco <gmonaco@redhat.com> Link: https://lore.kernel.org/r/20260223162407.147003-6-wander@redhat.com Signed-off-by: Gabriele Monaco <gmonaco@redhat.com>
1 parent b70bc5c commit c4258d8

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

tools/verification/rvgen/rvgen/dot2k.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -46,14 +46,14 @@ def fill_tracepoint_handlers_skel(self) -> str:
4646
buff.append("\t/* XXX: validate that this event is only valid in the initial state */")
4747
handle = "handle_start_run_event"
4848
if self.monitor_type == "per_task":
49-
buff.append("\tstruct task_struct *p = /* XXX: how do I get p? */;");
50-
buff.append(f"\tda_{handle}(p, {event}{self.enum_suffix});");
49+
buff.append("\tstruct task_struct *p = /* XXX: how do I get p? */;")
50+
buff.append(f"\tda_{handle}(p, {event}{self.enum_suffix});")
5151
elif self.monitor_type == "per_obj":
5252
buff.append("\tint id = /* XXX: how do I get the id? */;")
5353
buff.append("\tmonitor_target t = /* XXX: how do I get t? */;")
5454
buff.append(f"\tda_{handle}(id, t, {event}{self.enum_suffix});")
5555
else:
56-
buff.append(f"\tda_{handle}({event}{self.enum_suffix});");
56+
buff.append(f"\tda_{handle}({event}{self.enum_suffix});")
5757
buff.append("}")
5858
buff.append("")
5959
return '\n'.join(buff)

0 commit comments

Comments
 (0)