Date: Tue, 28 Mar 2006 20:05:12 +0200 From: des@des.no (Dag-Erling =?iso-8859-1?Q?Sm=F8rgrav?=) To: John Baldwin <jhb@freebsd.org> Cc: cvs-src@freebsd.org, src-committers@freebsd.org, cvs-all@freebsd.org Subject: Re: cvs commit: src/sys/i386/include atomic.h Message-ID: <86psk6wilj.fsf@xps.des.no> In-Reply-To: <200603281300.09419.jhb@freebsd.org> (John Baldwin's message of "Tue, 28 Mar 2006 13:00:06 -0500") References: <200603281434.k2SEYmaR031447@repoman.freebsd.org> <200603281032.10624.jhb@freebsd.org> <86y7yuwn01.fsf@xps.des.no> <200603281300.09419.jhb@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
John Baldwin <jhb@freebsd.org> writes: > One reason for not having the casts, btw, is that you lose type > checking. Huh? Before my patch, any use of atomic_*_ptr with warnings turned off would result in a slew of warnings because you'd be passing pointers to a function which is declared to take u_int. The only way to make this type safe is to use inline functions instead of the macros I wrote. DES --=20 Dag-Erling Sm=F8rgrav - des@des.no
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?86psk6wilj.fsf>