#!/usr/bin/env python # # Checks C++ files to make sure they conform to LLVM standards, as specified in # http://llvm.org/docs/CodingStandards.html . # # TODO: add unittests for the verifier functions: # http://docs.python.org/library/unittest.html . from __future__ import print_function import common_lint import re import sys def VerifyIncludes(filename, lines): """Makes sure the #includes are in proper order and no disallows files are #included. Args: filename: the file under consideration as string lines: contents of the file as string array """ lint = [] include_gtest_re = re.compile(r'^#include "gtest/(.*)"') include_llvm_re = re.compile(r'^#include "llvm/(.*)"') include_support_re = re.compile(r'^#include "(Support/.*)"') include_config_re = re.compile(r'^#include "(Config/.*)"') include_system_re = re.compile(r"^#include <(.*)>") DISALLOWED_SYSTEM_HEADERS = ["iostream"] line_num = 1 prev_config_header = None prev_system_header = None for line in lines: # TODO: implement private headers # TODO: implement gtest headers # TODO: implement top-level llvm/* headers # TODO: implement llvm/Support/* headers # Process Config/* headers config_header = include_config_re.match(line) if config_header: curr_config_header = config_header.group(1) if prev_config_header: if prev_config_header > curr_config_header: lint.append( ( filename, line_num, 'Config headers not in order: "%s" before "%s"' % (prev_config_header, curr_config_header), ) ) # Process system headers system_header = include_system_re.match(line) if system_header: curr_system_header = system_header.group(1) # Is it disallowed? if curr_system_header in DISALLOWED_SYSTEM_HEADERS: lint.append( ( filename, line_num, "Disallowed system header: <%s>" % curr_system_header, ) ) elif prev_system_header: # Make sure system headers are alphabetized amongst themselves if prev_system_header > curr_system_header: lint.append( ( filename, line_num, "System headers not in order: <%s> before <%s>" % (prev_system_header, curr_system_header), ) ) prev_system_header = curr_system_header line_num += 1 return lint class CppLint(common_lint.BaseLint): MAX_LINE_LENGTH = 80 def RunOnFile(self, filename, lines): lint = [] lint.extend(VerifyIncludes(filename, lines)) lint.extend( common_lint.VerifyLineLength(filename, lines, CppLint.MAX_LINE_LENGTH) ) lint.extend(common_lint.VerifyTabs(filename, lines)) lint.extend(common_lint.VerifyTrailingWhitespace(filename, lines)) return lint def CppLintMain(filenames): all_lint = common_lint.RunLintOverAllFiles(CppLint(), filenames) for lint in all_lint: print("%s:%d:%s" % (lint[0], lint[1], lint[2])) return 0 if __name__ == "__main__": sys.exit(CppLintMain(sys.argv[1:]))