[Ardour-Dev] C and C++ Style checking now enforced

Paul Davis paul at linuxaudiosystems.com
Fri May 8 23:32:13 PDT 2015


This list gives a detailed overview of reasons why code will be rejected.
Debate on any of them is welcome, but unlikely to succeed.

( re.compile ("^ "),          "leading space as indentation instead of tab
- use tabs to indent, spaces to align" )
( re.compile ("{[^\s]"),      "missing space after open brace" )
( re.compile ("[^\s]}"),      "missing space before close brace" )
( re.compile ("[ \t]+$"),     "contains trailing whitespace" )
( re.compile (",[^\s\n]"),            "missing space after comma" )
( re.compile (";[a-zA-Z0-9]"),        "missing space after semi-colon" )
( re.compile ("=[^\s\"'=]"),          "missing space after assignment" )
( re.compile ("[^\s\(\[\*&']\("),             "missing space before open
parenthesis" )
( re.compile ("\)(-[^>]|[^,'\s\n\)\]-])"),    "missing space after close
parenthesis" )
( re.compile ("\( [^;]"),                     "space after open
parenthesis" )
( re.compile ("[^;] \)"),                     "space before close
parenthesis" )
( re.compile ("\[ "),                                 "space after open
square brace" )
( re.compile (" \]"),                                 "space before close
square brace" )
( re.compile ("[^\s][\*/%+-][=][^\s]"),               "missing space around
opassign" )
( re.compile ("[^\s][<>!=^/][=]{1,2}[^\s]"),  "missing space around
comparison" )
( re.compile ("\s+return\s+\([a-zA-Z0-9_]+\)\s+;"),   "parens around return
value" )
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ardour.org/pipermail/ardour-dev-ardour.org/attachments/20150509/de1708dd/attachment.htm>


More information about the Ardour-Dev mailing list