Skip to content

Commit

Permalink
Merge pull request #3214 from delroth/master
Browse files Browse the repository at this point in the history
Add linter script which will replace the ad-hoc commands ran by the linter Buildbot
  • Loading branch information
lioncash committed Nov 1, 2015
2 parents 839dd2c + 15ad14a commit d7b486e
Showing 1 changed file with 37 additions and 0 deletions.
37 changes: 37 additions & 0 deletions Tools/lint.sh
@@ -0,0 +1,37 @@
#! /bin/sh
#
# Linter script that checks for common style issues in Dolphin's codebase.

REPO_BASE=$(realpath $(dirname $0)/..)

fail=0

# Step 1: check for includes ordering.
echo "[.] Checking #include statements ordering."
res=$(
${REPO_BASE}/Tools/check-includes.py \
$(find ${REPO_BASE}/Source/Core -name "*.cpp" -o -name "*.h") \
2>&1 \
>/dev/null
)
if [ -n "${res}" ]; then
echo "FAIL: ${res}"
fail=1
else
echo "OK"
fi

# Step 2: check for trailing whitespaces.
echo "[.] Checking for trailing whitespaces."
res=$(
find ${REPO_BASE}/Source/Core -name "*.cpp" -o -name "*.h" \
| xargs egrep -n "\s+$"
)
if [ -n "$res" ]; then
echo "FAIL: ${res}"
fail=1
else
echo "OK"
fi

exit ${fail}

0 comments on commit d7b486e

Please sign in to comment.