Date: Sat, 31 May 2003 22:37:52 +0200 From: Erik Trulsson <ertr1013@student.uu.se> To: Poul-Henning Kamp <phk@FreeBSD.org> Cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/i386/isa clock.c Message-ID: <20030531203751.GA38298@falcon.midgard.homeip.net> In-Reply-To: <200305312023.h4VKNkKq037126@repoman.freebsd.org> References: <200305312023.h4VKNkKq037126@repoman.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, May 31, 2003 at 01:23:46PM -0700, Poul-Henning Kamp wrote: > phk 2003/05/31 13:23:46 PDT > > FreeBSD src repository > > Modified files: > sys/i386/isa clock.c > Log: > Don't rely on boolean expression evaluating to 1 or 0 by default. Why not? In C the relational operators (including '==') *always* evaluate to 1 or 0. One can rely on that. I.e. this change doesn't really accomplish anything except make the expression somewhat more complicated. > > Found by: FlexeLint > > Revision Changes Path > 1.199 +1 -1 src/sys/i386/isa/clock.c > _______________________________________________ > cvs-all@freebsd.org mailing list > http://lists.freebsd.org/mailman/listinfo/cvs-all > To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org" -- <Insert your favourite quote here.> Erik Trulsson ertr1013@student.uu.se
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20030531203751.GA38298>