Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 16 Jul 2014 13:29:14 +0200
From:      Baptiste Daroussin <bapt@FreeBSD.org>
To:        Mateusz Guzik <mjguzik@gmail.com>
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:  <20140716112914.GI48710@ivaldir.etoilebsd.net>
In-Reply-To: <20140716104512.GB17076@dft-labs.eu>
References:  <201407160955.s6G9taro084054@svn.freebsd.org> <20140716102908.GA17076@dft-labs.eu> <20140716103411.GH48710@ivaldir.etoilebsd.net> <20140716104512.GB17076@dft-labs.eu>

next in thread | previous in thread | raw e-mail | index | archive | help

--yhze8HlyfmXt1APY
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Wed, Jul 16, 2014 at 12:45:12PM +0200, Mateusz Guzik wrote:
> 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-st=
atus]"
> > > > +	    " [--kill-after time | -k time] [--foreground] <duration> <co=
mmand>"
> > > > +	    " <arg ...>\n", getprogname());
> > > > +
> > >=20
> > > Missing newline at the begnning.
> > New line at the beginning?
> > >=20
>=20
> static void
> usage(void)
> {
>=20
> 	fprintf(...);

Fixed
>=20
> > > > +	exit(EX_USAGE);
> > > > +}
> > > > +
> > > > +	switch(signo) {
> > > > +	case 0:
> > >=20
> > > sig 0? I doubt it is ever delivered.
> >=20
> > one of the tests from the GNU testsuite is passing 0 signal
>=20
> 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.
>=20
> Does the testcaes fail without it or something?
>=20

Yes the test fails without it

regards,
Bapt

--yhze8HlyfmXt1APY
Content-Type: application/pgp-signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iEYEARECAAYFAlPGYgoACgkQ8kTtMUmk6ExTCgCeN5OA5fQShU36A1SBW2uMuGWs
WogAnA1/LZ2Q+cSZ+JgehXJHFEwq8KyV
=Bxr+
-----END PGP SIGNATURE-----

--yhze8HlyfmXt1APY--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20140716112914.GI48710>