blob: 54347ce7593a665fe6b5e571501467d57adb1a0d [file] [log] [blame]
Ed Schoutenbf041d92014-09-02 20:59:13 +00001#!/usr/bin/env python
Daniel Jasper9be2c5c2013-03-20 09:53:23 +00002#
3#===- clang-format-diff.py - ClangFormat Diff Reformatter ----*- python -*--===#
4#
5# The LLVM Compiler Infrastructure
6#
7# This file is distributed under the University of Illinois Open Source
8# License. See LICENSE.TXT for details.
9#
10#===------------------------------------------------------------------------===#
11
12r"""
13ClangFormat Diff Reformatter
14============================
15
16This script reads input from a unified diff and reformats all the changed
17lines. This is useful to reformat all the lines touched by a specific patch.
Daniel Jasper2e8600c2014-05-14 09:36:11 +000018Example usage for git/svn users:
Daniel Jasper9be2c5c2013-03-20 09:53:23 +000019
Sylvestre Ledruf4214032016-12-03 23:22:45 +000020 git diff -U0 --no-color HEAD^ | clang-format-diff.py -p1 -i
Daniel Jasper2e8600c2014-05-14 09:36:11 +000021 svn diff --diff-cmd=diff -x-U0 | clang-format-diff.py -i
Daniel Jasper9be2c5c2013-03-20 09:53:23 +000022
23"""
Serge Gueltonb748c0e2018-12-18 16:07:37 +000024from __future__ import absolute_import, division, print_function
Daniel Jasper9be2c5c2013-03-20 09:53:23 +000025
26import argparse
Alexander Kornienko95c009a2013-10-11 21:32:01 +000027import difflib
Daniel Jasper9be2c5c2013-03-20 09:53:23 +000028import re
29import subprocess
30import sys
Krasimir Georgievcb4dfae2018-08-03 10:04:58 +000031try:
32 from StringIO import StringIO
33except ImportError:
34 from io import StringIO
Daniel Jasper9be2c5c2013-03-20 09:53:23 +000035
36
Daniel Jasper9be2c5c2013-03-20 09:53:23 +000037def main():
38 parser = argparse.ArgumentParser(description=
Alexander Kornienko95c009a2013-10-11 21:32:01 +000039 'Reformat changed lines in diff. Without -i '
Alp Toker0e7f6da2013-12-04 00:48:22 +000040 'option just output the diff that would be '
Alexander Kornienko95c009a2013-10-11 21:32:01 +000041 'introduced.')
42 parser.add_argument('-i', action='store_true', default=False,
43 help='apply edits to files instead of displaying a diff')
Alp Toker8c7cbdf2013-12-10 13:51:53 +000044 parser.add_argument('-p', metavar='NUM', default=0,
Daniel Jasper9be2c5c2013-03-20 09:53:23 +000045 help='strip the smallest prefix containing P slashes')
Alp Toker679bf012013-12-18 21:34:07 +000046 parser.add_argument('-regex', metavar='PATTERN', default=None,
Alexander Kornienko25c68382013-12-16 10:57:30 +000047 help='custom pattern selecting file paths to reformat '
Daniel Jasperdb7933a2013-12-19 10:21:37 +000048 '(case sensitive, overrides -iregex)')
Alexander Kornienko25c68382013-12-16 10:57:30 +000049 parser.add_argument('-iregex', metavar='PATTERN', default=
Daniel Jasper8c68a642015-03-11 14:58:38 +000050 r'.*\.(cpp|cc|c\+\+|cxx|c|cl|h|hpp|m|mm|inc|js|ts|proto'
Daniel Jasper7c627a02014-12-08 19:39:03 +000051 r'|protodevel|java)',
Alexander Kornienko25c68382013-12-16 10:57:30 +000052 help='custom pattern selecting file paths to reformat '
Daniel Jasperdb7933a2013-12-19 10:21:37 +000053 '(case insensitive, overridden by -regex)')
Daniel Jasperdb125cb2015-10-07 17:00:20 +000054 parser.add_argument('-sort-includes', action='store_true', default=False,
55 help='let clang-format sort include blocks')
Daniel Jasperc6706882014-11-14 13:27:28 +000056 parser.add_argument('-v', '--verbose', action='store_true',
57 help='be more verbose, ineffective without -i')
Daniel Jasper18b1de32016-01-20 18:55:57 +000058 parser.add_argument('-style',
59 help='formatting style to apply (LLVM, Google, Chromium, '
60 'Mozilla, WebKit)')
61 parser.add_argument('-binary', default='clang-format',
62 help='location of binary to use for clang-format')
Daniel Jasper9be2c5c2013-03-20 09:53:23 +000063 args = parser.parse_args()
64
Daniel Jasperdcab7fb2013-09-18 12:14:09 +000065 # Extract changed lines for each file.
Daniel Jasper9be2c5c2013-03-20 09:53:23 +000066 filename = None
Daniel Jasperdcab7fb2013-09-18 12:14:09 +000067 lines_by_file = {}
Daniel Jasper9be2c5c2013-03-20 09:53:23 +000068 for line in sys.stdin:
69 match = re.search('^\+\+\+\ (.*?/){%s}(\S*)' % args.p, line)
70 if match:
71 filename = match.group(2)
72 if filename == None:
73 continue
74
Alp Toker679bf012013-12-18 21:34:07 +000075 if args.regex is not None:
76 if not re.match('^%s$' % args.regex, filename):
Alexander Kornienko25c68382013-12-16 10:57:30 +000077 continue
78 else:
Alp Toker679bf012013-12-18 21:34:07 +000079 if not re.match('^%s$' % args.iregex, filename, re.IGNORECASE):
Alexander Kornienko25c68382013-12-16 10:57:30 +000080 continue
Daniel Jasperdcab7fb2013-09-18 12:14:09 +000081
Daniel Jasper9be2c5c2013-03-20 09:53:23 +000082 match = re.search('^@@.*\+(\d+)(,(\d+))?', line)
83 if match:
Daniel Jasperdcab7fb2013-09-18 12:14:09 +000084 start_line = int(match.group(1))
Daniel Jasper164c8e12013-10-02 13:59:03 +000085 line_count = 1
Daniel Jasper9be2c5c2013-03-20 09:53:23 +000086 if match.group(3):
Daniel Jasper164c8e12013-10-02 13:59:03 +000087 line_count = int(match.group(3))
88 if line_count == 0:
89 continue
Krasimir Georgievcb4dfae2018-08-03 10:04:58 +000090 end_line = start_line + line_count - 1
Daniel Jasperdcab7fb2013-09-18 12:14:09 +000091 lines_by_file.setdefault(filename, []).extend(
92 ['-lines', str(start_line) + ':' + str(end_line)])
Daniel Jasper9be2c5c2013-03-20 09:53:23 +000093
Daniel Jasperdcab7fb2013-09-18 12:14:09 +000094 # Reformat files containing changes in place.
Krasimir Georgievcb4dfae2018-08-03 10:04:58 +000095 for filename, lines in lines_by_file.items():
Daniel Jasperc6706882014-11-14 13:27:28 +000096 if args.i and args.verbose:
Krasimir Georgievcb4dfae2018-08-03 10:04:58 +000097 print('Formatting {}'.format(filename))
Daniel Jasper18b1de32016-01-20 18:55:57 +000098 command = [args.binary, filename]
Alexander Kornienko95c009a2013-10-11 21:32:01 +000099 if args.i:
100 command.append('-i')
Daniel Jasperdb125cb2015-10-07 17:00:20 +0000101 if args.sort_includes:
102 command.append('-sort-includes')
Daniel Jasperdcab7fb2013-09-18 12:14:09 +0000103 command.extend(lines)
104 if args.style:
Daniel Jaspercf627f02013-09-21 10:05:02 +0000105 command.extend(['-style', args.style])
Krasimir Georgievcb4dfae2018-08-03 10:04:58 +0000106 p = subprocess.Popen(command,
107 stdout=subprocess.PIPE,
108 stderr=None,
109 stdin=subprocess.PIPE,
110 universal_newlines=True)
Daniel Jasperdcab7fb2013-09-18 12:14:09 +0000111 stdout, stderr = p.communicate()
Daniel Jaspere8845ad2013-10-08 15:54:36 +0000112 if p.returncode != 0:
Krasimir Georgievcb4dfae2018-08-03 10:04:58 +0000113 sys.exit(p.returncode)
Daniel Jasper9be2c5c2013-03-20 09:53:23 +0000114
Alexander Kornienko95c009a2013-10-11 21:32:01 +0000115 if not args.i:
116 with open(filename) as f:
117 code = f.readlines()
Krasimir Georgievcb4dfae2018-08-03 10:04:58 +0000118 formatted_code = StringIO(stdout).readlines()
Alexander Kornienko95c009a2013-10-11 21:32:01 +0000119 diff = difflib.unified_diff(code, formatted_code,
120 filename, filename,
121 '(before formatting)', '(after formatting)')
Krasimir Georgievcb4dfae2018-08-03 10:04:58 +0000122 diff_string = ''.join(diff)
Alexander Kornienko95c009a2013-10-11 21:32:01 +0000123 if len(diff_string) > 0:
Alp Tokerfcf30322013-12-05 08:14:54 +0000124 sys.stdout.write(diff_string)
Daniel Jasper9be2c5c2013-03-20 09:53:23 +0000125
126if __name__ == '__main__':
127 main()