Date: Wed, 16 Jul 2014 12:45:12 +0200 From: Mateusz Guzik <mjguzik@gmail.com> To: Baptiste Daroussin <bapt@FreeBSD.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r268745 - in head/usr.bin: . timeout Message-ID: <20140716104512.GB17076@dft-labs.eu> In-Reply-To: <20140716103411.GH48710@ivaldir.etoilebsd.net> References: <201407160955.s6G9taro084054@svn.freebsd.org> <20140716102908.GA17076@dft-labs.eu> <20140716103411.GH48710@ivaldir.etoilebsd.net>
next in thread | previous in thread | raw e-mail | index | archive | help
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>
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20140716104512.GB17076>