Function CheckStyle

cpplint.py:5423–5565

Checks rules from the 'C++ style rules' section of cppguide.html. Most of these rules are hard to test (naming, comment style), but we do what we can. In particular we check for 2-space indents, line lengths, tab usage, spaces inside code, etc. Args: filename: The name of th

(filename, clean_lines, linenum, file_extension, nesting_state, error, cppvar=None)

def CheckStyle(filename, clean_lines, linenum, file_extension, nesting_state, error, cppvar=None):
    """Checks rules from the 'C++ style rules' section of cppguide.html.

    Most of these rules are hard to test (naming, comment style), but we
    do what we can.  In particular we check for 2-space indents, line lengths,
    tab usage, spaces inside code, etc.

    Args:
      filename: The name of the current file.
      clean_lines: A CleansedLines instance containing the file.
      linenum: The number of the line to check.
      file_extension: The extension (without the dot) of the filename.
      nesting_state: A NestingState instance which maintains information about
                     the current stack of nested blocks being parsed.
      error: The function to call with any errors found.
      cppvar: The header guard variable returned by GetHeaderGuardCPPVar.
    """

    # Don't use "elided" lines here, otherwise we can't check commented lines.
    # Don't want to use "raw" either, because we don't want to check inside C++11
    # raw strings,
    raw_lines = clean_lines.lines_without_raw_strings
    line = raw_lines[linenum]
    prev = raw_lines[linenum - 1] if linenum > 0 else ""

    if line.find("\t") != -1:
        error(filename, linenum, "whitespace/tab", 1, "Tab found; better to use spaces")

    # One or three blank spaces at the beginning of the line is weird; it's
    # hard to reconcile that with 2-space indents.
    # NOTE: here are the conditions rob pike used for his tests.  Mine aren't
    # as sophisticated, but it may be worth becoming so:  RLENGTH==initial_spaces
    # if(RLENGTH > 20) complain = 0;
    # if(match($0, " +(error|private|public|protected):")) complain = 0;
    # if(match(prev, "&& *$")) complain = 0;
    # if(match(prev, "\\|\\| *$")) complain = 0;
    # if(match(prev, "[\",=><] *$")) complain = 0;
    # if(match($0, " <<")) complain = 0;
    # if(match(prev, " +for \\(")) complain = 0;
    # if(prevodd && match(prevprev, " +for \\(")) complain = 0;
    scope_or_label_pattern = r"\s*(?:public|private|protected|signals)(?:\s+(?:slots\s*)?)?:\s*\\?$"
    classinfo = nesting_state.InnermostClass()
    initial_spaces = 0
    cleansed_line = clean_lines.elided[linenum]
    while initial_spaces < len(line) and line[initial_spaces] == " ":
        initial_spaces += 1
    # There are certain situations we allow one space, notably for
    # section labels, and also lines containing multi-line raw strings.
    # We also don't check for lines that look like continuation lines
    # (of lines ending in double quotes, commas, equals, or angle brackets)
    # because the rules for how to indent those are non-trivial.
    if (
        not re.search(r'[",=><] *$', prev)
        and (initial_spaces in {1, 3})
        and not re.match(scope_or_label_pattern, cleansed_line)
        and not (clean_lines.raw_lines[linenum] != line and re.match(r'^\s*""', line))
    ):
        error(