Skip site navigation (1)Skip section navigation (2)
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>