blob: 4d83d201b872b046f3eb1ccf187548084cd85f75 [file] [log] [blame]
Martin v. Löwisef04c442008-03-19 05:04:44 +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
Benjamin Peterson8d26b0b2010-05-07 19:10:11 +000011from __future__ import with_statement
12
Martin v. Löwisef04c442008-03-19 05:04:44 +000013__author__ = "Guido van Rossum <guido@python.org>"
14
15
16# Python imports
17import os
18import sys
Martin v. Löwisef04c442008-03-19 05:04:44 +000019import logging
Benjamin Peterson8951b612008-09-03 02:27:16 +000020import operator
Benjamin Peterson3059b002009-07-20 16:42:03 +000021import collections
22import io
Christian Heimes81ee3ef2008-05-04 22:42:01 +000023from itertools import chain
Martin v. Löwisef04c442008-03-19 05:04:44 +000024
25# Local imports
Benjamin Peterson3059b002009-07-20 16:42:03 +000026from .pgen2 import driver, tokenize, token
Benjamin Peterson2c3ac6b2009-06-11 23:47:38 +000027from . import pytree, pygram
Martin v. Löwisef04c442008-03-19 05:04:44 +000028
Martin v. Löwisef04c442008-03-19 05:04:44 +000029
Benjamin Peterson8951b612008-09-03 02:27:16 +000030def get_all_fix_names(fixer_pkg, remove_prefix=True):
31 """Return a sorted list of all available fix names in the given package."""
32 pkg = __import__(fixer_pkg, [], [], ["*"])
33 fixer_dir = os.path.dirname(pkg.__file__)
Martin v. Löwisef04c442008-03-19 05:04:44 +000034 fix_names = []
Benjamin Peterson206e3072008-10-19 14:07:49 +000035 for name in sorted(os.listdir(fixer_dir)):
Martin v. Löwisef04c442008-03-19 05:04:44 +000036 if name.startswith("fix_") and name.endswith(".py"):
Benjamin Peterson8951b612008-09-03 02:27:16 +000037 if remove_prefix:
38 name = name[4:]
39 fix_names.append(name[:-3])
Martin v. Löwisef04c442008-03-19 05:04:44 +000040 return fix_names
41
Benjamin Peterson3059b002009-07-20 16:42:03 +000042
43class _EveryNode(Exception):
44 pass
45
46
47def _get_head_types(pat):
Christian Heimes81ee3ef2008-05-04 22:42:01 +000048 """ Accepts a pytree Pattern Node and returns a set
49 of the pattern types which will match first. """
50
51 if isinstance(pat, (pytree.NodePattern, pytree.LeafPattern)):
52 # NodePatters must either have no type and no content
53 # or a type and content -- so they don't get any farther
54 # Always return leafs
Benjamin Peterson3059b002009-07-20 16:42:03 +000055 if pat.type is None:
56 raise _EveryNode
Christian Heimes81ee3ef2008-05-04 22:42:01 +000057 return set([pat.type])
58
59 if isinstance(pat, pytree.NegatedPattern):
60 if pat.content:
Benjamin Peterson3059b002009-07-20 16:42:03 +000061 return _get_head_types(pat.content)
62 raise _EveryNode # Negated Patterns don't have a type
Christian Heimes81ee3ef2008-05-04 22:42:01 +000063
64 if isinstance(pat, pytree.WildcardPattern):
65 # Recurse on each node in content
66 r = set()
67 for p in pat.content:
68 for x in p:
Benjamin Peterson3059b002009-07-20 16:42:03 +000069 r.update(_get_head_types(x))
Christian Heimes81ee3ef2008-05-04 22:42:01 +000070 return r
71
72 raise Exception("Oh no! I don't understand pattern %s" %(pat))
73
Benjamin Peterson3059b002009-07-20 16:42:03 +000074
75def _get_headnode_dict(fixer_list):
Christian Heimes81ee3ef2008-05-04 22:42:01 +000076 """ Accepts a list of fixers and returns a dictionary
77 of head node type --> fixer list. """
Benjamin Peterson3059b002009-07-20 16:42:03 +000078 head_nodes = collections.defaultdict(list)
79 every = []
Christian Heimes81ee3ef2008-05-04 22:42:01 +000080 for fixer in fixer_list:
Benjamin Peterson3059b002009-07-20 16:42:03 +000081 if fixer.pattern:
82 try:
83 heads = _get_head_types(fixer.pattern)
84 except _EveryNode:
85 every.append(fixer)
86 else:
87 for node_type in heads:
88 head_nodes[node_type].append(fixer)
89 else:
90 if fixer._accept_type is not None:
91 head_nodes[fixer._accept_type].append(fixer)
92 else:
93 every.append(fixer)
94 for node_type in chain(pygram.python_grammar.symbol2number.values(),
95 pygram.python_grammar.tokens):
96 head_nodes[node_type].extend(every)
97 return dict(head_nodes)
98
Christian Heimes81ee3ef2008-05-04 22:42:01 +000099
Benjamin Peterson8951b612008-09-03 02:27:16 +0000100def get_fixers_from_package(pkg_name):
101 """
102 Return the fully qualified names for fixers in the package pkg_name.
103 """
104 return [pkg_name + "." + fix_name
105 for fix_name in get_all_fix_names(pkg_name, False)]
106
Benjamin Petersond481e3d2009-05-09 19:42:23 +0000107def _identity(obj):
108 return obj
109
110if sys.version_info < (3, 0):
111 import codecs
112 _open_with_encoding = codecs.open
113 # codecs.open doesn't translate newlines sadly.
114 def _from_system_newlines(input):
115 return input.replace("\r\n", "\n")
116 def _to_system_newlines(input):
117 if os.linesep != "\n":
118 return input.replace("\n", os.linesep)
119 else:
120 return input
121else:
122 _open_with_encoding = open
123 _from_system_newlines = _identity
124 _to_system_newlines = _identity
125
Martin v. Löwisef04c442008-03-19 05:04:44 +0000126
Benjamin Peterson8d26b0b2010-05-07 19:10:11 +0000127def _detect_future_features(source):
Benjamin Peterson3059b002009-07-20 16:42:03 +0000128 have_docstring = False
129 gen = tokenize.generate_tokens(io.StringIO(source).readline)
130 def advance():
131 tok = next(gen)
132 return tok[0], tok[1]
133 ignore = frozenset((token.NEWLINE, tokenize.NL, token.COMMENT))
Benjamin Peterson8d26b0b2010-05-07 19:10:11 +0000134 features = set()
Benjamin Peterson3059b002009-07-20 16:42:03 +0000135 try:
136 while True:
137 tp, value = advance()
138 if tp in ignore:
139 continue
140 elif tp == token.STRING:
141 if have_docstring:
142 break
143 have_docstring = True
Benjamin Peterson20211002009-11-25 18:34:42 +0000144 elif tp == token.NAME and value == "from":
145 tp, value = advance()
Benjamin Peterson8d26b0b2010-05-07 19:10:11 +0000146 if tp != token.NAME or value != "__future__":
Benjamin Peterson3059b002009-07-20 16:42:03 +0000147 break
Benjamin Peterson20211002009-11-25 18:34:42 +0000148 tp, value = advance()
Benjamin Peterson8d26b0b2010-05-07 19:10:11 +0000149 if tp != token.NAME or value != "import":
Benjamin Peterson20211002009-11-25 18:34:42 +0000150 break
151 tp, value = advance()
152 if tp == token.OP and value == "(":
153 tp, value = advance()
154 while tp == token.NAME:
Benjamin Peterson8d26b0b2010-05-07 19:10:11 +0000155 features.add(value)
Benjamin Peterson20211002009-11-25 18:34:42 +0000156 tp, value = advance()
Benjamin Peterson8d26b0b2010-05-07 19:10:11 +0000157 if tp != token.OP or value != ",":
Benjamin Peterson20211002009-11-25 18:34:42 +0000158 break
159 tp, value = advance()
Benjamin Peterson3059b002009-07-20 16:42:03 +0000160 else:
161 break
162 except StopIteration:
163 pass
Benjamin Peterson8d26b0b2010-05-07 19:10:11 +0000164 return frozenset(features)
Benjamin Peterson3059b002009-07-20 16:42:03 +0000165
166
Benjamin Petersond61de7f2008-09-27 22:17:35 +0000167class FixerError(Exception):
168 """A fixer could not be loaded."""
169
170
Martin v. Löwisef04c442008-03-19 05:04:44 +0000171class RefactoringTool(object):
172
Benjamin Peterson20211002009-11-25 18:34:42 +0000173 _default_options = {"print_function" : False}
Benjamin Peterson8951b612008-09-03 02:27:16 +0000174
Benjamin Petersond61de7f2008-09-27 22:17:35 +0000175 CLASS_PREFIX = "Fix" # The prefix for fixer classes
176 FILE_PREFIX = "fix_" # The prefix for modules with a fixer within
177
178 def __init__(self, fixer_names, options=None, explicit=None):
Martin v. Löwisef04c442008-03-19 05:04:44 +0000179 """Initializer.
180
Benjamin Petersondf6dc8f2008-06-15 02:57:40 +0000181 Args:
Benjamin Peterson8951b612008-09-03 02:27:16 +0000182 fixer_names: a list of fixers to import
183 options: an dict with configuration.
184 explicit: a list of fixers to run even if they are explicit.
Martin v. Löwisef04c442008-03-19 05:04:44 +0000185 """
Benjamin Peterson8951b612008-09-03 02:27:16 +0000186 self.fixers = fixer_names
Benjamin Petersond61de7f2008-09-27 22:17:35 +0000187 self.explicit = explicit or []
Benjamin Peterson8951b612008-09-03 02:27:16 +0000188 self.options = self._default_options.copy()
189 if options is not None:
190 self.options.update(options)
Benjamin Peterson20211002009-11-25 18:34:42 +0000191 if self.options["print_function"]:
192 self.grammar = pygram.python_grammar_no_print_statement
193 else:
194 self.grammar = pygram.python_grammar
Martin v. Löwisef04c442008-03-19 05:04:44 +0000195 self.errors = []
196 self.logger = logging.getLogger("RefactoringTool")
197 self.fixer_log = []
Benjamin Peterson8951b612008-09-03 02:27:16 +0000198 self.wrote = False
Benjamin Peterson20211002009-11-25 18:34:42 +0000199 self.driver = driver.Driver(self.grammar,
Martin v. Löwisef04c442008-03-19 05:04:44 +0000200 convert=pytree.convert,
201 logger=self.logger)
202 self.pre_order, self.post_order = self.get_fixers()
Christian Heimes81ee3ef2008-05-04 22:42:01 +0000203
Benjamin Peterson3059b002009-07-20 16:42:03 +0000204 self.pre_order_heads = _get_headnode_dict(self.pre_order)
205 self.post_order_heads = _get_headnode_dict(self.post_order)
Christian Heimes81ee3ef2008-05-04 22:42:01 +0000206
Martin v. Löwisef04c442008-03-19 05:04:44 +0000207 self.files = [] # List of files that were or should be modified
208
209 def get_fixers(self):
210 """Inspects the options to load the requested patterns and handlers.
Martin v. Löwisf733c602008-03-19 05:26:18 +0000211
Martin v. Löwisef04c442008-03-19 05:04:44 +0000212 Returns:
213 (pre_order, post_order), where pre_order is the list of fixers that
214 want a pre-order AST traversal, and post_order is the list that want
215 post-order traversal.
216 """
217 pre_order_fixers = []
218 post_order_fixers = []
Benjamin Peterson8951b612008-09-03 02:27:16 +0000219 for fix_mod_path in self.fixers:
Benjamin Petersond61de7f2008-09-27 22:17:35 +0000220 mod = __import__(fix_mod_path, {}, {}, ["*"])
Benjamin Peterson8951b612008-09-03 02:27:16 +0000221 fix_name = fix_mod_path.rsplit(".", 1)[-1]
Benjamin Petersond61de7f2008-09-27 22:17:35 +0000222 if fix_name.startswith(self.FILE_PREFIX):
223 fix_name = fix_name[len(self.FILE_PREFIX):]
Martin v. Löwisef04c442008-03-19 05:04:44 +0000224 parts = fix_name.split("_")
Benjamin Petersond61de7f2008-09-27 22:17:35 +0000225 class_name = self.CLASS_PREFIX + "".join([p.title() for p in parts])
Martin v. Löwisef04c442008-03-19 05:04:44 +0000226 try:
227 fix_class = getattr(mod, class_name)
228 except AttributeError:
Benjamin Petersond61de7f2008-09-27 22:17:35 +0000229 raise FixerError("Can't find %s.%s" % (fix_name, class_name))
230 fixer = fix_class(self.options, self.fixer_log)
Benjamin Peterson8951b612008-09-03 02:27:16 +0000231 if fixer.explicit and self.explicit is not True and \
232 fix_mod_path not in self.explicit:
Martin v. Löwisef04c442008-03-19 05:04:44 +0000233 self.log_message("Skipping implicit fixer: %s", fix_name)
234 continue
235
Benjamin Peterson8951b612008-09-03 02:27:16 +0000236 self.log_debug("Adding transformation: %s", fix_name)
Martin v. Löwisef04c442008-03-19 05:04:44 +0000237 if fixer.order == "pre":
238 pre_order_fixers.append(fixer)
239 elif fixer.order == "post":
240 post_order_fixers.append(fixer)
241 else:
Benjamin Petersond61de7f2008-09-27 22:17:35 +0000242 raise FixerError("Illegal fixer order: %r" % fixer.order)
Martin v. Löwis3faa84f2008-03-22 00:07:09 +0000243
Benjamin Peterson8951b612008-09-03 02:27:16 +0000244 key_func = operator.attrgetter("run_order")
245 pre_order_fixers.sort(key=key_func)
246 post_order_fixers.sort(key=key_func)
Martin v. Löwisef04c442008-03-19 05:04:44 +0000247 return (pre_order_fixers, post_order_fixers)
248
249 def log_error(self, msg, *args, **kwds):
Benjamin Petersond61de7f2008-09-27 22:17:35 +0000250 """Called when an error occurs."""
251 raise
Martin v. Löwisef04c442008-03-19 05:04:44 +0000252
253 def log_message(self, msg, *args):
254 """Hook to log a message."""
255 if args:
256 msg = msg % args
257 self.logger.info(msg)
258
Benjamin Peterson8951b612008-09-03 02:27:16 +0000259 def log_debug(self, msg, *args):
260 if args:
261 msg = msg % args
262 self.logger.debug(msg)
Martin v. Löwisef04c442008-03-19 05:04:44 +0000263
Benjamin Peterson3059b002009-07-20 16:42:03 +0000264 def print_output(self, old_text, new_text, filename, equal):
265 """Called with the old version, new version, and filename of a
266 refactored file."""
Benjamin Petersond61de7f2008-09-27 22:17:35 +0000267 pass
268
Benjamin Peterson8951b612008-09-03 02:27:16 +0000269 def refactor(self, items, write=False, doctests_only=False):
270 """Refactor a list of files and directories."""
271 for dir_or_file in items:
272 if os.path.isdir(dir_or_file):
Benjamin Petersond61de7f2008-09-27 22:17:35 +0000273 self.refactor_dir(dir_or_file, write, doctests_only)
Benjamin Peterson8951b612008-09-03 02:27:16 +0000274 else:
Benjamin Petersond61de7f2008-09-27 22:17:35 +0000275 self.refactor_file(dir_or_file, write, doctests_only)
Benjamin Peterson8951b612008-09-03 02:27:16 +0000276
277 def refactor_dir(self, dir_name, write=False, doctests_only=False):
Martin v. Löwisef04c442008-03-19 05:04:44 +0000278 """Descends down a directory and refactor every Python file found.
279
280 Python files are assumed to have a .py extension.
281
282 Files and subdirectories starting with '.' are skipped.
283 """
Benjamin Peterson8951b612008-09-03 02:27:16 +0000284 for dirpath, dirnames, filenames in os.walk(dir_name):
285 self.log_debug("Descending into %s", dirpath)
Martin v. Löwisef04c442008-03-19 05:04:44 +0000286 dirnames.sort()
287 filenames.sort()
288 for name in filenames:
Benjamin Peterson3059b002009-07-20 16:42:03 +0000289 if not name.startswith(".") and \
290 os.path.splitext(name)[1].endswith("py"):
Martin v. Löwisef04c442008-03-19 05:04:44 +0000291 fullname = os.path.join(dirpath, name)
Benjamin Peterson8951b612008-09-03 02:27:16 +0000292 self.refactor_file(fullname, write, doctests_only)
Martin v. Löwisef04c442008-03-19 05:04:44 +0000293 # Modify dirnames in-place to remove subdirs with leading dots
294 dirnames[:] = [dn for dn in dirnames if not dn.startswith(".")]
295
Benjamin Petersond481e3d2009-05-09 19:42:23 +0000296 def _read_python_source(self, filename):
297 """
298 Do our best to decode a Python source file correctly.
299 """
Martin v. Löwisef04c442008-03-19 05:04:44 +0000300 try:
Benjamin Petersond481e3d2009-05-09 19:42:23 +0000301 f = open(filename, "rb")
Martin v. Löwis8a5f8ca2008-03-19 05:33:36 +0000302 except IOError as err:
Martin v. Löwisef04c442008-03-19 05:04:44 +0000303 self.log_error("Can't open %s: %s", filename, err)
Benjamin Petersond481e3d2009-05-09 19:42:23 +0000304 return None, None
Martin v. Löwisef04c442008-03-19 05:04:44 +0000305 try:
Benjamin Petersond481e3d2009-05-09 19:42:23 +0000306 encoding = tokenize.detect_encoding(f.readline)[0]
Martin v. Löwisef04c442008-03-19 05:04:44 +0000307 finally:
308 f.close()
Benjamin Petersond481e3d2009-05-09 19:42:23 +0000309 with _open_with_encoding(filename, "r", encoding=encoding) as f:
310 return _from_system_newlines(f.read()), encoding
311
312 def refactor_file(self, filename, write=False, doctests_only=False):
313 """Refactors a file."""
314 input, encoding = self._read_python_source(filename)
315 if input is None:
316 # Reading the file failed.
317 return
318 input += "\n" # Silence certain parse errors
Benjamin Peterson8951b612008-09-03 02:27:16 +0000319 if doctests_only:
320 self.log_debug("Refactoring doctests in %s", filename)
Martin v. Löwisef04c442008-03-19 05:04:44 +0000321 output = self.refactor_docstring(input, filename)
322 if output != input:
Benjamin Petersond481e3d2009-05-09 19:42:23 +0000323 self.processed_file(output, filename, input, write, encoding)
Benjamin Peterson8951b612008-09-03 02:27:16 +0000324 else:
325 self.log_debug("No doctest changes in %s", filename)
Martin v. Löwisef04c442008-03-19 05:04:44 +0000326 else:
327 tree = self.refactor_string(input, filename)
328 if tree and tree.was_changed:
329 # The [:-1] is to take off the \n we added earlier
Benjamin Petersond481e3d2009-05-09 19:42:23 +0000330 self.processed_file(str(tree)[:-1], filename,
331 write=write, encoding=encoding)
Benjamin Peterson8951b612008-09-03 02:27:16 +0000332 else:
333 self.log_debug("No changes in %s", filename)
Martin v. Löwisef04c442008-03-19 05:04:44 +0000334
335 def refactor_string(self, data, name):
336 """Refactor a given input string.
Martin v. Löwisf733c602008-03-19 05:26:18 +0000337
Martin v. Löwisef04c442008-03-19 05:04:44 +0000338 Args:
339 data: a string holding the code to be refactored.
340 name: a human-readable name for use in error/log messages.
Martin v. Löwisf733c602008-03-19 05:26:18 +0000341
Martin v. Löwisef04c442008-03-19 05:04:44 +0000342 Returns:
343 An AST corresponding to the refactored input stream; None if
344 there were errors during the parse.
345 """
Benjamin Peterson8d26b0b2010-05-07 19:10:11 +0000346 features = _detect_future_features(data)
347 if "print_function" in features:
Benjamin Peterson3059b002009-07-20 16:42:03 +0000348 self.driver.grammar = pygram.python_grammar_no_print_statement
Martin v. Löwisef04c442008-03-19 05:04:44 +0000349 try:
Benjamin Peterson206e3072008-10-19 14:07:49 +0000350 tree = self.driver.parse_string(data)
Martin v. Löwis8a5f8ca2008-03-19 05:33:36 +0000351 except Exception as err:
Martin v. Löwisef04c442008-03-19 05:04:44 +0000352 self.log_error("Can't parse %s: %s: %s",
353 name, err.__class__.__name__, err)
354 return
Benjamin Peterson3059b002009-07-20 16:42:03 +0000355 finally:
Benjamin Peterson20211002009-11-25 18:34:42 +0000356 self.driver.grammar = self.grammar
Benjamin Peterson8d26b0b2010-05-07 19:10:11 +0000357 tree.future_features = features
Benjamin Peterson8951b612008-09-03 02:27:16 +0000358 self.log_debug("Refactoring %s", name)
Martin v. Löwisef04c442008-03-19 05:04:44 +0000359 self.refactor_tree(tree, name)
360 return tree
361
Benjamin Peterson8951b612008-09-03 02:27:16 +0000362 def refactor_stdin(self, doctests_only=False):
Martin v. Löwisef04c442008-03-19 05:04:44 +0000363 input = sys.stdin.read()
Benjamin Peterson8951b612008-09-03 02:27:16 +0000364 if doctests_only:
365 self.log_debug("Refactoring doctests in stdin")
Martin v. Löwisef04c442008-03-19 05:04:44 +0000366 output = self.refactor_docstring(input, "<stdin>")
367 if output != input:
Benjamin Peterson8951b612008-09-03 02:27:16 +0000368 self.processed_file(output, "<stdin>", input)
369 else:
370 self.log_debug("No doctest changes in stdin")
Martin v. Löwisef04c442008-03-19 05:04:44 +0000371 else:
372 tree = self.refactor_string(input, "<stdin>")
373 if tree and tree.was_changed:
Benjamin Peterson8951b612008-09-03 02:27:16 +0000374 self.processed_file(str(tree), "<stdin>", input)
375 else:
376 self.log_debug("No changes in stdin")
Martin v. Löwisef04c442008-03-19 05:04:44 +0000377
378 def refactor_tree(self, tree, name):
379 """Refactors a parse tree (modifying the tree in place).
Martin v. Löwisf733c602008-03-19 05:26:18 +0000380
Martin v. Löwisef04c442008-03-19 05:04:44 +0000381 Args:
382 tree: a pytree.Node instance representing the root of the tree
383 to be refactored.
384 name: a human-readable name for this tree.
Martin v. Löwisf733c602008-03-19 05:26:18 +0000385
Martin v. Löwisef04c442008-03-19 05:04:44 +0000386 Returns:
387 True if the tree was modified, False otherwise.
388 """
Benjamin Peterson8bcddca2009-01-03 16:53:14 +0000389 for fixer in chain(self.pre_order, self.post_order):
Martin v. Löwisef04c442008-03-19 05:04:44 +0000390 fixer.start_tree(tree, name)
391
Benjamin Peterson0b24b3d2008-12-16 03:57:54 +0000392 self.traverse_by(self.pre_order_heads, tree.pre_order())
393 self.traverse_by(self.post_order_heads, tree.post_order())
Martin v. Löwisef04c442008-03-19 05:04:44 +0000394
Benjamin Peterson8bcddca2009-01-03 16:53:14 +0000395 for fixer in chain(self.pre_order, self.post_order):
Martin v. Löwisef04c442008-03-19 05:04:44 +0000396 fixer.finish_tree(tree, name)
397 return tree.was_changed
398
399 def traverse_by(self, fixers, traversal):
400 """Traverse an AST, applying a set of fixers to each node.
Martin v. Löwisf733c602008-03-19 05:26:18 +0000401
Martin v. Löwisef04c442008-03-19 05:04:44 +0000402 This is a helper method for refactor_tree().
Martin v. Löwisf733c602008-03-19 05:26:18 +0000403
Martin v. Löwisef04c442008-03-19 05:04:44 +0000404 Args:
405 fixers: a list of fixer instances.
406 traversal: a generator that yields AST nodes.
Martin v. Löwisf733c602008-03-19 05:26:18 +0000407
Martin v. Löwisef04c442008-03-19 05:04:44 +0000408 Returns:
409 None
410 """
411 if not fixers:
412 return
413 for node in traversal:
Benjamin Peterson3059b002009-07-20 16:42:03 +0000414 for fixer in fixers[node.type]:
Martin v. Löwisef04c442008-03-19 05:04:44 +0000415 results = fixer.match(node)
416 if results:
417 new = fixer.transform(node, results)
Benjamin Peterson3059b002009-07-20 16:42:03 +0000418 if new is not None:
Martin v. Löwisef04c442008-03-19 05:04:44 +0000419 node.replace(new)
420 node = new
421
Benjamin Petersond481e3d2009-05-09 19:42:23 +0000422 def processed_file(self, new_text, filename, old_text=None, write=False,
423 encoding=None):
Benjamin Peterson8951b612008-09-03 02:27:16 +0000424 """
425 Called when a file has been refactored, and there are changes.
Martin v. Löwisef04c442008-03-19 05:04:44 +0000426 """
427 self.files.append(filename)
428 if old_text is None:
Benjamin Petersond481e3d2009-05-09 19:42:23 +0000429 old_text = self._read_python_source(filename)[0]
430 if old_text is None:
Martin v. Löwisef04c442008-03-19 05:04:44 +0000431 return
Benjamin Peterson3059b002009-07-20 16:42:03 +0000432 equal = old_text == new_text
433 self.print_output(old_text, new_text, filename, equal)
434 if equal:
Benjamin Peterson8951b612008-09-03 02:27:16 +0000435 self.log_debug("No changes to %s", filename)
Martin v. Löwisef04c442008-03-19 05:04:44 +0000436 return
Benjamin Peterson8951b612008-09-03 02:27:16 +0000437 if write:
Benjamin Petersond481e3d2009-05-09 19:42:23 +0000438 self.write_file(new_text, filename, old_text, encoding)
Benjamin Petersond61de7f2008-09-27 22:17:35 +0000439 else:
440 self.log_debug("Not writing changes to %s", filename)
Benjamin Peterson8951b612008-09-03 02:27:16 +0000441
Benjamin Petersond481e3d2009-05-09 19:42:23 +0000442 def write_file(self, new_text, filename, old_text, encoding=None):
Benjamin Peterson8951b612008-09-03 02:27:16 +0000443 """Writes a string to a file.
444
445 It first shows a unified diff between the old text and the new text, and
446 then rewrites the file; the latter is only done if the write option is
447 set.
448 """
Martin v. Löwisef04c442008-03-19 05:04:44 +0000449 try:
Benjamin Petersond481e3d2009-05-09 19:42:23 +0000450 f = _open_with_encoding(filename, "w", encoding=encoding)
Martin v. Löwis8a5f8ca2008-03-19 05:33:36 +0000451 except os.error as err:
Martin v. Löwisef04c442008-03-19 05:04:44 +0000452 self.log_error("Can't create %s: %s", filename, err)
453 return
454 try:
Benjamin Petersond481e3d2009-05-09 19:42:23 +0000455 f.write(_to_system_newlines(new_text))
Benjamin Petersonba558182008-11-10 22:21:33 +0000456 except os.error as err:
457 self.log_error("Can't write %s: %s", filename, err)
Martin v. Löwisef04c442008-03-19 05:04:44 +0000458 finally:
459 f.close()
Benjamin Peterson8951b612008-09-03 02:27:16 +0000460 self.log_debug("Wrote changes to %s", filename)
461 self.wrote = True
Martin v. Löwisef04c442008-03-19 05:04:44 +0000462
463 PS1 = ">>> "
464 PS2 = "... "
465
466 def refactor_docstring(self, input, filename):
467 """Refactors a docstring, looking for doctests.
468
469 This returns a modified version of the input string. It looks
470 for doctests, which start with a ">>>" prompt, and may be
471 continued with "..." prompts, as long as the "..." is indented
472 the same as the ">>>".
473
474 (Unfortunately we can't use the doctest module's parser,
475 since, like most parsers, it is not geared towards preserving
476 the original source.)
477 """
478 result = []
479 block = None
480 block_lineno = None
481 indent = None
482 lineno = 0
483 for line in input.splitlines(True):
484 lineno += 1
485 if line.lstrip().startswith(self.PS1):
486 if block is not None:
487 result.extend(self.refactor_doctest(block, block_lineno,
488 indent, filename))
489 block_lineno = lineno
490 block = [line]
491 i = line.find(self.PS1)
492 indent = line[:i]
493 elif (indent is not None and
494 (line.startswith(indent + self.PS2) or
495 line == indent + self.PS2.rstrip() + "\n")):
496 block.append(line)
497 else:
498 if block is not None:
499 result.extend(self.refactor_doctest(block, block_lineno,
500 indent, filename))
501 block = None
502 indent = None
503 result.append(line)
504 if block is not None:
505 result.extend(self.refactor_doctest(block, block_lineno,
506 indent, filename))
507 return "".join(result)
508
509 def refactor_doctest(self, block, lineno, indent, filename):
510 """Refactors one doctest.
511
512 A doctest is given as a block of lines, the first of which starts
513 with ">>>" (possibly indented), while the remaining lines start
514 with "..." (identically indented).
515
516 """
517 try:
518 tree = self.parse_block(block, lineno, indent)
Martin v. Löwis8a5f8ca2008-03-19 05:33:36 +0000519 except Exception as err:
Benjamin Peterson4eb5fa52010-08-08 19:01:25 +0000520 if self.logger.isEnabledFor(logging.DEBUG):
Martin v. Löwisef04c442008-03-19 05:04:44 +0000521 for line in block:
Benjamin Peterson8951b612008-09-03 02:27:16 +0000522 self.log_debug("Source: %s", line.rstrip("\n"))
Martin v. Löwisef04c442008-03-19 05:04:44 +0000523 self.log_error("Can't parse docstring in %s line %s: %s: %s",
524 filename, lineno, err.__class__.__name__, err)
525 return block
526 if self.refactor_tree(tree, filename):
527 new = str(tree).splitlines(True)
528 # Undo the adjustment of the line numbers in wrap_toks() below.
529 clipped, new = new[:lineno-1], new[lineno-1:]
530 assert clipped == ["\n"] * (lineno-1), clipped
531 if not new[-1].endswith("\n"):
532 new[-1] += "\n"
533 block = [indent + self.PS1 + new.pop(0)]
534 if new:
535 block += [indent + self.PS2 + line for line in new]
536 return block
537
538 def summarize(self):
Benjamin Peterson8951b612008-09-03 02:27:16 +0000539 if self.wrote:
Martin v. Löwisef04c442008-03-19 05:04:44 +0000540 were = "were"
541 else:
542 were = "need to be"
543 if not self.files:
544 self.log_message("No files %s modified.", were)
545 else:
546 self.log_message("Files that %s modified:", were)
547 for file in self.files:
548 self.log_message(file)
549 if self.fixer_log:
550 self.log_message("Warnings/messages while refactoring:")
551 for message in self.fixer_log:
552 self.log_message(message)
553 if self.errors:
554 if len(self.errors) == 1:
555 self.log_message("There was 1 error:")
556 else:
557 self.log_message("There were %d errors:", len(self.errors))
558 for msg, args, kwds in self.errors:
559 self.log_message(msg, *args, **kwds)
560
561 def parse_block(self, block, lineno, indent):
562 """Parses a block into a tree.
563
564 This is necessary to get correct line number / offset information
565 in the parser diagnostics and embedded into the parse tree.
566 """
Benjamin Peterson2243e5e2010-05-22 18:59:24 +0000567 tree = self.driver.parse_tokens(self.wrap_toks(block, lineno, indent))
568 tree.future_features = frozenset()
569 return tree
Martin v. Löwisef04c442008-03-19 05:04:44 +0000570
571 def wrap_toks(self, block, lineno, indent):
572 """Wraps a tokenize stream to systematically modify start/end."""
Martin v. Löwis8a5f8ca2008-03-19 05:33:36 +0000573 tokens = tokenize.generate_tokens(self.gen_lines(block, indent).__next__)
Martin v. Löwisef04c442008-03-19 05:04:44 +0000574 for type, value, (line0, col0), (line1, col1), line_text in tokens:
575 line0 += lineno - 1
576 line1 += lineno - 1
577 # Don't bother updating the columns; this is too complicated
578 # since line_text would also have to be updated and it would
579 # still break for tokens spanning lines. Let the user guess
580 # that the column numbers for doctests are relative to the
581 # end of the prompt string (PS1 or PS2).
582 yield type, value, (line0, col0), (line1, col1), line_text
583
584
585 def gen_lines(self, block, indent):
586 """Generates lines as expected by tokenize from a list of lines.
587
588 This strips the first len(indent + self.PS1) characters off each line.
589 """
590 prefix1 = indent + self.PS1
591 prefix2 = indent + self.PS2
592 prefix = prefix1
593 for line in block:
594 if line.startswith(prefix):
595 yield line[len(prefix):]
596 elif line == prefix.rstrip() + "\n":
597 yield "\n"
598 else:
599 raise AssertionError("line=%r, prefix=%r" % (line, prefix))
600 prefix = prefix2
601 while True:
602 yield ""
603
604
Benjamin Peterson608d8bc2009-05-05 23:23:31 +0000605class MultiprocessingUnsupported(Exception):
606 pass
607
608
609class MultiprocessRefactoringTool(RefactoringTool):
610
611 def __init__(self, *args, **kwargs):
612 super(MultiprocessRefactoringTool, self).__init__(*args, **kwargs)
613 self.queue = None
Benjamin Peterson8d26b0b2010-05-07 19:10:11 +0000614 self.output_lock = None
Benjamin Peterson608d8bc2009-05-05 23:23:31 +0000615
616 def refactor(self, items, write=False, doctests_only=False,
617 num_processes=1):
618 if num_processes == 1:
619 return super(MultiprocessRefactoringTool, self).refactor(
620 items, write, doctests_only)
621 try:
622 import multiprocessing
623 except ImportError:
624 raise MultiprocessingUnsupported
625 if self.queue is not None:
626 raise RuntimeError("already doing multiple processes")
627 self.queue = multiprocessing.JoinableQueue()
Benjamin Peterson8d26b0b2010-05-07 19:10:11 +0000628 self.output_lock = multiprocessing.Lock()
Benjamin Peterson608d8bc2009-05-05 23:23:31 +0000629 processes = [multiprocessing.Process(target=self._child)
Benjamin Peterson87b87192009-07-03 13:18:18 +0000630 for i in range(num_processes)]
Benjamin Peterson608d8bc2009-05-05 23:23:31 +0000631 try:
632 for p in processes:
633 p.start()
634 super(MultiprocessRefactoringTool, self).refactor(items, write,
635 doctests_only)
636 finally:
637 self.queue.join()
Benjamin Peterson87b87192009-07-03 13:18:18 +0000638 for i in range(num_processes):
Benjamin Peterson608d8bc2009-05-05 23:23:31 +0000639 self.queue.put(None)
640 for p in processes:
641 if p.is_alive():
642 p.join()
643 self.queue = None
644
645 def _child(self):
646 task = self.queue.get()
647 while task is not None:
648 args, kwargs = task
649 try:
650 super(MultiprocessRefactoringTool, self).refactor_file(
651 *args, **kwargs)
652 finally:
653 self.queue.task_done()
654 task = self.queue.get()
655
656 def refactor_file(self, *args, **kwargs):
657 if self.queue is not None:
658 self.queue.put((args, kwargs))
659 else:
660 return super(MultiprocessRefactoringTool, self).refactor_file(
661 *args, **kwargs)