|
1 # Copyright (C) 2010 Google Inc. All rights reserved. |
|
2 # Copyright (C) 2010 Chris Jerdonek (chris.jerdonek@gmail.com) |
|
3 # Copyright (C) 2010 ProFUSION embedded systems |
|
4 # |
|
5 # Redistribution and use in source and binary forms, with or without |
|
6 # modification, are permitted provided that the following conditions are |
|
7 # met: |
|
8 # |
|
9 # * Redistributions of source code must retain the above copyright |
|
10 # notice, this list of conditions and the following disclaimer. |
|
11 # * Redistributions in binary form must reproduce the above |
|
12 # copyright notice, this list of conditions and the following disclaimer |
|
13 # in the documentation and/or other materials provided with the |
|
14 # distribution. |
|
15 # * Neither the name of Google Inc. nor the names of its |
|
16 # contributors may be used to endorse or promote products derived from |
|
17 # this software without specific prior written permission. |
|
18 # |
|
19 # THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS |
|
20 # "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT |
|
21 # LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR |
|
22 # A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT |
|
23 # OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, |
|
24 # SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT |
|
25 # LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, |
|
26 # DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY |
|
27 # THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT |
|
28 # (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE |
|
29 # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. |
|
30 |
|
31 import logging |
|
32 |
|
33 from webkitpy.common.checkout.diff_parser import DiffParser |
|
34 |
|
35 |
|
36 _log = logging.getLogger("webkitpy.style.patchreader") |
|
37 |
|
38 |
|
39 class PatchReader(object): |
|
40 |
|
41 """Supports checking style in patches.""" |
|
42 |
|
43 def __init__(self, text_file_reader): |
|
44 """Create a PatchReader instance. |
|
45 |
|
46 Args: |
|
47 text_file_reader: A TextFileReader instance. |
|
48 |
|
49 """ |
|
50 self._text_file_reader = text_file_reader |
|
51 |
|
52 def check(self, patch_string): |
|
53 """Check style in the given patch.""" |
|
54 patch_files = DiffParser(patch_string.splitlines()).files |
|
55 |
|
56 # The diff variable is a DiffFile instance. |
|
57 for path, diff in patch_files.iteritems(): |
|
58 line_numbers = set() |
|
59 for line in diff.lines: |
|
60 # When deleted line is not set, it means that |
|
61 # the line is newly added (or modified). |
|
62 if not line[0]: |
|
63 line_numbers.add(line[1]) |
|
64 |
|
65 _log.debug('Found %s new or modified lines in: %s' |
|
66 % (len(line_numbers), path)) |
|
67 |
|
68 # If line_numbers is empty, the file has no new or |
|
69 # modified lines. In this case, we don't check the file |
|
70 # because we'll never output errors for the file. |
|
71 # This optimization also prevents the program from exiting |
|
72 # due to a deleted file. |
|
73 if line_numbers: |
|
74 self._text_file_reader.process_file(file_path=path, |
|
75 line_numbers=line_numbers) |