Re: linux/macros.h(new) and linux/list.h(mod) ...

From: Kai Germaschewski (
Date: Thu Jul 05 2001 - 17:03:24 EST

On Thu, 5 Jul 2001, Davide Libenzi wrote:

> This patch add a new linux/macros.h that is supposed to host utility macros
> that otherwise developers are forced to define in their files.
> This version contain only min(), max() and abs().

It's a good old tradition to put macros in uppercase letters. This would
have avoided one fatal error in your patch, the conflict with the gcc
        int abs(int);

which has it's prototype in include/linux/kernel.h. There's places which
depend on this and would break with your macro.

Also, unless you have more macros in mind, it may make sense to just place
MIN, MAX in kernel.h and of course to remove similar macro definitions
throughout the kernel and replace them by the commonly defined ones.


To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

This archive was generated by hypermail 2b29 : Sat Jul 07 2001 - 21:00:17 EST