Todd Vierling <[EMAIL PROTECTED]> writes:

> You have to bend over backwards to do the runtime check, and you sacrifice
> *maintainability* with the extra normally-dead code added to Wine.  I don't
> see a need to jump through extra hoops in this case.

I think that run-time checks are more maintainable that compile-time
checks, since it means the code gets exercised on all machines, so if
you break it it gets noticed at once. If you break something that's
inside a rarely used #ifdef block you may not find the problem until 6
months later. It's also a lot easier to follow the code if you don't
have to wonder which parts of it are actually compiled in.

Anyway it doesn't really matter for that specific issue, if you don't
feel the performance gain is worth the trouble that's your call. I was
just raising a general point that checking run-time features with
#ifdef is not a good idea, no matter how much autoconf encourages that
style.

-- 
Alexandre Julliard
[EMAIL PROTECTED]

Reply via email to