Re: [PATCH] tracing histograms: Simplify parse_actions() function

2024-01-08 Thread Steven Rostedt
On Mon, 8 Jan 2024 10:32:14 +0200 Andy Shevchenko wrote: > On Mon, Jan 8, 2024 at 3:31 AM Steven Rostedt wrote: > > > > From: "Steven Rostedt (Google)" > > > > The parse_actions() function uses 'len = str_has_prefix()' to test which > > action is in the string being parsed. But then it goes

Re: [PATCH] tracing histograms: Simplify parse_actions() function

2024-01-08 Thread Andy Shevchenko
On Mon, Jan 8, 2024 at 3:31 AM Steven Rostedt wrote: > > From: "Steven Rostedt (Google)" > > The parse_actions() function uses 'len = str_has_prefix()' to test which > action is in the string being parsed. But then it goes and repeats the > logic for each different action. This logic can be

[PATCH] tracing histograms: Simplify parse_actions() function

2024-01-07 Thread Steven Rostedt
From: "Steven Rostedt (Google)" The parse_actions() function uses 'len = str_has_prefix()' to test which action is in the string being parsed. But then it goes and repeats the logic for each different action. This logic can be simplified and duplicate code can be removed as 'len' contains the