From b04de2f2f4bb14c968ebcc765ec93c5d1591aa0d Mon Sep 17 00:00:00 2001 From: Dean Camera Date: Sun, 8 Apr 2012 11:30:28 +0000 Subject: [PATCH] Pull out common options in the StaticAnalysisTest build test makefile into a seperate make variable for clarity. Add missing cppcheck inline suppression comment. --- BuildTests/StaticAnalysisTest/makefile | 7 +++++-- Projects/XPLAINBridge/XPLAINBridge.h | 1 + 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/BuildTests/StaticAnalysisTest/makefile b/BuildTests/StaticAnalysisTest/makefile index b065a90320..817727ea34 100644 --- a/BuildTests/StaticAnalysisTest/makefile +++ b/BuildTests/StaticAnalysisTest/makefile @@ -26,6 +26,9 @@ SUPPRESS_WARNINGS = variableScope \ # Extra paths to search for include files INCLUDE_PATHS = $(LUFA_ROOT_PATH)/LUFA/CodeTemplates/ +# Configuration options to pass to cppcheck +CPPCHECK_OPTIONS = --template=$(MESSAGE_TEMPLATE) $(INCLUDE_PATHS:%=-I%) $(EXCLUDE_LIST:%=-i%) --inline-suppr --force --error-exitcode=1 --std=c99 + all: begin staticcheck end @@ -38,8 +41,8 @@ end: @echo staticcheck: - cppcheck --quiet --template=$(MESSAGE_TEMPLATE) $(INCLUDE_PATHS:%=-I%) $(EXCLUDE_LIST:%=-i%) --inline-suppr --force --check-config $(LUFA_ROOT_PATH) - cppcheck --quiet --template=$(MESSAGE_TEMPLATE) $(INCLUDE_PATHS:%=-I%) $(EXCLUDE_LIST:%=-i%) --inline-suppr --force --error-exitcode=1 --std=c99 --enable=all $(SUPPRESS_WARNINGS:%=--suppress=%) --inconclusive $(LUFA_ROOT_PATH) + cppcheck --quiet $(CPPCHECK_OPTIONS) --check-config $(LUFA_ROOT_PATH) + cppcheck --quiet $(CPPCHECK_OPTIONS) --enable=all $(SUPPRESS_WARNINGS:%=--suppress=%) $(LUFA_ROOT_PATH) %: \ No newline at end of file diff --git a/Projects/XPLAINBridge/XPLAINBridge.h b/Projects/XPLAINBridge/XPLAINBridge.h index f5c86c2f47..5570a1a0a6 100644 --- a/Projects/XPLAINBridge/XPLAINBridge.h +++ b/Projects/XPLAINBridge/XPLAINBridge.h @@ -48,6 +48,7 @@ // cppcheck-suppress missingInclude #include "Lib/V2Protocol.h" + // cppcheck-suppress missingInclude #include "Lib/SoftUART.h" #include