[email protected] | 8b8d8be | 2011-09-08 15:34:45 | [diff] [blame] | 1 | # Copyright (c) 2011 Google Inc. All rights reserved. |
| 2 | # Copyright (c) 2009 Torch Mobile Inc. |
| 3 | # |
| 4 | # Redistribution and use in source and binary forms, with or without |
| 5 | # modification, are permitted provided that the following conditions are |
| 6 | # met: |
| 7 | # |
| 8 | # * Redistributions of source code must retain the above copyright |
| 9 | # notice, this list of conditions and the following disclaimer. |
| 10 | # * Redistributions in binary form must reproduce the above |
| 11 | # copyright notice, this list of conditions and the following disclaimer |
| 12 | # in the documentation and/or other materials provided with the |
| 13 | # distribution. |
| 14 | # * Neither the name of Google Inc. nor the names of its |
| 15 | # contributors may be used to endorse or promote products derived from |
| 16 | # this software without specific prior written permission. |
| 17 | # |
| 18 | # THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS |
| 19 | # "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT |
| 20 | # LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR |
| 21 | # A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT |
| 22 | # OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, |
| 23 | # SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT |
| 24 | # LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, |
| 25 | # DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY |
| 26 | # THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT |
| 27 | # (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE |
| 28 | # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. |
| 29 | |
| 30 | import re |
| 31 | |
| 32 | # Matches Foo *foo declarations. |
| 33 | _RE_PATTERN_POINTER_DECLARATION_WHITESPACE = re.compile( |
| 34 | r'\s*\w+(?<!\breturn|\bdelete)\s+(?P<pointer_operator>\*|\&)\w+') |
| 35 | |
| 36 | def CheckPointerDeclarationWhitespace(filename, clean_lines, linenum, error): |
| 37 | """Checks for Foo *foo declarations. |
| 38 | |
| 39 | Args: |
| 40 | filename: The name of the current file. |
| 41 | clean_lines: A CleansedLines instance containing the file. |
| 42 | linenum: The number of the line to check. |
| 43 | error: The function to call with any errors found. |
| 44 | """ |
| 45 | line = clean_lines.elided[linenum] |
| 46 | matched = _RE_PATTERN_POINTER_DECLARATION_WHITESPACE.match(line) |
| 47 | if matched: |
| 48 | error(filename, linenum, 'whitespace/declaration', 3, |
| 49 | 'Declaration has space between type name and %s in %s' % |
| 50 | (matched.group('pointer_operator'), matched.group(0).strip())) |