blob: df5456e6f5b2bc5381170b6767c13e0d3a00d3f6 [file] [log] [blame]
Martin v. Löwis5e37bae2008-03-19 04:43:46 +00001# Copyright 2006 Google, Inc. All Rights Reserved.
2# Licensed to PSF under a Contributor Agreement.
3
4"""Refactoring framework.
5
6Used as a main program, this can refactor any number of files and/or
7recursively descend down directories. Imported as a module, this
8provides infrastructure to write your own refactoring tool.
9"""
10
11__author__ = "Guido van Rossum <guido@python.org>"
12
13
14# Python imports
15import os
16import sys
Martin v. Löwis5e37bae2008-03-19 04:43:46 +000017import logging
Benjamin Petersoneb55fd82008-09-03 00:21:32 +000018import operator
Benjamin Peterson840077c2009-07-20 15:33:09 +000019import collections
20import StringIO
Martin v. Löwis6780a9d2008-05-02 21:30:20 +000021from itertools import chain
Martin v. Löwis5e37bae2008-03-19 04:43:46 +000022
23# Local imports
Benjamin Peterson840077c2009-07-20 15:33:09 +000024from .pgen2 import driver, tokenize, token
Benjamin Peterson61180402009-06-11 22:06:46 +000025from . import pytree, pygram
Martin v. Löwis5e37bae2008-03-19 04:43:46 +000026
Martin v. Löwis5e37bae2008-03-19 04:43:46 +000027
Benjamin Petersoneb55fd82008-09-03 00:21:32 +000028def get_all_fix_names(fixer_pkg, remove_prefix=True):
29 """Return a sorted list of all available fix names in the given package."""
30 pkg = __import__(fixer_pkg, [], [], ["*"])
31 fixer_dir = os.path.dirname(pkg.__file__)
Martin v. Löwis5e37bae2008-03-19 04:43:46 +000032 fix_names = []
Benjamin Peterson6ae94ee2008-10-15 23:10:28 +000033 for name in sorted(os.listdir(fixer_dir)):
Martin v. Löwis5e37bae2008-03-19 04:43:46 +000034 if name.startswith("fix_") and name.endswith(".py"):
Benjamin Petersoneb55fd82008-09-03 00:21:32 +000035 if remove_prefix:
36 name = name[4:]
37 fix_names.append(name[:-3])
Martin v. Löwis5e37bae2008-03-19 04:43:46 +000038 return fix_names
39
Benjamin Peterson840077c2009-07-20 15:33:09 +000040
41class _EveryNode(Exception):
42 pass
43
44
45def _get_head_types(pat):
Martin v. Löwis6780a9d2008-05-02 21:30:20 +000046 """ Accepts a pytree Pattern Node and returns a set
47 of the pattern types which will match first. """
48
49 if isinstance(pat, (pytree.NodePattern, pytree.LeafPattern)):
50 # NodePatters must either have no type and no content
51 # or a type and content -- so they don't get any farther
52 # Always return leafs
Benjamin Peterson840077c2009-07-20 15:33:09 +000053 if pat.type is None:
54 raise _EveryNode
Martin v. Löwis6780a9d2008-05-02 21:30:20 +000055 return set([pat.type])
56
57 if isinstance(pat, pytree.NegatedPattern):
58 if pat.content:
Benjamin Peterson840077c2009-07-20 15:33:09 +000059 return _get_head_types(pat.content)
60 raise _EveryNode # Negated Patterns don't have a type
Martin v. Löwis6780a9d2008-05-02 21:30:20 +000061
62 if isinstance(pat, pytree.WildcardPattern):
63 # Recurse on each node in content
64 r = set()
65 for p in pat.content:
66 for x in p:
Benjamin Peterson840077c2009-07-20 15:33:09 +000067 r.update(_get_head_types(x))
Martin v. Löwis6780a9d2008-05-02 21:30:20 +000068 return r
69
70 raise Exception("Oh no! I don't understand pattern %s" %(pat))
71
Benjamin Peterson840077c2009-07-20 15:33:09 +000072
73def _get_headnode_dict(fixer_list):
Martin v. Löwis6780a9d2008-05-02 21:30:20 +000074 """ Accepts a list of fixers and returns a dictionary
75 of head node type --> fixer list. """
Benjamin Peterson840077c2009-07-20 15:33:09 +000076 head_nodes = collections.defaultdict(list)
77 every = []
Martin v. Löwis6780a9d2008-05-02 21:30:20 +000078 for fixer in fixer_list:
Benjamin Peterson840077c2009-07-20 15:33:09 +000079 if fixer.pattern:
80 try:
81 heads = _get_head_types(fixer.pattern)
82 except _EveryNode:
83 every.append(fixer)
84 else:
85 for node_type in heads:
86 head_nodes[node_type].append(fixer)
87 else:
88 if fixer._accept_type is not None:
89 head_nodes[fixer._accept_type].append(fixer)
90 else:
91 every.append(fixer)
92 for node_type in chain(pygram.python_grammar.symbol2number.itervalues(),
93 pygram.python_grammar.tokens):
94 head_nodes[node_type].extend(every)
95 return dict(head_nodes)
96
Martin v. Löwis6780a9d2008-05-02 21:30:20 +000097
Benjamin Petersoneb55fd82008-09-03 00:21:32 +000098def get_fixers_from_package(pkg_name):
99 """
100 Return the fully qualified names for fixers in the package pkg_name.
101 """
102 return [pkg_name + "." + fix_name
103 for fix_name in get_all_fix_names(pkg_name, False)]
104
Benjamin Peterson84ad84e2009-05-09 01:01:14 +0000105def _identity(obj):
106 return obj
107
108if sys.version_info < (3, 0):
109 import codecs
110 _open_with_encoding = codecs.open
111 # codecs.open doesn't translate newlines sadly.
112 def _from_system_newlines(input):
113 return input.replace(u"\r\n", u"\n")
114 def _to_system_newlines(input):
115 if os.linesep != "\n":
116 return input.replace(u"\n", os.linesep)
117 else:
118 return input
119else:
120 _open_with_encoding = open
121 _from_system_newlines = _identity
122 _to_system_newlines = _identity
123
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000124
Benjamin Peterson840077c2009-07-20 15:33:09 +0000125def _detect_future_print(source):
126 have_docstring = False
127 gen = tokenize.generate_tokens(StringIO.StringIO(source).readline)
128 def advance():
129 tok = next(gen)
130 return tok[0], tok[1]
131 ignore = frozenset((token.NEWLINE, tokenize.NL, token.COMMENT))
132 try:
133 while True:
134 tp, value = advance()
135 if tp in ignore:
136 continue
137 elif tp == token.STRING:
138 if have_docstring:
139 break
140 have_docstring = True
Benjamin Peterson42d26d92009-11-25 18:16:46 +0000141 elif tp == token.NAME and value == u"from":
142 tp, value = advance()
143 if tp != token.NAME and value != u"__future__":
Benjamin Peterson840077c2009-07-20 15:33:09 +0000144 break
Benjamin Peterson42d26d92009-11-25 18:16:46 +0000145 tp, value = advance()
146 if tp != token.NAME and value != u"import":
147 break
148 tp, value = advance()
149 if tp == token.OP and value == u"(":
150 tp, value = advance()
151 while tp == token.NAME:
152 if value == u"print_function":
153 return True
154 tp, value = advance()
155 if tp != token.OP and value != u",":
156 break
157 tp, value = advance()
Benjamin Peterson840077c2009-07-20 15:33:09 +0000158 else:
159 break
160 except StopIteration:
161 pass
162 return False
163
164
Benjamin Peterson08be2912008-09-27 21:09:10 +0000165class FixerError(Exception):
166 """A fixer could not be loaded."""
167
168
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000169class RefactoringTool(object):
170
Benjamin Peterson42d26d92009-11-25 18:16:46 +0000171 _default_options = {"print_function" : False}
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000172
Benjamin Peterson08be2912008-09-27 21:09:10 +0000173 CLASS_PREFIX = "Fix" # The prefix for fixer classes
174 FILE_PREFIX = "fix_" # The prefix for modules with a fixer within
175
176 def __init__(self, fixer_names, options=None, explicit=None):
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000177 """Initializer.
178
Benjamin Petersone6078232008-06-15 02:31:05 +0000179 Args:
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000180 fixer_names: a list of fixers to import
181 options: an dict with configuration.
182 explicit: a list of fixers to run even if they are explicit.
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000183 """
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000184 self.fixers = fixer_names
Benjamin Peterson08be2912008-09-27 21:09:10 +0000185 self.explicit = explicit or []
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000186 self.options = self._default_options.copy()
187 if options is not None:
188 self.options.update(options)
Benjamin Peterson42d26d92009-11-25 18:16:46 +0000189 if self.options["print_function"]:
190 self.grammar = pygram.python_grammar_no_print_statement
191 else:
192 self.grammar = pygram.python_grammar
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000193 self.errors = []
194 self.logger = logging.getLogger("RefactoringTool")
195 self.fixer_log = []
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000196 self.wrote = False
Benjamin Peterson42d26d92009-11-25 18:16:46 +0000197 self.driver = driver.Driver(self.grammar,
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000198 convert=pytree.convert,
199 logger=self.logger)
200 self.pre_order, self.post_order = self.get_fixers()
Martin v. Löwis6780a9d2008-05-02 21:30:20 +0000201
Benjamin Peterson840077c2009-07-20 15:33:09 +0000202 self.pre_order_heads = _get_headnode_dict(self.pre_order)
203 self.post_order_heads = _get_headnode_dict(self.post_order)
Martin v. Löwis6780a9d2008-05-02 21:30:20 +0000204
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000205 self.files = [] # List of files that were or should be modified
206
207 def get_fixers(self):
208 """Inspects the options to load the requested patterns and handlers.
Martin v. Löwisab41b372008-03-19 05:22:42 +0000209
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000210 Returns:
211 (pre_order, post_order), where pre_order is the list of fixers that
212 want a pre-order AST traversal, and post_order is the list that want
213 post-order traversal.
214 """
215 pre_order_fixers = []
216 post_order_fixers = []
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000217 for fix_mod_path in self.fixers:
Benjamin Peterson08be2912008-09-27 21:09:10 +0000218 mod = __import__(fix_mod_path, {}, {}, ["*"])
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000219 fix_name = fix_mod_path.rsplit(".", 1)[-1]
Benjamin Peterson08be2912008-09-27 21:09:10 +0000220 if fix_name.startswith(self.FILE_PREFIX):
221 fix_name = fix_name[len(self.FILE_PREFIX):]
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000222 parts = fix_name.split("_")
Benjamin Peterson08be2912008-09-27 21:09:10 +0000223 class_name = self.CLASS_PREFIX + "".join([p.title() for p in parts])
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000224 try:
225 fix_class = getattr(mod, class_name)
226 except AttributeError:
Benjamin Peterson08be2912008-09-27 21:09:10 +0000227 raise FixerError("Can't find %s.%s" % (fix_name, class_name))
228 fixer = fix_class(self.options, self.fixer_log)
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000229 if fixer.explicit and self.explicit is not True and \
230 fix_mod_path not in self.explicit:
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000231 self.log_message("Skipping implicit fixer: %s", fix_name)
232 continue
233
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000234 self.log_debug("Adding transformation: %s", fix_name)
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000235 if fixer.order == "pre":
236 pre_order_fixers.append(fixer)
237 elif fixer.order == "post":
238 post_order_fixers.append(fixer)
239 else:
Benjamin Peterson08be2912008-09-27 21:09:10 +0000240 raise FixerError("Illegal fixer order: %r" % fixer.order)
Martin v. Löwisbaf267c2008-03-22 00:01:12 +0000241
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000242 key_func = operator.attrgetter("run_order")
243 pre_order_fixers.sort(key=key_func)
244 post_order_fixers.sort(key=key_func)
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000245 return (pre_order_fixers, post_order_fixers)
246
247 def log_error(self, msg, *args, **kwds):
Benjamin Peterson08be2912008-09-27 21:09:10 +0000248 """Called when an error occurs."""
249 raise
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000250
251 def log_message(self, msg, *args):
252 """Hook to log a message."""
253 if args:
254 msg = msg % args
255 self.logger.info(msg)
256
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000257 def log_debug(self, msg, *args):
258 if args:
259 msg = msg % args
260 self.logger.debug(msg)
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000261
Benjamin Peterson840077c2009-07-20 15:33:09 +0000262 def print_output(self, old_text, new_text, filename, equal):
263 """Called with the old version, new version, and filename of a
264 refactored file."""
Benjamin Peterson08be2912008-09-27 21:09:10 +0000265 pass
266
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000267 def refactor(self, items, write=False, doctests_only=False):
268 """Refactor a list of files and directories."""
269 for dir_or_file in items:
270 if os.path.isdir(dir_or_file):
Benjamin Peterson08be2912008-09-27 21:09:10 +0000271 self.refactor_dir(dir_or_file, write, doctests_only)
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000272 else:
Benjamin Peterson08be2912008-09-27 21:09:10 +0000273 self.refactor_file(dir_or_file, write, doctests_only)
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000274
275 def refactor_dir(self, dir_name, write=False, doctests_only=False):
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000276 """Descends down a directory and refactor every Python file found.
277
278 Python files are assumed to have a .py extension.
279
280 Files and subdirectories starting with '.' are skipped.
281 """
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000282 for dirpath, dirnames, filenames in os.walk(dir_name):
283 self.log_debug("Descending into %s", dirpath)
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000284 dirnames.sort()
285 filenames.sort()
286 for name in filenames:
Benjamin Peterson840077c2009-07-20 15:33:09 +0000287 if not name.startswith(".") and \
288 os.path.splitext(name)[1].endswith("py"):
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000289 fullname = os.path.join(dirpath, name)
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000290 self.refactor_file(fullname, write, doctests_only)
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000291 # Modify dirnames in-place to remove subdirs with leading dots
292 dirnames[:] = [dn for dn in dirnames if not dn.startswith(".")]
293
Benjamin Peterson84ad84e2009-05-09 01:01:14 +0000294 def _read_python_source(self, filename):
295 """
296 Do our best to decode a Python source file correctly.
297 """
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000298 try:
Benjamin Peterson84ad84e2009-05-09 01:01:14 +0000299 f = open(filename, "rb")
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000300 except IOError, err:
301 self.log_error("Can't open %s: %s", filename, err)
Benjamin Peterson84ad84e2009-05-09 01:01:14 +0000302 return None, None
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000303 try:
Benjamin Peterson84ad84e2009-05-09 01:01:14 +0000304 encoding = tokenize.detect_encoding(f.readline)[0]
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000305 finally:
306 f.close()
Benjamin Peterson84ad84e2009-05-09 01:01:14 +0000307 with _open_with_encoding(filename, "r", encoding=encoding) as f:
308 return _from_system_newlines(f.read()), encoding
309
310 def refactor_file(self, filename, write=False, doctests_only=False):
311 """Refactors a file."""
312 input, encoding = self._read_python_source(filename)
313 if input is None:
314 # Reading the file failed.
315 return
316 input += u"\n" # Silence certain parse errors
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000317 if doctests_only:
318 self.log_debug("Refactoring doctests in %s", filename)
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000319 output = self.refactor_docstring(input, filename)
320 if output != input:
Benjamin Peterson84ad84e2009-05-09 01:01:14 +0000321 self.processed_file(output, filename, input, write, encoding)
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000322 else:
323 self.log_debug("No doctest changes in %s", filename)
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000324 else:
325 tree = self.refactor_string(input, filename)
326 if tree and tree.was_changed:
327 # The [:-1] is to take off the \n we added earlier
Benjamin Peterson84ad84e2009-05-09 01:01:14 +0000328 self.processed_file(unicode(tree)[:-1], filename,
329 write=write, encoding=encoding)
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000330 else:
331 self.log_debug("No changes in %s", filename)
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000332
333 def refactor_string(self, data, name):
334 """Refactor a given input string.
Martin v. Löwisab41b372008-03-19 05:22:42 +0000335
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000336 Args:
337 data: a string holding the code to be refactored.
338 name: a human-readable name for use in error/log messages.
Martin v. Löwisab41b372008-03-19 05:22:42 +0000339
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000340 Returns:
341 An AST corresponding to the refactored input stream; None if
342 there were errors during the parse.
343 """
Benjamin Peterson840077c2009-07-20 15:33:09 +0000344 if _detect_future_print(data):
345 self.driver.grammar = pygram.python_grammar_no_print_statement
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000346 try:
Benjamin Peterson6ae94ee2008-10-15 23:10:28 +0000347 tree = self.driver.parse_string(data)
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000348 except Exception, err:
349 self.log_error("Can't parse %s: %s: %s",
350 name, err.__class__.__name__, err)
351 return
Benjamin Peterson840077c2009-07-20 15:33:09 +0000352 finally:
Benjamin Peterson42d26d92009-11-25 18:16:46 +0000353 self.driver.grammar = self.grammar
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000354 self.log_debug("Refactoring %s", name)
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000355 self.refactor_tree(tree, name)
356 return tree
357
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000358 def refactor_stdin(self, doctests_only=False):
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000359 input = sys.stdin.read()
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000360 if doctests_only:
361 self.log_debug("Refactoring doctests in stdin")
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000362 output = self.refactor_docstring(input, "<stdin>")
363 if output != input:
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000364 self.processed_file(output, "<stdin>", input)
365 else:
366 self.log_debug("No doctest changes in stdin")
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000367 else:
368 tree = self.refactor_string(input, "<stdin>")
369 if tree and tree.was_changed:
Benjamin Peterson840077c2009-07-20 15:33:09 +0000370 self.processed_file(unicode(tree), "<stdin>", input)
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000371 else:
372 self.log_debug("No changes in stdin")
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000373
374 def refactor_tree(self, tree, name):
375 """Refactors a parse tree (modifying the tree in place).
Martin v. Löwisab41b372008-03-19 05:22:42 +0000376
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000377 Args:
378 tree: a pytree.Node instance representing the root of the tree
379 to be refactored.
380 name: a human-readable name for this tree.
Martin v. Löwisab41b372008-03-19 05:22:42 +0000381
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000382 Returns:
383 True if the tree was modified, False otherwise.
384 """
Benjamin Peterson37fc8232009-01-03 16:34:02 +0000385 for fixer in chain(self.pre_order, self.post_order):
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000386 fixer.start_tree(tree, name)
387
Benjamin Peterson43caaa02008-12-16 03:35:28 +0000388 self.traverse_by(self.pre_order_heads, tree.pre_order())
389 self.traverse_by(self.post_order_heads, tree.post_order())
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000390
Benjamin Peterson37fc8232009-01-03 16:34:02 +0000391 for fixer in chain(self.pre_order, self.post_order):
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000392 fixer.finish_tree(tree, name)
393 return tree.was_changed
394
395 def traverse_by(self, fixers, traversal):
396 """Traverse an AST, applying a set of fixers to each node.
Martin v. Löwisab41b372008-03-19 05:22:42 +0000397
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000398 This is a helper method for refactor_tree().
Martin v. Löwisab41b372008-03-19 05:22:42 +0000399
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000400 Args:
401 fixers: a list of fixer instances.
402 traversal: a generator that yields AST nodes.
Martin v. Löwisab41b372008-03-19 05:22:42 +0000403
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000404 Returns:
405 None
406 """
407 if not fixers:
408 return
409 for node in traversal:
Benjamin Peterson840077c2009-07-20 15:33:09 +0000410 for fixer in fixers[node.type]:
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000411 results = fixer.match(node)
412 if results:
413 new = fixer.transform(node, results)
Benjamin Peterson840077c2009-07-20 15:33:09 +0000414 if new is not None:
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000415 node.replace(new)
416 node = new
417
Benjamin Peterson84ad84e2009-05-09 01:01:14 +0000418 def processed_file(self, new_text, filename, old_text=None, write=False,
419 encoding=None):
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000420 """
421 Called when a file has been refactored, and there are changes.
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000422 """
423 self.files.append(filename)
424 if old_text is None:
Benjamin Peterson84ad84e2009-05-09 01:01:14 +0000425 old_text = self._read_python_source(filename)[0]
426 if old_text is None:
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000427 return
Benjamin Peterson840077c2009-07-20 15:33:09 +0000428 equal = old_text == new_text
429 self.print_output(old_text, new_text, filename, equal)
430 if equal:
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000431 self.log_debug("No changes to %s", filename)
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000432 return
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000433 if write:
Benjamin Peterson84ad84e2009-05-09 01:01:14 +0000434 self.write_file(new_text, filename, old_text, encoding)
Benjamin Peterson08be2912008-09-27 21:09:10 +0000435 else:
436 self.log_debug("Not writing changes to %s", filename)
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000437
Benjamin Peterson84ad84e2009-05-09 01:01:14 +0000438 def write_file(self, new_text, filename, old_text, encoding=None):
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000439 """Writes a string to a file.
440
441 It first shows a unified diff between the old text and the new text, and
442 then rewrites the file; the latter is only done if the write option is
443 set.
444 """
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000445 try:
Benjamin Peterson84ad84e2009-05-09 01:01:14 +0000446 f = _open_with_encoding(filename, "w", encoding=encoding)
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000447 except os.error, err:
448 self.log_error("Can't create %s: %s", filename, err)
449 return
450 try:
Benjamin Peterson84ad84e2009-05-09 01:01:14 +0000451 f.write(_to_system_newlines(new_text))
Benjamin Petersonba4d4802008-11-10 22:11:12 +0000452 except os.error, err:
453 self.log_error("Can't write %s: %s", filename, err)
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000454 finally:
455 f.close()
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000456 self.log_debug("Wrote changes to %s", filename)
457 self.wrote = True
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000458
459 PS1 = ">>> "
460 PS2 = "... "
461
462 def refactor_docstring(self, input, filename):
463 """Refactors a docstring, looking for doctests.
464
465 This returns a modified version of the input string. It looks
466 for doctests, which start with a ">>>" prompt, and may be
467 continued with "..." prompts, as long as the "..." is indented
468 the same as the ">>>".
469
470 (Unfortunately we can't use the doctest module's parser,
471 since, like most parsers, it is not geared towards preserving
472 the original source.)
473 """
474 result = []
475 block = None
476 block_lineno = None
477 indent = None
478 lineno = 0
479 for line in input.splitlines(True):
480 lineno += 1
481 if line.lstrip().startswith(self.PS1):
482 if block is not None:
483 result.extend(self.refactor_doctest(block, block_lineno,
484 indent, filename))
485 block_lineno = lineno
486 block = [line]
487 i = line.find(self.PS1)
488 indent = line[:i]
489 elif (indent is not None and
490 (line.startswith(indent + self.PS2) or
Benjamin Peterson84ad84e2009-05-09 01:01:14 +0000491 line == indent + self.PS2.rstrip() + u"\n")):
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000492 block.append(line)
493 else:
494 if block is not None:
495 result.extend(self.refactor_doctest(block, block_lineno,
496 indent, filename))
497 block = None
498 indent = None
499 result.append(line)
500 if block is not None:
501 result.extend(self.refactor_doctest(block, block_lineno,
502 indent, filename))
Benjamin Peterson84ad84e2009-05-09 01:01:14 +0000503 return u"".join(result)
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000504
505 def refactor_doctest(self, block, lineno, indent, filename):
506 """Refactors one doctest.
507
508 A doctest is given as a block of lines, the first of which starts
509 with ">>>" (possibly indented), while the remaining lines start
510 with "..." (identically indented).
511
512 """
513 try:
514 tree = self.parse_block(block, lineno, indent)
515 except Exception, err:
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000516 if self.log.isEnabledFor(logging.DEBUG):
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000517 for line in block:
Benjamin Peterson84ad84e2009-05-09 01:01:14 +0000518 self.log_debug("Source: %s", line.rstrip(u"\n"))
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000519 self.log_error("Can't parse docstring in %s line %s: %s: %s",
520 filename, lineno, err.__class__.__name__, err)
521 return block
522 if self.refactor_tree(tree, filename):
Benjamin Peterson840077c2009-07-20 15:33:09 +0000523 new = unicode(tree).splitlines(True)
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000524 # Undo the adjustment of the line numbers in wrap_toks() below.
525 clipped, new = new[:lineno-1], new[lineno-1:]
Benjamin Peterson84ad84e2009-05-09 01:01:14 +0000526 assert clipped == [u"\n"] * (lineno-1), clipped
527 if not new[-1].endswith(u"\n"):
528 new[-1] += u"\n"
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000529 block = [indent + self.PS1 + new.pop(0)]
530 if new:
531 block += [indent + self.PS2 + line for line in new]
532 return block
533
534 def summarize(self):
Benjamin Petersoneb55fd82008-09-03 00:21:32 +0000535 if self.wrote:
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000536 were = "were"
537 else:
538 were = "need to be"
539 if not self.files:
540 self.log_message("No files %s modified.", were)
541 else:
542 self.log_message("Files that %s modified:", were)
543 for file in self.files:
544 self.log_message(file)
545 if self.fixer_log:
546 self.log_message("Warnings/messages while refactoring:")
547 for message in self.fixer_log:
548 self.log_message(message)
549 if self.errors:
550 if len(self.errors) == 1:
551 self.log_message("There was 1 error:")
552 else:
553 self.log_message("There were %d errors:", len(self.errors))
554 for msg, args, kwds in self.errors:
555 self.log_message(msg, *args, **kwds)
556
557 def parse_block(self, block, lineno, indent):
558 """Parses a block into a tree.
559
560 This is necessary to get correct line number / offset information
561 in the parser diagnostics and embedded into the parse tree.
562 """
563 return self.driver.parse_tokens(self.wrap_toks(block, lineno, indent))
564
565 def wrap_toks(self, block, lineno, indent):
566 """Wraps a tokenize stream to systematically modify start/end."""
567 tokens = tokenize.generate_tokens(self.gen_lines(block, indent).next)
568 for type, value, (line0, col0), (line1, col1), line_text in tokens:
569 line0 += lineno - 1
570 line1 += lineno - 1
571 # Don't bother updating the columns; this is too complicated
572 # since line_text would also have to be updated and it would
573 # still break for tokens spanning lines. Let the user guess
574 # that the column numbers for doctests are relative to the
575 # end of the prompt string (PS1 or PS2).
576 yield type, value, (line0, col0), (line1, col1), line_text
577
578
579 def gen_lines(self, block, indent):
580 """Generates lines as expected by tokenize from a list of lines.
581
582 This strips the first len(indent + self.PS1) characters off each line.
583 """
584 prefix1 = indent + self.PS1
585 prefix2 = indent + self.PS2
586 prefix = prefix1
587 for line in block:
588 if line.startswith(prefix):
589 yield line[len(prefix):]
Benjamin Peterson84ad84e2009-05-09 01:01:14 +0000590 elif line == prefix.rstrip() + u"\n":
591 yield u"\n"
Martin v. Löwis5e37bae2008-03-19 04:43:46 +0000592 else:
593 raise AssertionError("line=%r, prefix=%r" % (line, prefix))
594 prefix = prefix2
595 while True:
596 yield ""
597
598
Benjamin Petersoneaeb4c62009-05-05 23:13:58 +0000599class MultiprocessingUnsupported(Exception):
600 pass
601
602
603class MultiprocessRefactoringTool(RefactoringTool):
604
605 def __init__(self, *args, **kwargs):
606 super(MultiprocessRefactoringTool, self).__init__(*args, **kwargs)
607 self.queue = None
608
609 def refactor(self, items, write=False, doctests_only=False,
610 num_processes=1):
611 if num_processes == 1:
612 return super(MultiprocessRefactoringTool, self).refactor(
613 items, write, doctests_only)
614 try:
615 import multiprocessing
616 except ImportError:
617 raise MultiprocessingUnsupported
618 if self.queue is not None:
619 raise RuntimeError("already doing multiple processes")
620 self.queue = multiprocessing.JoinableQueue()
621 processes = [multiprocessing.Process(target=self._child)
622 for i in xrange(num_processes)]
623 try:
624 for p in processes:
625 p.start()
626 super(MultiprocessRefactoringTool, self).refactor(items, write,
627 doctests_only)
628 finally:
629 self.queue.join()
630 for i in xrange(num_processes):
631 self.queue.put(None)
632 for p in processes:
633 if p.is_alive():
634 p.join()
635 self.queue = None
636
637 def _child(self):
638 task = self.queue.get()
639 while task is not None:
640 args, kwargs = task
641 try:
642 super(MultiprocessRefactoringTool, self).refactor_file(
643 *args, **kwargs)
644 finally:
645 self.queue.task_done()
646 task = self.queue.get()
647
648 def refactor_file(self, *args, **kwargs):
649 if self.queue is not None:
650 self.queue.put((args, kwargs))
651 else:
652 return super(MultiprocessRefactoringTool, self).refactor_file(
653 *args, **kwargs)