Re: 2.4.9 does not compile [PATCH]

Anton Altaparmakov
2001-08-16T23:25:39Z

At 23:31 16/08/2001, David S. Miller wrote:
> From: tpepper@vato.org
> Date: Thu, 16 Aug 2001 14:41:09 -0700
>
> Confirmed here. Looks like a pretty obvious goof to me. Does the
> following
> fix it for you?
>
>The args and semantics of min/max changed to take
>a type first argument, the problem with this ntfs file is that it
>fails to include linux/kernel.h

It has indeed. I do fail to see why that was necessary though...

IMHO, it would have been more elegant to use the typeof construct provided
by gcc in the new macro instead of introducing a type parameter like this...

#define min(x,y) \
({ typeof(x) __x = (x); typeof(y) __y = (y); __x < __y ? __x: __y; })

Best regards,

Anton

--
"Nothing succeeds like success." - Alexandre Dumas
--
Anton Altaparmakov <aia21 at cam.ac.uk> (replace at with @)
Linux NTFS Maintainer / WWW: http://linux-ntfs.sf.net/
ICQ: 8561279 / WWW: http://www-stu.christs.cam.ac.uk/~aia21/