[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    New Krazy Check:  cpp macros
From:       Allen Winter <winter () kde ! org>
Date:       2007-06-19 16:29:44
Message-ID: 200706191229.44450.winter () kde ! org
[Download RAW message or body]

Howdy,

Tonight I plan to install a new Krazy check for cpp stuff we don't
want to have in our code.

Currently it only tests for O/S and compiler specific macros.
eg.
we don't want  #if (__sgi) to test if header foo.h should be included.
rather, we want  #if (HAVE_FOO_H)

Oh, but I don't check in installed headers because they
can't have <config-pkg.h> includes.

As usual, you can tell Krazy to ignore by using //krazy:exclude=cpp

IOW: it checks for these.  Please let me know what I missed.
  Q_OS_*
  _WIN32
  _WIN64
  __APPLE__
  __svr4__
  __linux__
  __bsdi__
  __FreeBSD__
  __NetBSD__
  __OpenBSD__
  __DragonFly__
  __GNU__
  __osf__
  __hpux
  __sun
  __sgi
  __CYGWIN__

  Q_CC_*
__GNUC__
__BORLANDC__

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic