LINT fix & check all files with clang-format

Seems the diff mode doesn't work well, PR are detected as working whereas in master it's shown it's problematic (and really problematic). Use same check everywhere
This commit is contained in:
Loic Blot 2017-05-22 07:28:35 +02:00
parent 39f4a2f607
commit 05309229b8
No known key found for this signature in database
GPG Key ID: EFAA458E8C153987
2 changed files with 2 additions and 8 deletions

@ -88,7 +88,7 @@ int ModApiClient::l_display_chat_message(lua_State *L)
// send_chat_message(message) // send_chat_message(message)
int ModApiClient::l_send_chat_message(lua_State *L) int ModApiClient::l_send_chat_message(lua_State *L)
{ {
if (!lua_isstring(L,1)) if (!lua_isstring(L, 1))
return 0; return 0;
std::string message = luaL_checkstring(L, 1); std::string message = luaL_checkstring(L, 1);
getClient(L)->sendChatMessage(utf8_to_wide(message)); getClient(L)->sendChatMessage(utf8_to_wide(message));

@ -9,13 +9,7 @@ function perform_lint() {
echo "LINT: Using binary $CLANG_FORMAT" echo "LINT: Using binary $CLANG_FORMAT"
CLANG_FORMAT_WHITELIST="util/travis/clang-format-whitelist.txt" CLANG_FORMAT_WHITELIST="util/travis/clang-format-whitelist.txt"
if [ "$TRAVIS_EVENT_TYPE" = "pull_request" ]; then
# Get list of every file modified in this pull request
files_to_lint="$(git diff --name-only --diff-filter=ACMRTUXB $TRAVIS_COMMIT_RANGE | grep '^src/[^.]*[.]\(cpp\|h\)$' | true)"
else
# Check everything for branch pushes
files_to_lint="$(find src/ -name '*.cpp' -or -name '*.h')" files_to_lint="$(find src/ -name '*.cpp' -or -name '*.h')"
fi
local errorcount=0 local errorcount=0
local fail=0 local fail=0