Changed the CheckBaseMacrosInHeaders presubmit to pass over C++ comments

PRESUBMIT.py

Index: PRESUBMIT.py
diff --git a/PRESUBMIT.py b/PRESUBMIT.py
index 8d978fffe9649cf9a9c2d142254181b8281bf8b7..41e9703a5db1d10ff86406d210d953f028153ce5 100644
--- a/PRESUBMIT.py
+++ b/PRESUBMIT.py
@@ -1529,6 +1529,8 @@ def _CheckBaseMacrosInHeaders(input_api, output_api):
     if not f.LocalPath().endswith('.h'):
       continue
     for line_num, line in f.ChangedContents():
+      if input_api.re.match(r'//', line): # Strip C++ comment.

[Dan Beam, 2015/10/19 22:19:36]

this would only match lines that start with "//" and no spaces (I'm pretty
sure).  that's why I didn't port it from the other check.  alternatives:

  if input_api.re.search(r'//', line):  # Strip C++ comment.

(match -> search) OR

  if line.lstrip().startswith('//'):  # Strip C++ comment.

(also note: \s\s# instead of \s#)

personally prefer second line

also maybe update the other check?

[oystein (OOO til 10th of July), 2015/10/20 00:04:05]

Good catch; I just blindly copied the check from the other test. Done.

+        continue
       if any(d in line for d in disallows):
         contents = input_api.ReadFile(f)
         if not (macros in contents or basictypes in contents):