← Back to team overview

widelands-dev team mailing list archive

[Merge] lp:~hjd/widelands/codecheck-whitespace into lp:widelands

 

Hans Joachim Desserud has proposed merging lp:~hjd/widelands/codecheck-whitespace into lp:widelands.

Requested reviews:
  Widelands Developers (widelands-dev)

For more details, see:
https://code.launchpad.net/~hjd/widelands/codecheck-whitespace/+merge/267431

See commit message for details.

Codecheck works fine again with Python3 after applying this patch.
-- 
Your team Widelands Developers is requested to review the proposed merge of lp:~hjd/widelands/codecheck-whitespace into lp:widelands.
=== modified file 'cmake/codecheck/rules/no_using_namespace_in_header_files'
--- cmake/codecheck/rules/no_using_namespace_in_header_files	2015-07-31 14:57:43 +0000
+++ cmake/codecheck/rules/no_using_namespace_in_header_files	2015-08-08 11:07:30 +0000
@@ -5,7 +5,7 @@
     if "system_headers.h" in fn:
         return []
     if not fn.endswith(".h"):
-		 return []
+        return []
     for lineno, line in enumerate(lines, 1):
         if "using namespace" in line:
             return [ (fn, lineno,