On Sun, Oct 24, 2010 at 09:55:21AM +0700, Robert Elz wrote: > Date: Sat, 23 Oct 2010 16:33:37 +0100 > From: Matthias Scheler <t...@netbsd.org> > Message-ID: <20101023153336.ga...@colwyn.zhadum.org.uk> > > | I'm sorry but this whole code is horrible. > > I agree with that, but ... > > | What is wrong with something as simple as this? > > | +#define _FPOS_OVERFLOW(pos) ((pos) < LONG_MIN || (pos) > LONG_MAX) > > That evaluates its arg 0, 1 or 2 times -- 0 if you're right and the wole > thing is sometimes optimised away, 1 if the value is < LONG_MIN, and twice > (most commonly) otherwise. > > Macros that act like that without a very good reason are pure evil.
It doesn't have to be a macro anyway as it used only ones. Kind regards -- Matthias Scheler http://zhadum.org.uk/