--- 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 might almost be 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/
This archive was generated by hypermail 2b29 : Fri Aug 31 2001 - 21:00:15 EST