Messages in this thread | | | Date | Thu, 24 Nov 2005 17:46:22 +0000 | From | Russell King <> | Subject | Re: + shut-up-warnings-in-ipc-shmc.patch added to -mm tree |
| |
On Thu, Nov 24, 2005 at 08:00:12AM -0800, Matt Mackall wrote: > Unfortunately Russell didn't tell us which function caused the error > and I can't seem to find a tree that matches his line numbering. > But it looks like it's shm_unlock.
To make it completely clear, a lot of my "fix warning" patches are derived from the ARM Linux kernel autobuilder, which can be found at http://armlinux.simtec.co.uk/kautobuild/
and I only really look at the latest builds on there.
> The current ({0;}) seems wrong to me. I'd expect that expression to be > void. Hmm, looks like I'm wrong. It's quite ugly, not to mention confusing.
It seems that ({0;}) is used when something is expected to return zero. However, if it is used in a void context, gcc 4 generates an annoying warning.
> mm.h: > #define shm_lock(a, b) empty_int() > > The typechecking is nice in theory, but in practice I don't think it > really makes a difference for stubbing things out.
Depends if you end up with "blah is unused" warnings instead. It's all round _far_ safer to use the inline function method from that point of view.
Not that I particularly care, I just wanted to squash some of the rediculous number of warnings in the kernel and decided to hit the easy ones. However, they're turning out to be real pigs instead. 8(
-- Russell King Linux kernel 2.6 ARM Linux - http://www.arm.linux.org.uk/ maintainer of: 2.6 Serial core - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |