Re: [IDEA+RFC] Possible solution for min()/max() war

Brad Chapman (kakadu_croc@yahoo.com)
Fri, 24 Aug 2001 16:59:27 -0700 (PDT)


--- Ben LaHaise <bcrl@redhat.com> wrote:
> On Fri, 24 Aug 2001, Brad Chapman wrote:
>
> > This way, some hackers can use the two-arg min()/max() inside an #ifdef block,
> > other hackers can use the three-arg min()/max() inside an #ifdef block,
> > and people who don't care can select either.
>
> Have you no taste? Use of #ifdef's should be minimised as much as
> possible. For this kind of construct, the spurious preprocessor usage
> just makes me want to vomit.
>
> -ben

Mr. LaHaise,

Eeek! Sorry. That was just a preliminary. As I see it, there are
several ways to wire this up:

- make everyone use the new macros (some people are thinking :P)

- make everyone use #ifdef blocks with a config option (you think it's :P)

- make min()/max() typeof() wrappers for a switch-style stack of comparison
functions which work on the various types, i.e:

__u8 minimum = min(one, two) -> __u8 minimum = __u8_min(one, two)

(this may be too complex and is probably :P)

- make min()/max() inline functions, cast things to void, and use memcmp()
(this sounds reasonable, but is probably also :P)

- stay with the old-style macros (:P, :P, :P)

What do you think, sir?

Brad

=====
Brad Chapman

Permanent e-mail: kakadu_croc@yahoo.com
Current e-mail: kakadu@adelphia.net
Alternate e-mail: kakadu@netscape.net

__________________________________________________
Do You Yahoo!?
Make international calls for as low as $.04/minute with Yahoo! Messenger
http://phonecard.yahoo.com/
-
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/