Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Don't overwrite SDL_dynapi.h in checker-buildbot.sh.
  • Loading branch information
icculus committed Mar 19, 2014
1 parent 055d567 commit 7b893ac
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 7 deletions.
7 changes: 0 additions & 7 deletions build-scripts/checker-buildbot.sh
Expand Up @@ -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"
Expand Down Expand Up @@ -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"
Expand Down Expand Up @@ -84,7 +78,6 @@ else
fi

rm -rf checker-buildbot
hg revert src/dynapi/SDL_dynapi.h

echo "Done. Final output is in '$FINALDIR' ..."

Expand Down
2 changes: 2 additions & 0 deletions src/dynapi/SDL_dynapi.h
Expand Up @@ -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
Expand Down

0 comments on commit 7b893ac

Please sign in to comment.