Changes between Version 13 and Version 14 of CodingGuidelines/VIM

Timestamp:

Jul 8, 2010, 1:07:05 PM (14 years ago)

Author:

pulkomandy

Comment:

Add some rules that were in the python version but not here.

CodingGuidelines/VIM

@@ -6 +6 @@
 
 :fu FuncHaikuCheck()
-                call matchadd('Search', '\%>80v.\+', -1) " line with more than 80 chars
-        call matchadd('Search', '0\s* \s*', -1)        " probably wrong indenting
+        call matchadd('Search', '\%>80v.\+', -1) " line over 80 char
+        call matchadd('Search', '^\s* \s*', -1)  " spaces instead of tabs
         call matchadd('Search', '\(for\|if\|select\|while\)(', -1)
-                "keyword without space after it
-        call matchadd('Search', '[a-zA-Z0-9][,=>+\-*;][a-zA-Z0-9]', -1)
+                "missing space after control statement
+        call matchadd('Search', '//\S', -1) " Missing space at comment start
+        call matchadd('Search', '\w[,=>+\-*;]\w', -1)
                 "operator without space around it (without false positive on
                 "templated<type>)
-        call matchadd('Search', '^[^#].*[^<][a-zA-Z0-9]*/[a-zA-Z0-9]', -1)
+        call matchadd('Search', '^[^#].*[^<]\zs\w*/\w', -1)
                 "operator without space around it (without false positive on
                 "#include <dir/file.h>)
-        call matchadd('Search', '^[^/]\{2}.*[^*][=/+\- ]$', -1)
+        call matchadd('Search', '^[^/]\{2}.*\zs[^*][=/+\-< ]$', -1)
                 "operator at end of line (without false positives on /* and */, nor
                 "char*\nClass::method())
+        call matchadd('Search', '^[^#].*\zs[^<]>$', -1)
+                " > operator at end of line (without false positive on #include <file.h>)
+        call matchadd('Search', '){', -1) " Missing space after method header
+        call matchadd('Search', '}\n\s*else', -1) " Malformed else
         call matchadd('Search', '\s$', -1) "Spaces at end of line
         call matchadd('Search', ',\S', -1) " Missing space after comma
+        call matchadd('Search', '^}\n\{1,2}\S', -1) " Less than 2 lines between functions
+        call matchadd('Search', '^}\n\{4,}\S', -1) " More than 2 lines between functions
 :endfu
 }}}