linters/check-cpp: Demote #if lints to warnings
authorBen Gamari <ben@smart-cactus.org>
Fri, 21 Apr 2017 16:27:51 +0000 (12:27 -0400)
committerBen Gamari <ben@smart-cactus.org>
Fri, 21 Apr 2017 16:27:51 +0000 (12:27 -0400)
Errors trigger even for lines which the author didn't touch, which is undesired.

.arc-linters/check-cpp.py

index 1bbcbbd..a442e26 100755 (executable)
@@ -69,11 +69,14 @@ linters = [
     RegexpLinter(br'ASSERT\s+\(',
                  message='CPP macros should not have a space between the macro name and their argument list'),
     RegexpLinter(br'#ifdef\s+',
-                 message='`#if defined(x)` is preferred to `#ifdef x`'),
+                 message='`#if defined(x)` is preferred to `#ifdef x`',
+                 severity='warning'),
     RegexpLinter(br'#if\s+defined\s+',
-                 message='`#if defined(x)` is preferred to `#if defined x`'),
+                 message='`#if defined(x)` is preferred to `#if defined x`',
+                 severity='warning'),
     RegexpLinter(br'#ifndef\s+',
-                 message='`#if !defined(x)` is preferred to `#ifndef x`'),
+                 message='`#if !defined(x)` is preferred to `#ifndef x`',
+                 severity='warning'),
 ]
 
 if __name__ == '__main__':