On Wed, Jul 16, 2014 at 12:34:11PM +0200, Baptiste Daroussin wrote: > On Wed, Jul 16, 2014 at 12:29:08PM +0200, Mateusz Guzik wrote: > > On Wed, Jul 16, 2014 at 09:55:36AM +0000, Baptiste Daroussin wrote: > > > +static void > > > +usage(void) > > > +{ > > > + fprintf(stderr, "Usage: %s [--signal sig | -s sig] [--preserve-status]" > > > + " [--kill-after time | -k time] [--foreground] <duration> <command>" > > > + " <arg ...>\n", getprogname()); > > > + > > > > Missing newline at the begnning. > New line at the beginning? > >
static void usage(void) { fprintf(...); > > > + exit(EX_USAGE); > > > +} > > > + > > > + switch(signo) { > > > + case 0: > > > > sig 0? I doubt it is ever delivered. > > one of the tests from the GNU testsuite is passing 0 signal I even checked. The kernel is not going to deliver signal 0, so checking for it in signal handler does not seem to make sense. Does the testcaes fail without it or something? -- Mateusz Guzik <mjguzik gmail.com> _______________________________________________ svn-src-all@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/svn-src-all To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"