Messages in this thread | | | Date | Wed, 16 Jul 2008 23:12:01 +0100 | From | Al Viro <> | Subject | Re: [PATCH] dlm: sparse endian annotations |
| |
On Wed, Jul 16, 2008 at 02:43:41PM -0700, Harvey Harrison wrote: > On Wed, 2008-07-16 at 22:38 +0100, Al Viro wrote: > > On Wed, Jul 16, 2008 at 01:16:07PM -0700, Harvey Harrison wrote: > > > > > > NAK on ones below. You are only hiding the warnings; ...s() is not making > > it any better. > > > > I'd suggest that any use of {endian}s() points to code that should be > looked at. But if you'd also rather have the warnings, so be it.
Frankly, I would rather have the rest of byteswaps in dlm eliminated...
| |