Browse Source

Don't overwrite SDL_dynapi.h in checker-buildbot.sh.

Ryan C. Gordon 11 years ago
parent
commit
7b893acdc1
2 changed files with 2 additions and 7 deletions
  1. 0 7
      build-scripts/checker-buildbot.sh
  2. 2 0
      src/dynapi/SDL_dynapi.h

+ 0 - 7
build-scripts/checker-buildbot.sh

@@ -8,8 +8,6 @@
 # And download checker at http://clang-analyzer.llvm.org/ and unpack it in
 #  /usr/local ... update CHECKERDIR as appropriate.
 
-# this currently expects a mercurial working copy that it can modify a little.
-
 FINALDIR="$1"
 
 CHECKERDIR="/usr/local/checker-276"
@@ -53,10 +51,6 @@ set -e
 cd `dirname "$0"`
 cd ..
 
-# Turn off the dynamic API. Makes the analysis output clearer.
-echo '#pragma once' >src/dynapi/SDL_dynapi.h
-echo '#define SDL_DYNAMIC_API 0' >>src/dynapi/SDL_dynapi.h
-
 rm -rf checker-buildbot analysis
 if [ ! -z "$FINALDIR" ]; then
     rm -rf "$FINALDIR"
@@ -84,7 +78,6 @@ else
 fi
 
 rm -rf checker-buildbot
-hg revert src/dynapi/SDL_dynapi.h
 
 echo "Done. Final output is in '$FINALDIR' ..."
 

+ 2 - 0
src/dynapi/SDL_dynapi.h

@@ -47,6 +47,8 @@
 #define SDL_DYNAMIC_API 0
 #elif SDL_BUILDING_WINRT /* probaly not useful on WinRT, given current .dll loading restrictions */
 #define SDL_DYNAMIC_API 0
+#elif defined(__clang_analyzer__)
+#define SDL_DYNAMIC_API 0  /* Turn off for static analysis, so reports are more clear. */
 #else   /* everyone else. */
 #define SDL_DYNAMIC_API 1
 #endif