[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-core-devel
Subject: Re: Help with making stuff compile on non-Linux
From: Alex Merry <huntedhacker () tiscali ! co ! uk>
Date: 2008-07-24 17:35:01
Message-ID: 200807241835.01539.huntedhacker () tiscali ! co ! uk
[Download RAW message or body]
On Thursday 24 July 2008 08:05:25 Sebastian Trüg wrote:
> On Wednesday 23 July 2008 22:10:21 Alex Merry wrote:
> > I don't really see why you need to bother with the HAVE_* macros - why
> > not just use #ifdef SCHED_BATCH and #ifdef SYS_ioprio_set directly?
> >
> > IMO, this make the code more readable, because it's immediately obvious
> > what the #ifdefs guard against. The HAVE_* macros are really for CMake
> > checks.
>
> Only one reason: in case there is a fancier check one would only have to
> replace it in one place.
But each check is only used in one place. And since whether SCHED_BATCH and
SYS_ioprio_set are defined are exactly what you want to check, there are no
possible fancier checks that could do a better job. So it just looks like
needless obfuscation to me.
Alex
--
KDE: http://www.kde.org
Ubuntu/Kubuntu: http://www.ubuntu.org http://www.kubuntu.org
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic