Skip site navigation (1)Skip section navigation (2)
Date:      11 Mar 2002 00:17:27 +0200
From:      Maxim Sobolev <sobomax@FreeBSD.org>
To:        David Malone <dwmalone@maths.tcd.ie>
Cc:        cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/kern kern_linker.c
Message-ID:  <1015798536.527.38.camel@notebook>
In-Reply-To: <20020310202940.GA94559@walton.maths.tcd.ie>
References:  <200203101920.g2AJK1B38135@freefall.freebsd.org>  <20020310202940.GA94559@walton.maths.tcd.ie>

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

--=-BG+9gR96JkQpWFK+Yizt
Content-Type: text/plain
Content-Transfer-Encoding: quoted-printable

On Sun, 2002-03-10 at 22:29, David Malone wrote:
> On Sun, Mar 10, 2002 at 11:20:01AM -0800, Maxim Sobolev wrote:
> > sobomax     2002/03/10 11:20:01 PST
> >=20
> >   Modified files:
> >     sys/kern             kern_linker.c=20
> >   Log:
> >   Fix a breakage introduced in rev.1.75 (supposedly style cleanup), whi=
ch results
> >   in "missing dependencies" error when loading some kld modules. It is =
sad to
> >   see how often these days style cleanus break doesn't broken things. P=
erhaps
> >   people should recall good old principle: "don't fix it if it isn't br=
oken".
>=20
> Maxim,
>=20
> Would you mind if I chaned all the "err =3D strcmp() if (err =3D=3D 0)"s
> in this file into "if (strcmp() =3D=3D 0)". The fix I proposed in the
> PR mentioned by Steven did this (it fixed the same bug and made the
> strcmps a little more sane).

Yeah, please do.

-Maxim

--=-BG+9gR96JkQpWFK+Yizt
Content-Type: application/pgp-signature; name=signature.asc
Content-Description: This is a digitally signed message part

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.6 (FreeBSD)
Comment: For info see http://www.gnupg.org

iD8DBQA8i9sIoNu5t4iCBa8RApdOAJ0QPeuRt/4qAOUz2ZZsggkyakMu0gCfac9m
F69I9lRpnH+yGsThW0ktMe8=
=79cW
-----END PGP SIGNATURE-----

--=-BG+9gR96JkQpWFK+Yizt--


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




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