Re: [PATCH] tracing/tools: Remove unneeded semicolon

From: Steven Rostedt
Date: Fri Mar 05 2021 - 09:22:22 EST


On Fri, 5 Mar 2021 10:42:08 +0000
<Viktor.Rosendahl@xxxxxx> wrote:

> Looks good but I would suggest to fix also the other instances of superfluous
> semicolons with the same patch.
>
> There seems to be a lonely semicolon on line 1944, i.e this code:
>
> if (value < 0) {
> warnx("TIME must be >= 0\n");
> show_usage();
> ;
> }
>
> In that case the lonely semicolon should be replaced with "exit(0);"

Agreed. I'll wait for v2 of the patch.

Thanks,

-- Steve