Used but never set HAVE_FOO macros
Allen Winter
winter at kde.org
Sat Mar 21 15:19:28 GMT 2009
On Saturday 21 March 2009 10:29:21 am Allen Winter wrote:
> Howdy,
>
> Based on a discussion in the buildsystem mailing list, I am looking
> for HAVE_FOO macros we are using in our code, but not setting
> anywhere in the buildsystem.
>
> For kdeilbs, I found the following.
> Please
> fix the code;
> make a CMake check for the feature;
> or tell me I have false positive in my checker program
>
> kinit/proctitle.cpp: HAVE___PROGNAME, HAVE___PROGNAME_FULL, HAVE____PROGNAME_FULL
>
False positive for HAVE__PROGNAME and HAVE__PROGNAME_FULL. I fixed my checker.
However, there is a HAVE____PROGNAME_FULL used in proctitle.cpp that I don't
set anywhere. Definitely looks like an extra underscore. I hope someone
who knows this code will take a look.
> khtml/misc/borderarcstroker.cpp: HAVE_SSE
>
False positive. I'll fix.
>
> kdecore/network/k3socketdevice.cpp: HAVE_SYS_SELECT
>
I fixed this. should have been HAVE_SYS_SELECT_H
--
Allen Winter | Software Engineer | 1-888-872-9339
KDAB, Inc. | "Platform-independent software solutions"
http://kdab.com | 1-866-777-5322 (US) | +46-563-540090 (Sweden)
More information about the kde-core-devel
mailing list