From 2a43618264ce4871c4f9385eb50e7e1cdbd3ae7e Mon Sep 17 00:00:00 2001 From: Martin Quinson Date: Mon, 5 Dec 2016 20:45:09 +0100 Subject: [PATCH] fix the syntax in the sonar configuration (fix #126) I drop the #126 PR that propose to kill that config item all together because I really think that sonar should not feel too much concerned by our logging macros. Yes they increase the nesting level if you expend them, but nobody expends them mentaly. We use them as inline functions even if we cannot implement them as functions for technical reasons. --- sonar-project.properties | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sonar-project.properties b/sonar-project.properties index 933fb1ff50..4e2666cd56 100644 --- a/sonar-project.properties +++ b/sonar-project.properties @@ -41,8 +41,8 @@ sonar.cpp.std=gnu++11 sonar.c.std=gnu11 sonar.java.source=7 -# Disable the whole logging mechanism during SonarQube analysis -sonar.cfamily.predefinedMacros=XBT_LOG_MAYDAY +# Disable the whole logging mechanism during SonarQube analysis (our logging macro are crude, but should not harm Sonar) +sonar.cfamily.predefinedMacros=#define XBT_LOG_MAYDAY # Talk to me (travis don't like it, it's too much) # sonar.verbose=true -- 2.20.1