summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>2018-12-18 22:34:50 +0200
committerLaurent Pinchart <laurent.pinchart@ideasonboard.com>2018-12-19 14:11:16 +0200
commitcf04a496a6e70d180446b519a7f90618b36b4479 (patch)
tree14dfb234ada8a6c02210f78b991be3ee9d52a64d
parent2887934ca0385fa7da6d4c1a9654010b979708ab (diff)
utils: checkstyle.py: Highlight trailing white space at end of line
In order to facilitate interpretation of diffs, highlight trailing white space at end of lines with a red background. Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
-rwxr-xr-xutils/checkstyle.py67
1 files changed, 51 insertions, 16 deletions
diff --git a/utils/checkstyle.py b/utils/checkstyle.py
index b7c4d876..dadb55df 100755
--- a/utils/checkstyle.py
+++ b/utils/checkstyle.py
@@ -40,18 +40,43 @@ source_extensions = (
class Colours:
Default = 0
+ Black = 0
Red = 31
Green = 32
+ Yellow = 33
+ Blue = 34
+ Magenta = 35
Cyan = 36
+ LightGrey = 37
+ DarkGrey = 90
+ LightRed = 91
+ LightGreen = 92
+ Lightyellow = 93
+ LightBlue = 94
+ LightMagenta = 95
+ LightCyan = 96
+ White = 97
+
+ @staticmethod
+ def fg(colour):
+ if sys.stdout.isatty():
+ return '\033[%um' % colour
+ else:
+ return ''
-for attr in Colours.__dict__.keys():
- if attr.startswith('_'):
- continue
+ @staticmethod
+ def bg(colour):
+ if sys.stdout.isatty():
+ return '\033[%um' % (colour + 10)
+ else:
+ return ''
- if sys.stdout.isatty():
- setattr(Colours, attr, '\033[0;%um' % getattr(Colours, attr))
- else:
- setattr(Colours, attr, '')
+ @staticmethod
+ def reset():
+ if sys.stdout.isatty():
+ return '\033[0m'
+ else:
+ return ''
class DiffHunkSide(object):
@@ -82,20 +107,30 @@ class DiffHunk(object):
def __repr__(self):
s = '%s@@ -%u,%u +%u,%u @@\n' % \
- (Colours.Cyan,
+ (Colours.fg(Colours.Cyan),
self.__from.start, len(self.__from),
self.__to.start, len(self.__to))
for line in self.lines:
if line[0] == '-':
- s += Colours.Red
+ s += Colours.fg(Colours.Red)
elif line[0] == '+':
- s += Colours.Green
- else:
- s += Colours.Default
+ s += Colours.fg(Colours.Green)
+
+ if line[0] == '-':
+ spaces = 0
+ for i in range(len(line)):
+ if line[-i-1].isspace():
+ spaces += 1
+ else:
+ break
+ spaces = len(line) - spaces
+ line = line[0:spaces] + Colours.bg(Colours.Red) + line[spaces:]
+
s += line
+ s += Colours.reset()
+ s += '\n'
- s += Colours.Default
return s
def append(self, line):
@@ -111,7 +146,7 @@ class DiffHunk(object):
self.__to.touched.append(self.__to_line)
self.__to_line += 1
- self.lines.append(line)
+ self.lines.append(line.rstrip('\n'))
def intersects(self, lines):
for line in lines:
@@ -180,8 +215,8 @@ def check_file(top_level, commit, filename):
if len(formatted_diff) == 0:
return 0
- print('%s---' % Colours.Red, filename)
- print('%s+++' % Colours.Green, filename)
+ print('%s---' % Colours.fg(Colours.Red), filename)
+ print('%s+++' % Colours.fg(Colours.Green), filename)
for hunk in formatted_diff:
print(hunk)