On Fri, 5 Mar 2021 10:42:08 +0000 <[email protected]> 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
