2008-03-19 01:43:46 -03:00
|
|
|
#!/usr/bin/env python2.5
|
|
|
|
# Copyright 2006 Google, Inc. All Rights Reserved.
|
|
|
|
# Licensed to PSF under a Contributor Agreement.
|
|
|
|
|
|
|
|
"""Refactoring framework.
|
|
|
|
|
|
|
|
Used as a main program, this can refactor any number of files and/or
|
|
|
|
recursively descend down directories. Imported as a module, this
|
|
|
|
provides infrastructure to write your own refactoring tool.
|
|
|
|
"""
|
|
|
|
|
|
|
|
__author__ = "Guido van Rossum <guido@python.org>"
|
|
|
|
|
|
|
|
|
|
|
|
# Python imports
|
|
|
|
import os
|
|
|
|
import sys
|
|
|
|
import difflib
|
|
|
|
import logging
|
2008-09-02 21:21:32 -03:00
|
|
|
import operator
|
2008-05-02 18:30:20 -03:00
|
|
|
from collections import defaultdict
|
|
|
|
from itertools import chain
|
2008-03-19 01:43:46 -03:00
|
|
|
|
|
|
|
# Local imports
|
|
|
|
from .pgen2 import driver
|
|
|
|
from .pgen2 import tokenize
|
|
|
|
|
|
|
|
from . import pytree
|
|
|
|
from . import patcomp
|
|
|
|
from . import fixes
|
|
|
|
from . import pygram
|
|
|
|
|
|
|
|
|
2008-09-02 21:21:32 -03:00
|
|
|
def get_all_fix_names(fixer_pkg, remove_prefix=True):
|
|
|
|
"""Return a sorted list of all available fix names in the given package."""
|
|
|
|
pkg = __import__(fixer_pkg, [], [], ["*"])
|
|
|
|
fixer_dir = os.path.dirname(pkg.__file__)
|
2008-03-19 01:43:46 -03:00
|
|
|
fix_names = []
|
Merged revisions 66805,66841,66860,66884-66886,66893,66907,66910 via svnmerge from
svn+ssh://pythondev@svn.python.org/sandbox/trunk/2to3/lib2to3
........
r66805 | benjamin.peterson | 2008-10-04 20:11:02 -0500 (Sat, 04 Oct 2008) | 1 line
mention what the fixes directory is for
........
r66841 | benjamin.peterson | 2008-10-07 17:48:12 -0500 (Tue, 07 Oct 2008) | 1 line
use assertFalse and assertTrue
........
r66860 | benjamin.peterson | 2008-10-08 16:05:07 -0500 (Wed, 08 Oct 2008) | 1 line
instead of abusing the pattern matcher, use start_tree to find a next binding
........
r66884 | benjamin.peterson | 2008-10-13 15:50:30 -0500 (Mon, 13 Oct 2008) | 1 line
don't print tokens to stdout when -v is given
........
r66885 | benjamin.peterson | 2008-10-13 16:28:57 -0500 (Mon, 13 Oct 2008) | 1 line
add the -x option to disable fixers
........
r66886 | benjamin.peterson | 2008-10-13 16:33:53 -0500 (Mon, 13 Oct 2008) | 1 line
cut down on some crud
........
r66893 | benjamin.peterson | 2008-10-14 17:16:54 -0500 (Tue, 14 Oct 2008) | 1 line
add an optional set literal fixer
........
r66907 | benjamin.peterson | 2008-10-15 16:59:41 -0500 (Wed, 15 Oct 2008) | 1 line
don't write backup files by default
........
r66910 | benjamin.peterson | 2008-10-15 17:43:10 -0500 (Wed, 15 Oct 2008) | 1 line
add the -n option; it stops backupfiles from being written
........
2008-10-15 20:10:28 -03:00
|
|
|
for name in sorted(os.listdir(fixer_dir)):
|
2008-03-19 01:43:46 -03:00
|
|
|
if name.startswith("fix_") and name.endswith(".py"):
|
2008-09-02 21:21:32 -03:00
|
|
|
if remove_prefix:
|
|
|
|
name = name[4:]
|
|
|
|
fix_names.append(name[:-3])
|
2008-03-19 01:43:46 -03:00
|
|
|
return fix_names
|
|
|
|
|
2008-05-02 18:30:20 -03:00
|
|
|
def get_head_types(pat):
|
|
|
|
""" Accepts a pytree Pattern Node and returns a set
|
|
|
|
of the pattern types which will match first. """
|
|
|
|
|
|
|
|
if isinstance(pat, (pytree.NodePattern, pytree.LeafPattern)):
|
|
|
|
# NodePatters must either have no type and no content
|
|
|
|
# or a type and content -- so they don't get any farther
|
|
|
|
# Always return leafs
|
|
|
|
return set([pat.type])
|
|
|
|
|
|
|
|
if isinstance(pat, pytree.NegatedPattern):
|
|
|
|
if pat.content:
|
|
|
|
return get_head_types(pat.content)
|
|
|
|
return set([None]) # Negated Patterns don't have a type
|
|
|
|
|
|
|
|
if isinstance(pat, pytree.WildcardPattern):
|
|
|
|
# Recurse on each node in content
|
|
|
|
r = set()
|
|
|
|
for p in pat.content:
|
|
|
|
for x in p:
|
|
|
|
r.update(get_head_types(x))
|
|
|
|
return r
|
|
|
|
|
|
|
|
raise Exception("Oh no! I don't understand pattern %s" %(pat))
|
|
|
|
|
|
|
|
def get_headnode_dict(fixer_list):
|
|
|
|
""" Accepts a list of fixers and returns a dictionary
|
|
|
|
of head node type --> fixer list. """
|
|
|
|
head_nodes = defaultdict(list)
|
|
|
|
for fixer in fixer_list:
|
|
|
|
if not fixer.pattern:
|
|
|
|
head_nodes[None].append(fixer)
|
|
|
|
continue
|
|
|
|
for t in get_head_types(fixer.pattern):
|
|
|
|
head_nodes[t].append(fixer)
|
|
|
|
return head_nodes
|
|
|
|
|
2008-09-02 21:21:32 -03:00
|
|
|
def get_fixers_from_package(pkg_name):
|
|
|
|
"""
|
|
|
|
Return the fully qualified names for fixers in the package pkg_name.
|
|
|
|
"""
|
|
|
|
return [pkg_name + "." + fix_name
|
|
|
|
for fix_name in get_all_fix_names(pkg_name, False)]
|
|
|
|
|
2008-03-19 01:43:46 -03:00
|
|
|
|
Merged revisions 66511,66548-66549,66644,66646-66652 via svnmerge from
svn+ssh://pythondev@svn.python.org/sandbox/trunk/2to3/lib2to3
........
r66511 | benjamin.peterson | 2008-09-18 21:49:27 -0500 (Thu, 18 Sep 2008) | 1 line
remove a useless if __name__ == '__main__'
........
r66548 | benjamin.peterson | 2008-09-21 21:14:14 -0500 (Sun, 21 Sep 2008) | 1 line
avoid the perils of mutable default arguments
........
r66549 | benjamin.peterson | 2008-09-21 21:26:11 -0500 (Sun, 21 Sep 2008) | 1 line
some places in RefactoringTool should raise an error instead of logging it
........
r66644 | benjamin.peterson | 2008-09-27 10:45:10 -0500 (Sat, 27 Sep 2008) | 1 line
fix doctest refactoring
........
r66646 | benjamin.peterson | 2008-09-27 11:40:13 -0500 (Sat, 27 Sep 2008) | 1 line
don't print to stdout when 2to3 is used as a library
........
r66647 | benjamin.peterson | 2008-09-27 12:28:28 -0500 (Sat, 27 Sep 2008) | 1 line
let fixer modules and classes have different prefixes
........
r66648 | benjamin.peterson | 2008-09-27 14:02:13 -0500 (Sat, 27 Sep 2008) | 1 line
raise errors when 2to3 is used as a library
........
r66649 | benjamin.peterson | 2008-09-27 14:03:38 -0500 (Sat, 27 Sep 2008) | 1 line
fix docstring
........
r66650 | benjamin.peterson | 2008-09-27 14:22:21 -0500 (Sat, 27 Sep 2008) | 1 line
make use of enumerate
........
r66651 | benjamin.peterson | 2008-09-27 14:24:13 -0500 (Sat, 27 Sep 2008) | 1 line
revert last revision; it breaks things
........
r66652 | benjamin.peterson | 2008-09-27 16:03:06 -0500 (Sat, 27 Sep 2008) | 1 line
add tests for lib2to3.refactor
........
2008-09-27 18:09:10 -03:00
|
|
|
class FixerError(Exception):
|
|
|
|
"""A fixer could not be loaded."""
|
|
|
|
|
|
|
|
|
2008-03-19 01:43:46 -03:00
|
|
|
class RefactoringTool(object):
|
|
|
|
|
2008-09-02 21:21:32 -03:00
|
|
|
_default_options = {"print_function": False}
|
|
|
|
|
Merged revisions 66511,66548-66549,66644,66646-66652 via svnmerge from
svn+ssh://pythondev@svn.python.org/sandbox/trunk/2to3/lib2to3
........
r66511 | benjamin.peterson | 2008-09-18 21:49:27 -0500 (Thu, 18 Sep 2008) | 1 line
remove a useless if __name__ == '__main__'
........
r66548 | benjamin.peterson | 2008-09-21 21:14:14 -0500 (Sun, 21 Sep 2008) | 1 line
avoid the perils of mutable default arguments
........
r66549 | benjamin.peterson | 2008-09-21 21:26:11 -0500 (Sun, 21 Sep 2008) | 1 line
some places in RefactoringTool should raise an error instead of logging it
........
r66644 | benjamin.peterson | 2008-09-27 10:45:10 -0500 (Sat, 27 Sep 2008) | 1 line
fix doctest refactoring
........
r66646 | benjamin.peterson | 2008-09-27 11:40:13 -0500 (Sat, 27 Sep 2008) | 1 line
don't print to stdout when 2to3 is used as a library
........
r66647 | benjamin.peterson | 2008-09-27 12:28:28 -0500 (Sat, 27 Sep 2008) | 1 line
let fixer modules and classes have different prefixes
........
r66648 | benjamin.peterson | 2008-09-27 14:02:13 -0500 (Sat, 27 Sep 2008) | 1 line
raise errors when 2to3 is used as a library
........
r66649 | benjamin.peterson | 2008-09-27 14:03:38 -0500 (Sat, 27 Sep 2008) | 1 line
fix docstring
........
r66650 | benjamin.peterson | 2008-09-27 14:22:21 -0500 (Sat, 27 Sep 2008) | 1 line
make use of enumerate
........
r66651 | benjamin.peterson | 2008-09-27 14:24:13 -0500 (Sat, 27 Sep 2008) | 1 line
revert last revision; it breaks things
........
r66652 | benjamin.peterson | 2008-09-27 16:03:06 -0500 (Sat, 27 Sep 2008) | 1 line
add tests for lib2to3.refactor
........
2008-09-27 18:09:10 -03:00
|
|
|
CLASS_PREFIX = "Fix" # The prefix for fixer classes
|
|
|
|
FILE_PREFIX = "fix_" # The prefix for modules with a fixer within
|
|
|
|
|
|
|
|
def __init__(self, fixer_names, options=None, explicit=None):
|
2008-03-19 01:43:46 -03:00
|
|
|
"""Initializer.
|
|
|
|
|
Merged revisions 63661,63666,63695,63711,63729,63769,63790,63880,63886 via svnmerge from
svn+ssh://pythondev@svn.python.org/sandbox/trunk/2to3/lib2to3
........
r63661 | georg.brandl | 2008-05-26 05:26:20 -0500 (Mon, 26 May 2008) | 2 lines
Add import fixes for dbm package.
........
r63666 | georg.brandl | 2008-05-26 05:49:09 -0500 (Mon, 26 May 2008) | 2 lines
Add xmlrpc package fixes.
........
r63695 | georg.brandl | 2008-05-26 10:14:33 -0500 (Mon, 26 May 2008) | 2 lines
Add fixer entries for http package.
........
r63711 | benjamin.peterson | 2008-05-26 13:43:51 -0500 (Mon, 26 May 2008) | 2 lines
add import mapping for test.test_support -> test.support
........
r63729 | benjamin.peterson | 2008-05-26 16:31:03 -0500 (Mon, 26 May 2008) | 2 lines
mapping for commands module -> subprocess
........
r63769 | brett.cannon | 2008-05-29 00:13:13 -0500 (Thu, 29 May 2008) | 1 line
Fixer for UserString.UserString over to the collections module.
........
r63790 | brett.cannon | 2008-05-29 14:13:51 -0500 (Thu, 29 May 2008) | 4 lines
Add a fixer for UserList.
Closes issue #2878. Thanks to Quentin Gallet-Gilles for the patch.
........
r63880 | collin.winter | 2008-06-01 18:09:38 -0500 (Sun, 01 Jun 2008) | 6 lines
Move lib2to3/fixes/{basefix,util}.py down to lib2to3/.
This is step 1 of turning lib2to3/ into a general-purpose refactoring
library, reusable by other projects.
........
r63886 | collin.winter | 2008-06-01 22:15:01 -0500 (Sun, 01 Jun 2008) | 5 lines
Allow refactoring tools to specify a directory for fixer modules.
This is step 2 of turning lib2to3/ into a general-purpose refactoring
library, reusable by other projects. Step 1: r63880.
........
2008-06-14 23:31:05 -03:00
|
|
|
Args:
|
2008-09-02 21:21:32 -03:00
|
|
|
fixer_names: a list of fixers to import
|
|
|
|
options: an dict with configuration.
|
|
|
|
explicit: a list of fixers to run even if they are explicit.
|
2008-03-19 01:43:46 -03:00
|
|
|
"""
|
2008-09-02 21:21:32 -03:00
|
|
|
self.fixers = fixer_names
|
Merged revisions 66511,66548-66549,66644,66646-66652 via svnmerge from
svn+ssh://pythondev@svn.python.org/sandbox/trunk/2to3/lib2to3
........
r66511 | benjamin.peterson | 2008-09-18 21:49:27 -0500 (Thu, 18 Sep 2008) | 1 line
remove a useless if __name__ == '__main__'
........
r66548 | benjamin.peterson | 2008-09-21 21:14:14 -0500 (Sun, 21 Sep 2008) | 1 line
avoid the perils of mutable default arguments
........
r66549 | benjamin.peterson | 2008-09-21 21:26:11 -0500 (Sun, 21 Sep 2008) | 1 line
some places in RefactoringTool should raise an error instead of logging it
........
r66644 | benjamin.peterson | 2008-09-27 10:45:10 -0500 (Sat, 27 Sep 2008) | 1 line
fix doctest refactoring
........
r66646 | benjamin.peterson | 2008-09-27 11:40:13 -0500 (Sat, 27 Sep 2008) | 1 line
don't print to stdout when 2to3 is used as a library
........
r66647 | benjamin.peterson | 2008-09-27 12:28:28 -0500 (Sat, 27 Sep 2008) | 1 line
let fixer modules and classes have different prefixes
........
r66648 | benjamin.peterson | 2008-09-27 14:02:13 -0500 (Sat, 27 Sep 2008) | 1 line
raise errors when 2to3 is used as a library
........
r66649 | benjamin.peterson | 2008-09-27 14:03:38 -0500 (Sat, 27 Sep 2008) | 1 line
fix docstring
........
r66650 | benjamin.peterson | 2008-09-27 14:22:21 -0500 (Sat, 27 Sep 2008) | 1 line
make use of enumerate
........
r66651 | benjamin.peterson | 2008-09-27 14:24:13 -0500 (Sat, 27 Sep 2008) | 1 line
revert last revision; it breaks things
........
r66652 | benjamin.peterson | 2008-09-27 16:03:06 -0500 (Sat, 27 Sep 2008) | 1 line
add tests for lib2to3.refactor
........
2008-09-27 18:09:10 -03:00
|
|
|
self.explicit = explicit or []
|
2008-09-02 21:21:32 -03:00
|
|
|
self.options = self._default_options.copy()
|
|
|
|
if options is not None:
|
|
|
|
self.options.update(options)
|
2008-03-19 01:43:46 -03:00
|
|
|
self.errors = []
|
|
|
|
self.logger = logging.getLogger("RefactoringTool")
|
|
|
|
self.fixer_log = []
|
2008-09-02 21:21:32 -03:00
|
|
|
self.wrote = False
|
|
|
|
if self.options["print_function"]:
|
2008-03-19 01:43:46 -03:00
|
|
|
del pygram.python_grammar.keywords["print"]
|
|
|
|
self.driver = driver.Driver(pygram.python_grammar,
|
|
|
|
convert=pytree.convert,
|
|
|
|
logger=self.logger)
|
|
|
|
self.pre_order, self.post_order = self.get_fixers()
|
2008-05-02 18:30:20 -03:00
|
|
|
|
Merged revisions 67427,67431,67433,67435,67630,67652,67656-67657,67674-67675,67678-67679,67705-67706,67716,67723,67765-67771,67774,67776,67778 via svnmerge from
svn+ssh://pythondev@svn.python.org/sandbox/trunk/2to3/lib2to3
........
r67427 | benjamin.peterson | 2008-11-28 16:07:41 -0600 (Fri, 28 Nov 2008) | 1 line
fix spelling in comment
........
r67431 | benjamin.peterson | 2008-11-28 17:14:08 -0600 (Fri, 28 Nov 2008) | 1 line
add a scripts directory; move things to it
........
r67433 | benjamin.peterson | 2008-11-28 17:18:48 -0600 (Fri, 28 Nov 2008) | 1 line
run svneol.py
........
r67435 | benjamin.peterson | 2008-11-28 17:25:03 -0600 (Fri, 28 Nov 2008) | 1 line
rename pre/post_order_mapping to pre/post_order_heads
........
r67630 | alexandre.vassalotti | 2008-12-06 21:51:56 -0600 (Sat, 06 Dec 2008) | 2 lines
Fix typo in the urllib2.HTTPDigestAuthHandler fixer.
........
r67652 | armin.ronacher | 2008-12-07 15:39:43 -0600 (Sun, 07 Dec 2008) | 5 lines
Added a fixer that cleans up a tuple argument to isinstance after the tokens
in it were fixed. This is mainly used to remove double occurrences of
tokens as a leftover of the long -> int / unicode -> str conversion.
........
r67656 | armin.ronacher | 2008-12-07 16:54:16 -0600 (Sun, 07 Dec 2008) | 3 lines
Added missing copyright fo 2to3 fix_isinstance.
........
r67657 | armin.ronacher | 2008-12-07 18:29:35 -0600 (Sun, 07 Dec 2008) | 3 lines
2to3: intern and reduce fixes now add the imports if missing. Because that is a common task the fixer_util module now has a function "touch_import" that adds imports if missing.
........
r67674 | benjamin.peterson | 2008-12-08 19:58:11 -0600 (Mon, 08 Dec 2008) | 1 line
copy permission bits when making backup files #4602
........
r67675 | benjamin.peterson | 2008-12-08 19:59:11 -0600 (Mon, 08 Dec 2008) | 1 line
add forgotten import
........
r67678 | benjamin.peterson | 2008-12-08 20:08:30 -0600 (Mon, 08 Dec 2008) | 1 line
fix #4602 for real
........
r67679 | armin.ronacher | 2008-12-09 00:54:03 -0600 (Tue, 09 Dec 2008) | 3 lines
Removed redudant code from the 2to3 long fixer. This fixes #4590.
........
r67705 | benjamin.peterson | 2008-12-11 13:04:08 -0600 (Thu, 11 Dec 2008) | 1 line
put trailers after a range call after the list()
........
r67706 | benjamin.peterson | 2008-12-11 13:17:57 -0600 (Thu, 11 Dec 2008) | 1 line
add html related modules to the fix_imports mapping
........
r67716 | benjamin.peterson | 2008-12-11 22:16:47 -0600 (Thu, 11 Dec 2008) | 1 line
consolidate tests
........
r67723 | benjamin.peterson | 2008-12-12 19:49:31 -0600 (Fri, 12 Dec 2008) | 1 line
fix name
........
r67765 | benjamin.peterson | 2008-12-14 14:05:05 -0600 (Sun, 14 Dec 2008) | 1 line
run fix_isinstance after fix_long and fix_unicode
........
r67766 | benjamin.peterson | 2008-12-14 14:13:05 -0600 (Sun, 14 Dec 2008) | 1 line
use run_order instead of order
........
r67767 | benjamin.peterson | 2008-12-14 14:28:12 -0600 (Sun, 14 Dec 2008) | 1 line
don't retain parenthesis if there is only one item left
........
r67768 | benjamin.peterson | 2008-12-14 14:32:30 -0600 (Sun, 14 Dec 2008) | 1 line
use insert_child()
........
r67769 | benjamin.peterson | 2008-12-14 14:59:10 -0600 (Sun, 14 Dec 2008) | 1 line
parenthesize doesn't belong in pygram or FixerBase
........
r67770 | alexandre.vassalotti | 2008-12-14 15:15:36 -0600 (Sun, 14 Dec 2008) | 2 lines
Fix typo: html.paser -> html.parser.
........
r67771 | benjamin.peterson | 2008-12-14 15:22:09 -0600 (Sun, 14 Dec 2008) | 1 line
altering .children needs to call changed()
........
r67774 | benjamin.peterson | 2008-12-14 15:55:38 -0600 (Sun, 14 Dec 2008) | 1 line
employ an evil hack to fix multiple names in the same import statement
........
r67776 | benjamin.peterson | 2008-12-14 16:22:38 -0600 (Sun, 14 Dec 2008) | 1 line
make a common mixin class for Test_imports and friends
........
r67778 | alexandre.vassalotti | 2008-12-14 17:48:20 -0600 (Sun, 14 Dec 2008) | 2 lines
Make fix_imports refactor multiple imports as.
........
2008-12-15 23:35:28 -04:00
|
|
|
self.pre_order_heads = get_headnode_dict(self.pre_order)
|
|
|
|
self.post_order_heads = get_headnode_dict(self.post_order)
|
2008-05-02 18:30:20 -03:00
|
|
|
|
2008-03-19 01:43:46 -03:00
|
|
|
self.files = [] # List of files that were or should be modified
|
|
|
|
|
|
|
|
def get_fixers(self):
|
|
|
|
"""Inspects the options to load the requested patterns and handlers.
|
2008-03-19 02:22:42 -03:00
|
|
|
|
2008-03-19 01:43:46 -03:00
|
|
|
Returns:
|
|
|
|
(pre_order, post_order), where pre_order is the list of fixers that
|
|
|
|
want a pre-order AST traversal, and post_order is the list that want
|
|
|
|
post-order traversal.
|
|
|
|
"""
|
|
|
|
pre_order_fixers = []
|
|
|
|
post_order_fixers = []
|
2008-09-02 21:21:32 -03:00
|
|
|
for fix_mod_path in self.fixers:
|
Merged revisions 66511,66548-66549,66644,66646-66652 via svnmerge from
svn+ssh://pythondev@svn.python.org/sandbox/trunk/2to3/lib2to3
........
r66511 | benjamin.peterson | 2008-09-18 21:49:27 -0500 (Thu, 18 Sep 2008) | 1 line
remove a useless if __name__ == '__main__'
........
r66548 | benjamin.peterson | 2008-09-21 21:14:14 -0500 (Sun, 21 Sep 2008) | 1 line
avoid the perils of mutable default arguments
........
r66549 | benjamin.peterson | 2008-09-21 21:26:11 -0500 (Sun, 21 Sep 2008) | 1 line
some places in RefactoringTool should raise an error instead of logging it
........
r66644 | benjamin.peterson | 2008-09-27 10:45:10 -0500 (Sat, 27 Sep 2008) | 1 line
fix doctest refactoring
........
r66646 | benjamin.peterson | 2008-09-27 11:40:13 -0500 (Sat, 27 Sep 2008) | 1 line
don't print to stdout when 2to3 is used as a library
........
r66647 | benjamin.peterson | 2008-09-27 12:28:28 -0500 (Sat, 27 Sep 2008) | 1 line
let fixer modules and classes have different prefixes
........
r66648 | benjamin.peterson | 2008-09-27 14:02:13 -0500 (Sat, 27 Sep 2008) | 1 line
raise errors when 2to3 is used as a library
........
r66649 | benjamin.peterson | 2008-09-27 14:03:38 -0500 (Sat, 27 Sep 2008) | 1 line
fix docstring
........
r66650 | benjamin.peterson | 2008-09-27 14:22:21 -0500 (Sat, 27 Sep 2008) | 1 line
make use of enumerate
........
r66651 | benjamin.peterson | 2008-09-27 14:24:13 -0500 (Sat, 27 Sep 2008) | 1 line
revert last revision; it breaks things
........
r66652 | benjamin.peterson | 2008-09-27 16:03:06 -0500 (Sat, 27 Sep 2008) | 1 line
add tests for lib2to3.refactor
........
2008-09-27 18:09:10 -03:00
|
|
|
mod = __import__(fix_mod_path, {}, {}, ["*"])
|
2008-09-02 21:21:32 -03:00
|
|
|
fix_name = fix_mod_path.rsplit(".", 1)[-1]
|
Merged revisions 66511,66548-66549,66644,66646-66652 via svnmerge from
svn+ssh://pythondev@svn.python.org/sandbox/trunk/2to3/lib2to3
........
r66511 | benjamin.peterson | 2008-09-18 21:49:27 -0500 (Thu, 18 Sep 2008) | 1 line
remove a useless if __name__ == '__main__'
........
r66548 | benjamin.peterson | 2008-09-21 21:14:14 -0500 (Sun, 21 Sep 2008) | 1 line
avoid the perils of mutable default arguments
........
r66549 | benjamin.peterson | 2008-09-21 21:26:11 -0500 (Sun, 21 Sep 2008) | 1 line
some places in RefactoringTool should raise an error instead of logging it
........
r66644 | benjamin.peterson | 2008-09-27 10:45:10 -0500 (Sat, 27 Sep 2008) | 1 line
fix doctest refactoring
........
r66646 | benjamin.peterson | 2008-09-27 11:40:13 -0500 (Sat, 27 Sep 2008) | 1 line
don't print to stdout when 2to3 is used as a library
........
r66647 | benjamin.peterson | 2008-09-27 12:28:28 -0500 (Sat, 27 Sep 2008) | 1 line
let fixer modules and classes have different prefixes
........
r66648 | benjamin.peterson | 2008-09-27 14:02:13 -0500 (Sat, 27 Sep 2008) | 1 line
raise errors when 2to3 is used as a library
........
r66649 | benjamin.peterson | 2008-09-27 14:03:38 -0500 (Sat, 27 Sep 2008) | 1 line
fix docstring
........
r66650 | benjamin.peterson | 2008-09-27 14:22:21 -0500 (Sat, 27 Sep 2008) | 1 line
make use of enumerate
........
r66651 | benjamin.peterson | 2008-09-27 14:24:13 -0500 (Sat, 27 Sep 2008) | 1 line
revert last revision; it breaks things
........
r66652 | benjamin.peterson | 2008-09-27 16:03:06 -0500 (Sat, 27 Sep 2008) | 1 line
add tests for lib2to3.refactor
........
2008-09-27 18:09:10 -03:00
|
|
|
if fix_name.startswith(self.FILE_PREFIX):
|
|
|
|
fix_name = fix_name[len(self.FILE_PREFIX):]
|
2008-03-19 01:43:46 -03:00
|
|
|
parts = fix_name.split("_")
|
Merged revisions 66511,66548-66549,66644,66646-66652 via svnmerge from
svn+ssh://pythondev@svn.python.org/sandbox/trunk/2to3/lib2to3
........
r66511 | benjamin.peterson | 2008-09-18 21:49:27 -0500 (Thu, 18 Sep 2008) | 1 line
remove a useless if __name__ == '__main__'
........
r66548 | benjamin.peterson | 2008-09-21 21:14:14 -0500 (Sun, 21 Sep 2008) | 1 line
avoid the perils of mutable default arguments
........
r66549 | benjamin.peterson | 2008-09-21 21:26:11 -0500 (Sun, 21 Sep 2008) | 1 line
some places in RefactoringTool should raise an error instead of logging it
........
r66644 | benjamin.peterson | 2008-09-27 10:45:10 -0500 (Sat, 27 Sep 2008) | 1 line
fix doctest refactoring
........
r66646 | benjamin.peterson | 2008-09-27 11:40:13 -0500 (Sat, 27 Sep 2008) | 1 line
don't print to stdout when 2to3 is used as a library
........
r66647 | benjamin.peterson | 2008-09-27 12:28:28 -0500 (Sat, 27 Sep 2008) | 1 line
let fixer modules and classes have different prefixes
........
r66648 | benjamin.peterson | 2008-09-27 14:02:13 -0500 (Sat, 27 Sep 2008) | 1 line
raise errors when 2to3 is used as a library
........
r66649 | benjamin.peterson | 2008-09-27 14:03:38 -0500 (Sat, 27 Sep 2008) | 1 line
fix docstring
........
r66650 | benjamin.peterson | 2008-09-27 14:22:21 -0500 (Sat, 27 Sep 2008) | 1 line
make use of enumerate
........
r66651 | benjamin.peterson | 2008-09-27 14:24:13 -0500 (Sat, 27 Sep 2008) | 1 line
revert last revision; it breaks things
........
r66652 | benjamin.peterson | 2008-09-27 16:03:06 -0500 (Sat, 27 Sep 2008) | 1 line
add tests for lib2to3.refactor
........
2008-09-27 18:09:10 -03:00
|
|
|
class_name = self.CLASS_PREFIX + "".join([p.title() for p in parts])
|
2008-03-19 01:43:46 -03:00
|
|
|
try:
|
|
|
|
fix_class = getattr(mod, class_name)
|
|
|
|
except AttributeError:
|
Merged revisions 66511,66548-66549,66644,66646-66652 via svnmerge from
svn+ssh://pythondev@svn.python.org/sandbox/trunk/2to3/lib2to3
........
r66511 | benjamin.peterson | 2008-09-18 21:49:27 -0500 (Thu, 18 Sep 2008) | 1 line
remove a useless if __name__ == '__main__'
........
r66548 | benjamin.peterson | 2008-09-21 21:14:14 -0500 (Sun, 21 Sep 2008) | 1 line
avoid the perils of mutable default arguments
........
r66549 | benjamin.peterson | 2008-09-21 21:26:11 -0500 (Sun, 21 Sep 2008) | 1 line
some places in RefactoringTool should raise an error instead of logging it
........
r66644 | benjamin.peterson | 2008-09-27 10:45:10 -0500 (Sat, 27 Sep 2008) | 1 line
fix doctest refactoring
........
r66646 | benjamin.peterson | 2008-09-27 11:40:13 -0500 (Sat, 27 Sep 2008) | 1 line
don't print to stdout when 2to3 is used as a library
........
r66647 | benjamin.peterson | 2008-09-27 12:28:28 -0500 (Sat, 27 Sep 2008) | 1 line
let fixer modules and classes have different prefixes
........
r66648 | benjamin.peterson | 2008-09-27 14:02:13 -0500 (Sat, 27 Sep 2008) | 1 line
raise errors when 2to3 is used as a library
........
r66649 | benjamin.peterson | 2008-09-27 14:03:38 -0500 (Sat, 27 Sep 2008) | 1 line
fix docstring
........
r66650 | benjamin.peterson | 2008-09-27 14:22:21 -0500 (Sat, 27 Sep 2008) | 1 line
make use of enumerate
........
r66651 | benjamin.peterson | 2008-09-27 14:24:13 -0500 (Sat, 27 Sep 2008) | 1 line
revert last revision; it breaks things
........
r66652 | benjamin.peterson | 2008-09-27 16:03:06 -0500 (Sat, 27 Sep 2008) | 1 line
add tests for lib2to3.refactor
........
2008-09-27 18:09:10 -03:00
|
|
|
raise FixerError("Can't find %s.%s" % (fix_name, class_name))
|
|
|
|
fixer = fix_class(self.options, self.fixer_log)
|
2008-09-02 21:21:32 -03:00
|
|
|
if fixer.explicit and self.explicit is not True and \
|
|
|
|
fix_mod_path not in self.explicit:
|
2008-03-19 01:43:46 -03:00
|
|
|
self.log_message("Skipping implicit fixer: %s", fix_name)
|
|
|
|
continue
|
|
|
|
|
2008-09-02 21:21:32 -03:00
|
|
|
self.log_debug("Adding transformation: %s", fix_name)
|
2008-03-19 01:43:46 -03:00
|
|
|
if fixer.order == "pre":
|
|
|
|
pre_order_fixers.append(fixer)
|
|
|
|
elif fixer.order == "post":
|
|
|
|
post_order_fixers.append(fixer)
|
|
|
|
else:
|
Merged revisions 66511,66548-66549,66644,66646-66652 via svnmerge from
svn+ssh://pythondev@svn.python.org/sandbox/trunk/2to3/lib2to3
........
r66511 | benjamin.peterson | 2008-09-18 21:49:27 -0500 (Thu, 18 Sep 2008) | 1 line
remove a useless if __name__ == '__main__'
........
r66548 | benjamin.peterson | 2008-09-21 21:14:14 -0500 (Sun, 21 Sep 2008) | 1 line
avoid the perils of mutable default arguments
........
r66549 | benjamin.peterson | 2008-09-21 21:26:11 -0500 (Sun, 21 Sep 2008) | 1 line
some places in RefactoringTool should raise an error instead of logging it
........
r66644 | benjamin.peterson | 2008-09-27 10:45:10 -0500 (Sat, 27 Sep 2008) | 1 line
fix doctest refactoring
........
r66646 | benjamin.peterson | 2008-09-27 11:40:13 -0500 (Sat, 27 Sep 2008) | 1 line
don't print to stdout when 2to3 is used as a library
........
r66647 | benjamin.peterson | 2008-09-27 12:28:28 -0500 (Sat, 27 Sep 2008) | 1 line
let fixer modules and classes have different prefixes
........
r66648 | benjamin.peterson | 2008-09-27 14:02:13 -0500 (Sat, 27 Sep 2008) | 1 line
raise errors when 2to3 is used as a library
........
r66649 | benjamin.peterson | 2008-09-27 14:03:38 -0500 (Sat, 27 Sep 2008) | 1 line
fix docstring
........
r66650 | benjamin.peterson | 2008-09-27 14:22:21 -0500 (Sat, 27 Sep 2008) | 1 line
make use of enumerate
........
r66651 | benjamin.peterson | 2008-09-27 14:24:13 -0500 (Sat, 27 Sep 2008) | 1 line
revert last revision; it breaks things
........
r66652 | benjamin.peterson | 2008-09-27 16:03:06 -0500 (Sat, 27 Sep 2008) | 1 line
add tests for lib2to3.refactor
........
2008-09-27 18:09:10 -03:00
|
|
|
raise FixerError("Illegal fixer order: %r" % fixer.order)
|
Merged revisions 61602-61723 via svnmerge from
svn+ssh://pythondev@svn.python.org/sandbox/trunk/2to3/lib2to3
........
r61626 | david.wolever | 2008-03-19 17:19:16 +0100 (Mi, 19 Mär 2008) | 1 line
Added fixer for implicit local imports. See #2414.
........
r61628 | david.wolever | 2008-03-19 17:57:43 +0100 (Mi, 19 Mär 2008) | 1 line
Added a class for tests which should not run if a particular import is found.
........
r61629 | collin.winter | 2008-03-19 17:58:19 +0100 (Mi, 19 Mär 2008) | 1 line
Two more relative import fixes in pgen2.
........
r61635 | david.wolever | 2008-03-19 20:16:03 +0100 (Mi, 19 Mär 2008) | 1 line
Fixed print fixer so it will do the Right Thing when it encounters __future__.print_function. 2to3 gets upset, though, so the tests have been commented out.
........
r61637 | david.wolever | 2008-03-19 21:37:17 +0100 (Mi, 19 Mär 2008) | 3 lines
Added a fixer for itertools imports (from itertools import imap, ifilterfalse --> from itertools import filterfalse)
........
r61645 | david.wolever | 2008-03-19 23:22:35 +0100 (Mi, 19 Mär 2008) | 1 line
SVN is happier when you add the files you create... -_-'
........
r61654 | david.wolever | 2008-03-20 01:09:56 +0100 (Do, 20 Mär 2008) | 1 line
Added an explicit sort order to fixers -- fixes problems like #2427
........
r61664 | david.wolever | 2008-03-20 04:32:40 +0100 (Do, 20 Mär 2008) | 3 lines
Fixes #2428 -- comments are no longer eatten by __future__ fixer.
........
r61673 | david.wolever | 2008-03-20 17:22:40 +0100 (Do, 20 Mär 2008) | 1 line
Added 2to3 node pretty-printer
........
r61679 | david.wolever | 2008-03-20 20:50:42 +0100 (Do, 20 Mär 2008) | 1 line
Made node printing a little bit prettier
........
r61723 | martin.v.loewis | 2008-03-22 00:59:27 +0100 (Sa, 22 Mär 2008) | 2 lines
Fix whitespace.
........
2008-03-21 21:01:12 -03:00
|
|
|
|
2008-09-02 21:21:32 -03:00
|
|
|
key_func = operator.attrgetter("run_order")
|
|
|
|
pre_order_fixers.sort(key=key_func)
|
|
|
|
post_order_fixers.sort(key=key_func)
|
2008-03-19 01:43:46 -03:00
|
|
|
return (pre_order_fixers, post_order_fixers)
|
|
|
|
|
|
|
|
def log_error(self, msg, *args, **kwds):
|
Merged revisions 66511,66548-66549,66644,66646-66652 via svnmerge from
svn+ssh://pythondev@svn.python.org/sandbox/trunk/2to3/lib2to3
........
r66511 | benjamin.peterson | 2008-09-18 21:49:27 -0500 (Thu, 18 Sep 2008) | 1 line
remove a useless if __name__ == '__main__'
........
r66548 | benjamin.peterson | 2008-09-21 21:14:14 -0500 (Sun, 21 Sep 2008) | 1 line
avoid the perils of mutable default arguments
........
r66549 | benjamin.peterson | 2008-09-21 21:26:11 -0500 (Sun, 21 Sep 2008) | 1 line
some places in RefactoringTool should raise an error instead of logging it
........
r66644 | benjamin.peterson | 2008-09-27 10:45:10 -0500 (Sat, 27 Sep 2008) | 1 line
fix doctest refactoring
........
r66646 | benjamin.peterson | 2008-09-27 11:40:13 -0500 (Sat, 27 Sep 2008) | 1 line
don't print to stdout when 2to3 is used as a library
........
r66647 | benjamin.peterson | 2008-09-27 12:28:28 -0500 (Sat, 27 Sep 2008) | 1 line
let fixer modules and classes have different prefixes
........
r66648 | benjamin.peterson | 2008-09-27 14:02:13 -0500 (Sat, 27 Sep 2008) | 1 line
raise errors when 2to3 is used as a library
........
r66649 | benjamin.peterson | 2008-09-27 14:03:38 -0500 (Sat, 27 Sep 2008) | 1 line
fix docstring
........
r66650 | benjamin.peterson | 2008-09-27 14:22:21 -0500 (Sat, 27 Sep 2008) | 1 line
make use of enumerate
........
r66651 | benjamin.peterson | 2008-09-27 14:24:13 -0500 (Sat, 27 Sep 2008) | 1 line
revert last revision; it breaks things
........
r66652 | benjamin.peterson | 2008-09-27 16:03:06 -0500 (Sat, 27 Sep 2008) | 1 line
add tests for lib2to3.refactor
........
2008-09-27 18:09:10 -03:00
|
|
|
"""Called when an error occurs."""
|
|
|
|
raise
|
2008-03-19 01:43:46 -03:00
|
|
|
|
|
|
|
def log_message(self, msg, *args):
|
|
|
|
"""Hook to log a message."""
|
|
|
|
if args:
|
|
|
|
msg = msg % args
|
|
|
|
self.logger.info(msg)
|
|
|
|
|
2008-09-02 21:21:32 -03:00
|
|
|
def log_debug(self, msg, *args):
|
|
|
|
if args:
|
|
|
|
msg = msg % args
|
|
|
|
self.logger.debug(msg)
|
|
|
|
|
Merged revisions 66511,66548-66549,66644,66646-66652 via svnmerge from
svn+ssh://pythondev@svn.python.org/sandbox/trunk/2to3/lib2to3
........
r66511 | benjamin.peterson | 2008-09-18 21:49:27 -0500 (Thu, 18 Sep 2008) | 1 line
remove a useless if __name__ == '__main__'
........
r66548 | benjamin.peterson | 2008-09-21 21:14:14 -0500 (Sun, 21 Sep 2008) | 1 line
avoid the perils of mutable default arguments
........
r66549 | benjamin.peterson | 2008-09-21 21:26:11 -0500 (Sun, 21 Sep 2008) | 1 line
some places in RefactoringTool should raise an error instead of logging it
........
r66644 | benjamin.peterson | 2008-09-27 10:45:10 -0500 (Sat, 27 Sep 2008) | 1 line
fix doctest refactoring
........
r66646 | benjamin.peterson | 2008-09-27 11:40:13 -0500 (Sat, 27 Sep 2008) | 1 line
don't print to stdout when 2to3 is used as a library
........
r66647 | benjamin.peterson | 2008-09-27 12:28:28 -0500 (Sat, 27 Sep 2008) | 1 line
let fixer modules and classes have different prefixes
........
r66648 | benjamin.peterson | 2008-09-27 14:02:13 -0500 (Sat, 27 Sep 2008) | 1 line
raise errors when 2to3 is used as a library
........
r66649 | benjamin.peterson | 2008-09-27 14:03:38 -0500 (Sat, 27 Sep 2008) | 1 line
fix docstring
........
r66650 | benjamin.peterson | 2008-09-27 14:22:21 -0500 (Sat, 27 Sep 2008) | 1 line
make use of enumerate
........
r66651 | benjamin.peterson | 2008-09-27 14:24:13 -0500 (Sat, 27 Sep 2008) | 1 line
revert last revision; it breaks things
........
r66652 | benjamin.peterson | 2008-09-27 16:03:06 -0500 (Sat, 27 Sep 2008) | 1 line
add tests for lib2to3.refactor
........
2008-09-27 18:09:10 -03:00
|
|
|
def print_output(self, lines):
|
|
|
|
"""Called with lines of output to give to the user."""
|
|
|
|
pass
|
|
|
|
|
2008-09-02 21:21:32 -03:00
|
|
|
def refactor(self, items, write=False, doctests_only=False):
|
|
|
|
"""Refactor a list of files and directories."""
|
|
|
|
for dir_or_file in items:
|
|
|
|
if os.path.isdir(dir_or_file):
|
Merged revisions 66511,66548-66549,66644,66646-66652 via svnmerge from
svn+ssh://pythondev@svn.python.org/sandbox/trunk/2to3/lib2to3
........
r66511 | benjamin.peterson | 2008-09-18 21:49:27 -0500 (Thu, 18 Sep 2008) | 1 line
remove a useless if __name__ == '__main__'
........
r66548 | benjamin.peterson | 2008-09-21 21:14:14 -0500 (Sun, 21 Sep 2008) | 1 line
avoid the perils of mutable default arguments
........
r66549 | benjamin.peterson | 2008-09-21 21:26:11 -0500 (Sun, 21 Sep 2008) | 1 line
some places in RefactoringTool should raise an error instead of logging it
........
r66644 | benjamin.peterson | 2008-09-27 10:45:10 -0500 (Sat, 27 Sep 2008) | 1 line
fix doctest refactoring
........
r66646 | benjamin.peterson | 2008-09-27 11:40:13 -0500 (Sat, 27 Sep 2008) | 1 line
don't print to stdout when 2to3 is used as a library
........
r66647 | benjamin.peterson | 2008-09-27 12:28:28 -0500 (Sat, 27 Sep 2008) | 1 line
let fixer modules and classes have different prefixes
........
r66648 | benjamin.peterson | 2008-09-27 14:02:13 -0500 (Sat, 27 Sep 2008) | 1 line
raise errors when 2to3 is used as a library
........
r66649 | benjamin.peterson | 2008-09-27 14:03:38 -0500 (Sat, 27 Sep 2008) | 1 line
fix docstring
........
r66650 | benjamin.peterson | 2008-09-27 14:22:21 -0500 (Sat, 27 Sep 2008) | 1 line
make use of enumerate
........
r66651 | benjamin.peterson | 2008-09-27 14:24:13 -0500 (Sat, 27 Sep 2008) | 1 line
revert last revision; it breaks things
........
r66652 | benjamin.peterson | 2008-09-27 16:03:06 -0500 (Sat, 27 Sep 2008) | 1 line
add tests for lib2to3.refactor
........
2008-09-27 18:09:10 -03:00
|
|
|
self.refactor_dir(dir_or_file, write, doctests_only)
|
2008-03-19 01:43:46 -03:00
|
|
|
else:
|
Merged revisions 66511,66548-66549,66644,66646-66652 via svnmerge from
svn+ssh://pythondev@svn.python.org/sandbox/trunk/2to3/lib2to3
........
r66511 | benjamin.peterson | 2008-09-18 21:49:27 -0500 (Thu, 18 Sep 2008) | 1 line
remove a useless if __name__ == '__main__'
........
r66548 | benjamin.peterson | 2008-09-21 21:14:14 -0500 (Sun, 21 Sep 2008) | 1 line
avoid the perils of mutable default arguments
........
r66549 | benjamin.peterson | 2008-09-21 21:26:11 -0500 (Sun, 21 Sep 2008) | 1 line
some places in RefactoringTool should raise an error instead of logging it
........
r66644 | benjamin.peterson | 2008-09-27 10:45:10 -0500 (Sat, 27 Sep 2008) | 1 line
fix doctest refactoring
........
r66646 | benjamin.peterson | 2008-09-27 11:40:13 -0500 (Sat, 27 Sep 2008) | 1 line
don't print to stdout when 2to3 is used as a library
........
r66647 | benjamin.peterson | 2008-09-27 12:28:28 -0500 (Sat, 27 Sep 2008) | 1 line
let fixer modules and classes have different prefixes
........
r66648 | benjamin.peterson | 2008-09-27 14:02:13 -0500 (Sat, 27 Sep 2008) | 1 line
raise errors when 2to3 is used as a library
........
r66649 | benjamin.peterson | 2008-09-27 14:03:38 -0500 (Sat, 27 Sep 2008) | 1 line
fix docstring
........
r66650 | benjamin.peterson | 2008-09-27 14:22:21 -0500 (Sat, 27 Sep 2008) | 1 line
make use of enumerate
........
r66651 | benjamin.peterson | 2008-09-27 14:24:13 -0500 (Sat, 27 Sep 2008) | 1 line
revert last revision; it breaks things
........
r66652 | benjamin.peterson | 2008-09-27 16:03:06 -0500 (Sat, 27 Sep 2008) | 1 line
add tests for lib2to3.refactor
........
2008-09-27 18:09:10 -03:00
|
|
|
self.refactor_file(dir_or_file, write, doctests_only)
|
2008-03-19 01:43:46 -03:00
|
|
|
|
2008-09-02 21:21:32 -03:00
|
|
|
def refactor_dir(self, dir_name, write=False, doctests_only=False):
|
2008-03-19 01:43:46 -03:00
|
|
|
"""Descends down a directory and refactor every Python file found.
|
|
|
|
|
|
|
|
Python files are assumed to have a .py extension.
|
|
|
|
|
|
|
|
Files and subdirectories starting with '.' are skipped.
|
|
|
|
"""
|
2008-09-02 21:21:32 -03:00
|
|
|
for dirpath, dirnames, filenames in os.walk(dir_name):
|
|
|
|
self.log_debug("Descending into %s", dirpath)
|
2008-03-19 01:43:46 -03:00
|
|
|
dirnames.sort()
|
|
|
|
filenames.sort()
|
|
|
|
for name in filenames:
|
|
|
|
if not name.startswith(".") and name.endswith("py"):
|
|
|
|
fullname = os.path.join(dirpath, name)
|
2008-09-02 21:21:32 -03:00
|
|
|
self.refactor_file(fullname, write, doctests_only)
|
2008-03-19 01:43:46 -03:00
|
|
|
# Modify dirnames in-place to remove subdirs with leading dots
|
|
|
|
dirnames[:] = [dn for dn in dirnames if not dn.startswith(".")]
|
|
|
|
|
2008-09-02 21:21:32 -03:00
|
|
|
def refactor_file(self, filename, write=False, doctests_only=False):
|
2008-03-19 01:43:46 -03:00
|
|
|
"""Refactors a file."""
|
|
|
|
try:
|
|
|
|
f = open(filename)
|
|
|
|
except IOError, err:
|
|
|
|
self.log_error("Can't open %s: %s", filename, err)
|
|
|
|
return
|
|
|
|
try:
|
|
|
|
input = f.read() + "\n" # Silence certain parse errors
|
|
|
|
finally:
|
|
|
|
f.close()
|
2008-09-02 21:21:32 -03:00
|
|
|
if doctests_only:
|
|
|
|
self.log_debug("Refactoring doctests in %s", filename)
|
2008-03-19 01:43:46 -03:00
|
|
|
output = self.refactor_docstring(input, filename)
|
|
|
|
if output != input:
|
2008-09-02 21:21:32 -03:00
|
|
|
self.processed_file(output, filename, input, write=write)
|
|
|
|
else:
|
|
|
|
self.log_debug("No doctest changes in %s", filename)
|
2008-03-19 01:43:46 -03:00
|
|
|
else:
|
|
|
|
tree = self.refactor_string(input, filename)
|
|
|
|
if tree and tree.was_changed:
|
|
|
|
# The [:-1] is to take off the \n we added earlier
|
2008-09-02 21:21:32 -03:00
|
|
|
self.processed_file(str(tree)[:-1], filename, write=write)
|
|
|
|
else:
|
|
|
|
self.log_debug("No changes in %s", filename)
|
2008-03-19 01:43:46 -03:00
|
|
|
|
|
|
|
def refactor_string(self, data, name):
|
|
|
|
"""Refactor a given input string.
|
2008-03-19 02:22:42 -03:00
|
|
|
|
2008-03-19 01:43:46 -03:00
|
|
|
Args:
|
|
|
|
data: a string holding the code to be refactored.
|
|
|
|
name: a human-readable name for use in error/log messages.
|
2008-03-19 02:22:42 -03:00
|
|
|
|
2008-03-19 01:43:46 -03:00
|
|
|
Returns:
|
|
|
|
An AST corresponding to the refactored input stream; None if
|
|
|
|
there were errors during the parse.
|
|
|
|
"""
|
|
|
|
try:
|
Merged revisions 66805,66841,66860,66884-66886,66893,66907,66910 via svnmerge from
svn+ssh://pythondev@svn.python.org/sandbox/trunk/2to3/lib2to3
........
r66805 | benjamin.peterson | 2008-10-04 20:11:02 -0500 (Sat, 04 Oct 2008) | 1 line
mention what the fixes directory is for
........
r66841 | benjamin.peterson | 2008-10-07 17:48:12 -0500 (Tue, 07 Oct 2008) | 1 line
use assertFalse and assertTrue
........
r66860 | benjamin.peterson | 2008-10-08 16:05:07 -0500 (Wed, 08 Oct 2008) | 1 line
instead of abusing the pattern matcher, use start_tree to find a next binding
........
r66884 | benjamin.peterson | 2008-10-13 15:50:30 -0500 (Mon, 13 Oct 2008) | 1 line
don't print tokens to stdout when -v is given
........
r66885 | benjamin.peterson | 2008-10-13 16:28:57 -0500 (Mon, 13 Oct 2008) | 1 line
add the -x option to disable fixers
........
r66886 | benjamin.peterson | 2008-10-13 16:33:53 -0500 (Mon, 13 Oct 2008) | 1 line
cut down on some crud
........
r66893 | benjamin.peterson | 2008-10-14 17:16:54 -0500 (Tue, 14 Oct 2008) | 1 line
add an optional set literal fixer
........
r66907 | benjamin.peterson | 2008-10-15 16:59:41 -0500 (Wed, 15 Oct 2008) | 1 line
don't write backup files by default
........
r66910 | benjamin.peterson | 2008-10-15 17:43:10 -0500 (Wed, 15 Oct 2008) | 1 line
add the -n option; it stops backupfiles from being written
........
2008-10-15 20:10:28 -03:00
|
|
|
tree = self.driver.parse_string(data)
|
2008-03-19 01:43:46 -03:00
|
|
|
except Exception, err:
|
|
|
|
self.log_error("Can't parse %s: %s: %s",
|
|
|
|
name, err.__class__.__name__, err)
|
|
|
|
return
|
2008-09-02 21:21:32 -03:00
|
|
|
self.log_debug("Refactoring %s", name)
|
2008-03-19 01:43:46 -03:00
|
|
|
self.refactor_tree(tree, name)
|
|
|
|
return tree
|
|
|
|
|
2008-09-02 21:21:32 -03:00
|
|
|
def refactor_stdin(self, doctests_only=False):
|
2008-03-19 01:43:46 -03:00
|
|
|
input = sys.stdin.read()
|
2008-09-02 21:21:32 -03:00
|
|
|
if doctests_only:
|
|
|
|
self.log_debug("Refactoring doctests in stdin")
|
2008-03-19 01:43:46 -03:00
|
|
|
output = self.refactor_docstring(input, "<stdin>")
|
|
|
|
if output != input:
|
2008-09-02 21:21:32 -03:00
|
|
|
self.processed_file(output, "<stdin>", input)
|
|
|
|
else:
|
|
|
|
self.log_debug("No doctest changes in stdin")
|
2008-03-19 01:43:46 -03:00
|
|
|
else:
|
|
|
|
tree = self.refactor_string(input, "<stdin>")
|
|
|
|
if tree and tree.was_changed:
|
2008-09-02 21:21:32 -03:00
|
|
|
self.processed_file(str(tree), "<stdin>", input)
|
|
|
|
else:
|
|
|
|
self.log_debug("No changes in stdin")
|
2008-03-19 01:43:46 -03:00
|
|
|
|
|
|
|
def refactor_tree(self, tree, name):
|
|
|
|
"""Refactors a parse tree (modifying the tree in place).
|
2008-03-19 02:22:42 -03:00
|
|
|
|
2008-03-19 01:43:46 -03:00
|
|
|
Args:
|
|
|
|
tree: a pytree.Node instance representing the root of the tree
|
|
|
|
to be refactored.
|
|
|
|
name: a human-readable name for this tree.
|
2008-03-19 02:22:42 -03:00
|
|
|
|
2008-03-19 01:43:46 -03:00
|
|
|
Returns:
|
|
|
|
True if the tree was modified, False otherwise.
|
|
|
|
"""
|
Merged revisions 67900-67901,67919,67928,67984,67991-67993,68106-68108,68110 via svnmerge from
svn+ssh://pythondev@svn.python.org/sandbox/trunk/2to3/lib2to3
........
r67900 | benjamin.peterson | 2008-12-22 14:02:45 -0600 (Mon, 22 Dec 2008) | 4 lines
fix_execfile: wrap the open(fn).read() call in compile(), so the filename is preserved
also add unittests for the fixer
........
r67901 | benjamin.peterson | 2008-12-22 14:09:55 -0600 (Mon, 22 Dec 2008) | 1 line
remove unused import
........
r67919 | benjamin.peterson | 2008-12-23 13:12:22 -0600 (Tue, 23 Dec 2008) | 1 line
copy permission bits from the backup to the original
........
r67928 | benjamin.peterson | 2008-12-26 20:49:30 -0600 (Fri, 26 Dec 2008) | 1 line
don't be so idiot about multiple local imports in fix_import; still won't handle absolute and local imports on the same line
........
r67984 | benjamin.peterson | 2008-12-28 09:55:16 -0600 (Sun, 28 Dec 2008) | 1 line
don't need loop
........
r67991 | benjamin.peterson | 2008-12-28 14:30:26 -0600 (Sun, 28 Dec 2008) | 1 line
actually call finish_tree()
........
r67992 | benjamin.peterson | 2008-12-28 14:34:47 -0600 (Sun, 28 Dec 2008) | 1 line
remove useless test
........
r67993 | benjamin.peterson | 2008-12-28 15:04:32 -0600 (Sun, 28 Dec 2008) | 1 line
update pyk3's test grammar
........
r68106 | benjamin.peterson | 2008-12-31 11:53:58 -0600 (Wed, 31 Dec 2008) | 1 line
#2734 don't convert every instance of long (eg if it's an attribute)
........
r68107 | benjamin.peterson | 2008-12-31 11:55:10 -0600 (Wed, 31 Dec 2008) | 1 line
add another test
........
r68108 | benjamin.peterson | 2008-12-31 12:00:12 -0600 (Wed, 31 Dec 2008) | 1 line
don't change long even if it's the only argument name
........
r68110 | benjamin.peterson | 2008-12-31 14:13:26 -0600 (Wed, 31 Dec 2008) | 1 line
remove unused import
........
2009-01-03 12:34:02 -04:00
|
|
|
for fixer in chain(self.pre_order, self.post_order):
|
2008-03-19 01:43:46 -03:00
|
|
|
fixer.start_tree(tree, name)
|
|
|
|
|
Merged revisions 67427,67431,67433,67435,67630,67652,67656-67657,67674-67675,67678-67679,67705-67706,67716,67723,67765-67771,67774,67776,67778 via svnmerge from
svn+ssh://pythondev@svn.python.org/sandbox/trunk/2to3/lib2to3
........
r67427 | benjamin.peterson | 2008-11-28 16:07:41 -0600 (Fri, 28 Nov 2008) | 1 line
fix spelling in comment
........
r67431 | benjamin.peterson | 2008-11-28 17:14:08 -0600 (Fri, 28 Nov 2008) | 1 line
add a scripts directory; move things to it
........
r67433 | benjamin.peterson | 2008-11-28 17:18:48 -0600 (Fri, 28 Nov 2008) | 1 line
run svneol.py
........
r67435 | benjamin.peterson | 2008-11-28 17:25:03 -0600 (Fri, 28 Nov 2008) | 1 line
rename pre/post_order_mapping to pre/post_order_heads
........
r67630 | alexandre.vassalotti | 2008-12-06 21:51:56 -0600 (Sat, 06 Dec 2008) | 2 lines
Fix typo in the urllib2.HTTPDigestAuthHandler fixer.
........
r67652 | armin.ronacher | 2008-12-07 15:39:43 -0600 (Sun, 07 Dec 2008) | 5 lines
Added a fixer that cleans up a tuple argument to isinstance after the tokens
in it were fixed. This is mainly used to remove double occurrences of
tokens as a leftover of the long -> int / unicode -> str conversion.
........
r67656 | armin.ronacher | 2008-12-07 16:54:16 -0600 (Sun, 07 Dec 2008) | 3 lines
Added missing copyright fo 2to3 fix_isinstance.
........
r67657 | armin.ronacher | 2008-12-07 18:29:35 -0600 (Sun, 07 Dec 2008) | 3 lines
2to3: intern and reduce fixes now add the imports if missing. Because that is a common task the fixer_util module now has a function "touch_import" that adds imports if missing.
........
r67674 | benjamin.peterson | 2008-12-08 19:58:11 -0600 (Mon, 08 Dec 2008) | 1 line
copy permission bits when making backup files #4602
........
r67675 | benjamin.peterson | 2008-12-08 19:59:11 -0600 (Mon, 08 Dec 2008) | 1 line
add forgotten import
........
r67678 | benjamin.peterson | 2008-12-08 20:08:30 -0600 (Mon, 08 Dec 2008) | 1 line
fix #4602 for real
........
r67679 | armin.ronacher | 2008-12-09 00:54:03 -0600 (Tue, 09 Dec 2008) | 3 lines
Removed redudant code from the 2to3 long fixer. This fixes #4590.
........
r67705 | benjamin.peterson | 2008-12-11 13:04:08 -0600 (Thu, 11 Dec 2008) | 1 line
put trailers after a range call after the list()
........
r67706 | benjamin.peterson | 2008-12-11 13:17:57 -0600 (Thu, 11 Dec 2008) | 1 line
add html related modules to the fix_imports mapping
........
r67716 | benjamin.peterson | 2008-12-11 22:16:47 -0600 (Thu, 11 Dec 2008) | 1 line
consolidate tests
........
r67723 | benjamin.peterson | 2008-12-12 19:49:31 -0600 (Fri, 12 Dec 2008) | 1 line
fix name
........
r67765 | benjamin.peterson | 2008-12-14 14:05:05 -0600 (Sun, 14 Dec 2008) | 1 line
run fix_isinstance after fix_long and fix_unicode
........
r67766 | benjamin.peterson | 2008-12-14 14:13:05 -0600 (Sun, 14 Dec 2008) | 1 line
use run_order instead of order
........
r67767 | benjamin.peterson | 2008-12-14 14:28:12 -0600 (Sun, 14 Dec 2008) | 1 line
don't retain parenthesis if there is only one item left
........
r67768 | benjamin.peterson | 2008-12-14 14:32:30 -0600 (Sun, 14 Dec 2008) | 1 line
use insert_child()
........
r67769 | benjamin.peterson | 2008-12-14 14:59:10 -0600 (Sun, 14 Dec 2008) | 1 line
parenthesize doesn't belong in pygram or FixerBase
........
r67770 | alexandre.vassalotti | 2008-12-14 15:15:36 -0600 (Sun, 14 Dec 2008) | 2 lines
Fix typo: html.paser -> html.parser.
........
r67771 | benjamin.peterson | 2008-12-14 15:22:09 -0600 (Sun, 14 Dec 2008) | 1 line
altering .children needs to call changed()
........
r67774 | benjamin.peterson | 2008-12-14 15:55:38 -0600 (Sun, 14 Dec 2008) | 1 line
employ an evil hack to fix multiple names in the same import statement
........
r67776 | benjamin.peterson | 2008-12-14 16:22:38 -0600 (Sun, 14 Dec 2008) | 1 line
make a common mixin class for Test_imports and friends
........
r67778 | alexandre.vassalotti | 2008-12-14 17:48:20 -0600 (Sun, 14 Dec 2008) | 2 lines
Make fix_imports refactor multiple imports as.
........
2008-12-15 23:35:28 -04:00
|
|
|
self.traverse_by(self.pre_order_heads, tree.pre_order())
|
|
|
|
self.traverse_by(self.post_order_heads, tree.post_order())
|
2008-03-19 01:43:46 -03:00
|
|
|
|
Merged revisions 67900-67901,67919,67928,67984,67991-67993,68106-68108,68110 via svnmerge from
svn+ssh://pythondev@svn.python.org/sandbox/trunk/2to3/lib2to3
........
r67900 | benjamin.peterson | 2008-12-22 14:02:45 -0600 (Mon, 22 Dec 2008) | 4 lines
fix_execfile: wrap the open(fn).read() call in compile(), so the filename is preserved
also add unittests for the fixer
........
r67901 | benjamin.peterson | 2008-12-22 14:09:55 -0600 (Mon, 22 Dec 2008) | 1 line
remove unused import
........
r67919 | benjamin.peterson | 2008-12-23 13:12:22 -0600 (Tue, 23 Dec 2008) | 1 line
copy permission bits from the backup to the original
........
r67928 | benjamin.peterson | 2008-12-26 20:49:30 -0600 (Fri, 26 Dec 2008) | 1 line
don't be so idiot about multiple local imports in fix_import; still won't handle absolute and local imports on the same line
........
r67984 | benjamin.peterson | 2008-12-28 09:55:16 -0600 (Sun, 28 Dec 2008) | 1 line
don't need loop
........
r67991 | benjamin.peterson | 2008-12-28 14:30:26 -0600 (Sun, 28 Dec 2008) | 1 line
actually call finish_tree()
........
r67992 | benjamin.peterson | 2008-12-28 14:34:47 -0600 (Sun, 28 Dec 2008) | 1 line
remove useless test
........
r67993 | benjamin.peterson | 2008-12-28 15:04:32 -0600 (Sun, 28 Dec 2008) | 1 line
update pyk3's test grammar
........
r68106 | benjamin.peterson | 2008-12-31 11:53:58 -0600 (Wed, 31 Dec 2008) | 1 line
#2734 don't convert every instance of long (eg if it's an attribute)
........
r68107 | benjamin.peterson | 2008-12-31 11:55:10 -0600 (Wed, 31 Dec 2008) | 1 line
add another test
........
r68108 | benjamin.peterson | 2008-12-31 12:00:12 -0600 (Wed, 31 Dec 2008) | 1 line
don't change long even if it's the only argument name
........
r68110 | benjamin.peterson | 2008-12-31 14:13:26 -0600 (Wed, 31 Dec 2008) | 1 line
remove unused import
........
2009-01-03 12:34:02 -04:00
|
|
|
for fixer in chain(self.pre_order, self.post_order):
|
2008-03-19 01:43:46 -03:00
|
|
|
fixer.finish_tree(tree, name)
|
|
|
|
return tree.was_changed
|
|
|
|
|
|
|
|
def traverse_by(self, fixers, traversal):
|
|
|
|
"""Traverse an AST, applying a set of fixers to each node.
|
2008-03-19 02:22:42 -03:00
|
|
|
|
2008-03-19 01:43:46 -03:00
|
|
|
This is a helper method for refactor_tree().
|
2008-03-19 02:22:42 -03:00
|
|
|
|
2008-03-19 01:43:46 -03:00
|
|
|
Args:
|
|
|
|
fixers: a list of fixer instances.
|
|
|
|
traversal: a generator that yields AST nodes.
|
2008-03-19 02:22:42 -03:00
|
|
|
|
2008-03-19 01:43:46 -03:00
|
|
|
Returns:
|
|
|
|
None
|
|
|
|
"""
|
|
|
|
if not fixers:
|
|
|
|
return
|
|
|
|
for node in traversal:
|
2008-05-02 18:30:20 -03:00
|
|
|
for fixer in fixers[node.type] + fixers[None]:
|
2008-03-19 01:43:46 -03:00
|
|
|
results = fixer.match(node)
|
|
|
|
if results:
|
|
|
|
new = fixer.transform(node, results)
|
|
|
|
if new is not None and (new != node or
|
|
|
|
str(new) != str(node)):
|
|
|
|
node.replace(new)
|
|
|
|
node = new
|
|
|
|
|
2008-09-02 21:21:32 -03:00
|
|
|
def processed_file(self, new_text, filename, old_text=None, write=False):
|
|
|
|
"""
|
|
|
|
Called when a file has been refactored, and there are changes.
|
2008-03-19 01:43:46 -03:00
|
|
|
"""
|
|
|
|
self.files.append(filename)
|
|
|
|
if old_text is None:
|
|
|
|
try:
|
|
|
|
f = open(filename, "r")
|
|
|
|
except IOError, err:
|
|
|
|
self.log_error("Can't read %s: %s", filename, err)
|
|
|
|
return
|
|
|
|
try:
|
|
|
|
old_text = f.read()
|
|
|
|
finally:
|
|
|
|
f.close()
|
|
|
|
if old_text == new_text:
|
2008-09-02 21:21:32 -03:00
|
|
|
self.log_debug("No changes to %s", filename)
|
2008-03-19 01:43:46 -03:00
|
|
|
return
|
Merged revisions 66511,66548-66549,66644,66646-66652 via svnmerge from
svn+ssh://pythondev@svn.python.org/sandbox/trunk/2to3/lib2to3
........
r66511 | benjamin.peterson | 2008-09-18 21:49:27 -0500 (Thu, 18 Sep 2008) | 1 line
remove a useless if __name__ == '__main__'
........
r66548 | benjamin.peterson | 2008-09-21 21:14:14 -0500 (Sun, 21 Sep 2008) | 1 line
avoid the perils of mutable default arguments
........
r66549 | benjamin.peterson | 2008-09-21 21:26:11 -0500 (Sun, 21 Sep 2008) | 1 line
some places in RefactoringTool should raise an error instead of logging it
........
r66644 | benjamin.peterson | 2008-09-27 10:45:10 -0500 (Sat, 27 Sep 2008) | 1 line
fix doctest refactoring
........
r66646 | benjamin.peterson | 2008-09-27 11:40:13 -0500 (Sat, 27 Sep 2008) | 1 line
don't print to stdout when 2to3 is used as a library
........
r66647 | benjamin.peterson | 2008-09-27 12:28:28 -0500 (Sat, 27 Sep 2008) | 1 line
let fixer modules and classes have different prefixes
........
r66648 | benjamin.peterson | 2008-09-27 14:02:13 -0500 (Sat, 27 Sep 2008) | 1 line
raise errors when 2to3 is used as a library
........
r66649 | benjamin.peterson | 2008-09-27 14:03:38 -0500 (Sat, 27 Sep 2008) | 1 line
fix docstring
........
r66650 | benjamin.peterson | 2008-09-27 14:22:21 -0500 (Sat, 27 Sep 2008) | 1 line
make use of enumerate
........
r66651 | benjamin.peterson | 2008-09-27 14:24:13 -0500 (Sat, 27 Sep 2008) | 1 line
revert last revision; it breaks things
........
r66652 | benjamin.peterson | 2008-09-27 16:03:06 -0500 (Sat, 27 Sep 2008) | 1 line
add tests for lib2to3.refactor
........
2008-09-27 18:09:10 -03:00
|
|
|
self.print_output(diff_texts(old_text, new_text, filename))
|
2008-09-02 21:21:32 -03:00
|
|
|
if write:
|
2008-09-02 23:14:03 -03:00
|
|
|
self.write_file(new_text, filename, old_text)
|
Merged revisions 66511,66548-66549,66644,66646-66652 via svnmerge from
svn+ssh://pythondev@svn.python.org/sandbox/trunk/2to3/lib2to3
........
r66511 | benjamin.peterson | 2008-09-18 21:49:27 -0500 (Thu, 18 Sep 2008) | 1 line
remove a useless if __name__ == '__main__'
........
r66548 | benjamin.peterson | 2008-09-21 21:14:14 -0500 (Sun, 21 Sep 2008) | 1 line
avoid the perils of mutable default arguments
........
r66549 | benjamin.peterson | 2008-09-21 21:26:11 -0500 (Sun, 21 Sep 2008) | 1 line
some places in RefactoringTool should raise an error instead of logging it
........
r66644 | benjamin.peterson | 2008-09-27 10:45:10 -0500 (Sat, 27 Sep 2008) | 1 line
fix doctest refactoring
........
r66646 | benjamin.peterson | 2008-09-27 11:40:13 -0500 (Sat, 27 Sep 2008) | 1 line
don't print to stdout when 2to3 is used as a library
........
r66647 | benjamin.peterson | 2008-09-27 12:28:28 -0500 (Sat, 27 Sep 2008) | 1 line
let fixer modules and classes have different prefixes
........
r66648 | benjamin.peterson | 2008-09-27 14:02:13 -0500 (Sat, 27 Sep 2008) | 1 line
raise errors when 2to3 is used as a library
........
r66649 | benjamin.peterson | 2008-09-27 14:03:38 -0500 (Sat, 27 Sep 2008) | 1 line
fix docstring
........
r66650 | benjamin.peterson | 2008-09-27 14:22:21 -0500 (Sat, 27 Sep 2008) | 1 line
make use of enumerate
........
r66651 | benjamin.peterson | 2008-09-27 14:24:13 -0500 (Sat, 27 Sep 2008) | 1 line
revert last revision; it breaks things
........
r66652 | benjamin.peterson | 2008-09-27 16:03:06 -0500 (Sat, 27 Sep 2008) | 1 line
add tests for lib2to3.refactor
........
2008-09-27 18:09:10 -03:00
|
|
|
else:
|
|
|
|
self.log_debug("Not writing changes to %s", filename)
|
2008-09-02 21:21:32 -03:00
|
|
|
|
Merged revisions 66805,66841,66860,66884-66886,66893,66907,66910 via svnmerge from
svn+ssh://pythondev@svn.python.org/sandbox/trunk/2to3/lib2to3
........
r66805 | benjamin.peterson | 2008-10-04 20:11:02 -0500 (Sat, 04 Oct 2008) | 1 line
mention what the fixes directory is for
........
r66841 | benjamin.peterson | 2008-10-07 17:48:12 -0500 (Tue, 07 Oct 2008) | 1 line
use assertFalse and assertTrue
........
r66860 | benjamin.peterson | 2008-10-08 16:05:07 -0500 (Wed, 08 Oct 2008) | 1 line
instead of abusing the pattern matcher, use start_tree to find a next binding
........
r66884 | benjamin.peterson | 2008-10-13 15:50:30 -0500 (Mon, 13 Oct 2008) | 1 line
don't print tokens to stdout when -v is given
........
r66885 | benjamin.peterson | 2008-10-13 16:28:57 -0500 (Mon, 13 Oct 2008) | 1 line
add the -x option to disable fixers
........
r66886 | benjamin.peterson | 2008-10-13 16:33:53 -0500 (Mon, 13 Oct 2008) | 1 line
cut down on some crud
........
r66893 | benjamin.peterson | 2008-10-14 17:16:54 -0500 (Tue, 14 Oct 2008) | 1 line
add an optional set literal fixer
........
r66907 | benjamin.peterson | 2008-10-15 16:59:41 -0500 (Wed, 15 Oct 2008) | 1 line
don't write backup files by default
........
r66910 | benjamin.peterson | 2008-10-15 17:43:10 -0500 (Wed, 15 Oct 2008) | 1 line
add the -n option; it stops backupfiles from being written
........
2008-10-15 20:10:28 -03:00
|
|
|
def write_file(self, new_text, filename, old_text):
|
2008-09-02 21:21:32 -03:00
|
|
|
"""Writes a string to a file.
|
|
|
|
|
|
|
|
It first shows a unified diff between the old text and the new text, and
|
|
|
|
then rewrites the file; the latter is only done if the write option is
|
|
|
|
set.
|
|
|
|
"""
|
2008-03-19 01:43:46 -03:00
|
|
|
try:
|
|
|
|
f = open(filename, "w")
|
|
|
|
except os.error, err:
|
|
|
|
self.log_error("Can't create %s: %s", filename, err)
|
|
|
|
return
|
|
|
|
try:
|
Merged revisions 66985,67170,67173,67177-67179 via svnmerge from
svn+ssh://pythondev@svn.python.org/sandbox/trunk/2to3/lib2to3
........
r66985 | benjamin.peterson | 2008-10-20 16:43:46 -0500 (Mon, 20 Oct 2008) | 1 line
no need to use nested try, except, finally
........
r67170 | benjamin.peterson | 2008-11-08 12:28:31 -0600 (Sat, 08 Nov 2008) | 1 line
fix #4271: fix_imports didn't recognize imports with parenthesis (ie from x import (a, b))
........
r67173 | benjamin.peterson | 2008-11-08 17:42:08 -0600 (Sat, 08 Nov 2008) | 1 line
consolidate test
........
r67177 | benjamin.peterson | 2008-11-09 21:52:52 -0600 (Sun, 09 Nov 2008) | 1 line
let the metclass fixer handle complex assignments in the class body gracefully
........
r67178 | benjamin.peterson | 2008-11-10 15:26:43 -0600 (Mon, 10 Nov 2008) | 1 line
the metaclass fixers shouldn't die when bases are not a simple name
........
r67179 | benjamin.peterson | 2008-11-10 15:29:58 -0600 (Mon, 10 Nov 2008) | 1 line
allow the fix_import pattern to catch from imports with parenthesis
........
2008-11-10 18:11:12 -04:00
|
|
|
f.write(new_text)
|
|
|
|
except os.error, err:
|
|
|
|
self.log_error("Can't write %s: %s", filename, err)
|
2008-03-19 01:43:46 -03:00
|
|
|
finally:
|
|
|
|
f.close()
|
2008-09-02 21:21:32 -03:00
|
|
|
self.log_debug("Wrote changes to %s", filename)
|
|
|
|
self.wrote = True
|
2008-03-19 01:43:46 -03:00
|
|
|
|
|
|
|
PS1 = ">>> "
|
|
|
|
PS2 = "... "
|
|
|
|
|
|
|
|
def refactor_docstring(self, input, filename):
|
|
|
|
"""Refactors a docstring, looking for doctests.
|
|
|
|
|
|
|
|
This returns a modified version of the input string. It looks
|
|
|
|
for doctests, which start with a ">>>" prompt, and may be
|
|
|
|
continued with "..." prompts, as long as the "..." is indented
|
|
|
|
the same as the ">>>".
|
|
|
|
|
|
|
|
(Unfortunately we can't use the doctest module's parser,
|
|
|
|
since, like most parsers, it is not geared towards preserving
|
|
|
|
the original source.)
|
|
|
|
"""
|
|
|
|
result = []
|
|
|
|
block = None
|
|
|
|
block_lineno = None
|
|
|
|
indent = None
|
|
|
|
lineno = 0
|
|
|
|
for line in input.splitlines(True):
|
|
|
|
lineno += 1
|
|
|
|
if line.lstrip().startswith(self.PS1):
|
|
|
|
if block is not None:
|
|
|
|
result.extend(self.refactor_doctest(block, block_lineno,
|
|
|
|
indent, filename))
|
|
|
|
block_lineno = lineno
|
|
|
|
block = [line]
|
|
|
|
i = line.find(self.PS1)
|
|
|
|
indent = line[:i]
|
|
|
|
elif (indent is not None and
|
|
|
|
(line.startswith(indent + self.PS2) or
|
|
|
|
line == indent + self.PS2.rstrip() + "\n")):
|
|
|
|
block.append(line)
|
|
|
|
else:
|
|
|
|
if block is not None:
|
|
|
|
result.extend(self.refactor_doctest(block, block_lineno,
|
|
|
|
indent, filename))
|
|
|
|
block = None
|
|
|
|
indent = None
|
|
|
|
result.append(line)
|
|
|
|
if block is not None:
|
|
|
|
result.extend(self.refactor_doctest(block, block_lineno,
|
|
|
|
indent, filename))
|
|
|
|
return "".join(result)
|
|
|
|
|
|
|
|
def refactor_doctest(self, block, lineno, indent, filename):
|
|
|
|
"""Refactors one doctest.
|
|
|
|
|
|
|
|
A doctest is given as a block of lines, the first of which starts
|
|
|
|
with ">>>" (possibly indented), while the remaining lines start
|
|
|
|
with "..." (identically indented).
|
|
|
|
|
|
|
|
"""
|
|
|
|
try:
|
|
|
|
tree = self.parse_block(block, lineno, indent)
|
|
|
|
except Exception, err:
|
2008-09-02 21:21:32 -03:00
|
|
|
if self.log.isEnabledFor(logging.DEBUG):
|
2008-03-19 01:43:46 -03:00
|
|
|
for line in block:
|
2008-09-02 21:21:32 -03:00
|
|
|
self.log_debug("Source: %s", line.rstrip("\n"))
|
2008-03-19 01:43:46 -03:00
|
|
|
self.log_error("Can't parse docstring in %s line %s: %s: %s",
|
|
|
|
filename, lineno, err.__class__.__name__, err)
|
|
|
|
return block
|
|
|
|
if self.refactor_tree(tree, filename):
|
|
|
|
new = str(tree).splitlines(True)
|
|
|
|
# Undo the adjustment of the line numbers in wrap_toks() below.
|
|
|
|
clipped, new = new[:lineno-1], new[lineno-1:]
|
|
|
|
assert clipped == ["\n"] * (lineno-1), clipped
|
|
|
|
if not new[-1].endswith("\n"):
|
|
|
|
new[-1] += "\n"
|
|
|
|
block = [indent + self.PS1 + new.pop(0)]
|
|
|
|
if new:
|
|
|
|
block += [indent + self.PS2 + line for line in new]
|
|
|
|
return block
|
|
|
|
|
|
|
|
def summarize(self):
|
2008-09-02 21:21:32 -03:00
|
|
|
if self.wrote:
|
2008-03-19 01:43:46 -03:00
|
|
|
were = "were"
|
|
|
|
else:
|
|
|
|
were = "need to be"
|
|
|
|
if not self.files:
|
|
|
|
self.log_message("No files %s modified.", were)
|
|
|
|
else:
|
|
|
|
self.log_message("Files that %s modified:", were)
|
|
|
|
for file in self.files:
|
|
|
|
self.log_message(file)
|
|
|
|
if self.fixer_log:
|
|
|
|
self.log_message("Warnings/messages while refactoring:")
|
|
|
|
for message in self.fixer_log:
|
|
|
|
self.log_message(message)
|
|
|
|
if self.errors:
|
|
|
|
if len(self.errors) == 1:
|
|
|
|
self.log_message("There was 1 error:")
|
|
|
|
else:
|
|
|
|
self.log_message("There were %d errors:", len(self.errors))
|
|
|
|
for msg, args, kwds in self.errors:
|
|
|
|
self.log_message(msg, *args, **kwds)
|
|
|
|
|
|
|
|
def parse_block(self, block, lineno, indent):
|
|
|
|
"""Parses a block into a tree.
|
|
|
|
|
|
|
|
This is necessary to get correct line number / offset information
|
|
|
|
in the parser diagnostics and embedded into the parse tree.
|
|
|
|
"""
|
|
|
|
return self.driver.parse_tokens(self.wrap_toks(block, lineno, indent))
|
|
|
|
|
|
|
|
def wrap_toks(self, block, lineno, indent):
|
|
|
|
"""Wraps a tokenize stream to systematically modify start/end."""
|
|
|
|
tokens = tokenize.generate_tokens(self.gen_lines(block, indent).next)
|
|
|
|
for type, value, (line0, col0), (line1, col1), line_text in tokens:
|
|
|
|
line0 += lineno - 1
|
|
|
|
line1 += lineno - 1
|
|
|
|
# Don't bother updating the columns; this is too complicated
|
|
|
|
# since line_text would also have to be updated and it would
|
|
|
|
# still break for tokens spanning lines. Let the user guess
|
|
|
|
# that the column numbers for doctests are relative to the
|
|
|
|
# end of the prompt string (PS1 or PS2).
|
|
|
|
yield type, value, (line0, col0), (line1, col1), line_text
|
|
|
|
|
|
|
|
|
|
|
|
def gen_lines(self, block, indent):
|
|
|
|
"""Generates lines as expected by tokenize from a list of lines.
|
|
|
|
|
|
|
|
This strips the first len(indent + self.PS1) characters off each line.
|
|
|
|
"""
|
|
|
|
prefix1 = indent + self.PS1
|
|
|
|
prefix2 = indent + self.PS2
|
|
|
|
prefix = prefix1
|
|
|
|
for line in block:
|
|
|
|
if line.startswith(prefix):
|
|
|
|
yield line[len(prefix):]
|
|
|
|
elif line == prefix.rstrip() + "\n":
|
|
|
|
yield "\n"
|
|
|
|
else:
|
|
|
|
raise AssertionError("line=%r, prefix=%r" % (line, prefix))
|
|
|
|
prefix = prefix2
|
|
|
|
while True:
|
|
|
|
yield ""
|
|
|
|
|
|
|
|
|
|
|
|
def diff_texts(a, b, filename):
|
Merged revisions 66511,66548-66549,66644,66646-66652 via svnmerge from
svn+ssh://pythondev@svn.python.org/sandbox/trunk/2to3/lib2to3
........
r66511 | benjamin.peterson | 2008-09-18 21:49:27 -0500 (Thu, 18 Sep 2008) | 1 line
remove a useless if __name__ == '__main__'
........
r66548 | benjamin.peterson | 2008-09-21 21:14:14 -0500 (Sun, 21 Sep 2008) | 1 line
avoid the perils of mutable default arguments
........
r66549 | benjamin.peterson | 2008-09-21 21:26:11 -0500 (Sun, 21 Sep 2008) | 1 line
some places in RefactoringTool should raise an error instead of logging it
........
r66644 | benjamin.peterson | 2008-09-27 10:45:10 -0500 (Sat, 27 Sep 2008) | 1 line
fix doctest refactoring
........
r66646 | benjamin.peterson | 2008-09-27 11:40:13 -0500 (Sat, 27 Sep 2008) | 1 line
don't print to stdout when 2to3 is used as a library
........
r66647 | benjamin.peterson | 2008-09-27 12:28:28 -0500 (Sat, 27 Sep 2008) | 1 line
let fixer modules and classes have different prefixes
........
r66648 | benjamin.peterson | 2008-09-27 14:02:13 -0500 (Sat, 27 Sep 2008) | 1 line
raise errors when 2to3 is used as a library
........
r66649 | benjamin.peterson | 2008-09-27 14:03:38 -0500 (Sat, 27 Sep 2008) | 1 line
fix docstring
........
r66650 | benjamin.peterson | 2008-09-27 14:22:21 -0500 (Sat, 27 Sep 2008) | 1 line
make use of enumerate
........
r66651 | benjamin.peterson | 2008-09-27 14:24:13 -0500 (Sat, 27 Sep 2008) | 1 line
revert last revision; it breaks things
........
r66652 | benjamin.peterson | 2008-09-27 16:03:06 -0500 (Sat, 27 Sep 2008) | 1 line
add tests for lib2to3.refactor
........
2008-09-27 18:09:10 -03:00
|
|
|
"""Return a unified diff of two strings."""
|
2008-03-19 01:43:46 -03:00
|
|
|
a = a.splitlines()
|
|
|
|
b = b.splitlines()
|
Merged revisions 66511,66548-66549,66644,66646-66652 via svnmerge from
svn+ssh://pythondev@svn.python.org/sandbox/trunk/2to3/lib2to3
........
r66511 | benjamin.peterson | 2008-09-18 21:49:27 -0500 (Thu, 18 Sep 2008) | 1 line
remove a useless if __name__ == '__main__'
........
r66548 | benjamin.peterson | 2008-09-21 21:14:14 -0500 (Sun, 21 Sep 2008) | 1 line
avoid the perils of mutable default arguments
........
r66549 | benjamin.peterson | 2008-09-21 21:26:11 -0500 (Sun, 21 Sep 2008) | 1 line
some places in RefactoringTool should raise an error instead of logging it
........
r66644 | benjamin.peterson | 2008-09-27 10:45:10 -0500 (Sat, 27 Sep 2008) | 1 line
fix doctest refactoring
........
r66646 | benjamin.peterson | 2008-09-27 11:40:13 -0500 (Sat, 27 Sep 2008) | 1 line
don't print to stdout when 2to3 is used as a library
........
r66647 | benjamin.peterson | 2008-09-27 12:28:28 -0500 (Sat, 27 Sep 2008) | 1 line
let fixer modules and classes have different prefixes
........
r66648 | benjamin.peterson | 2008-09-27 14:02:13 -0500 (Sat, 27 Sep 2008) | 1 line
raise errors when 2to3 is used as a library
........
r66649 | benjamin.peterson | 2008-09-27 14:03:38 -0500 (Sat, 27 Sep 2008) | 1 line
fix docstring
........
r66650 | benjamin.peterson | 2008-09-27 14:22:21 -0500 (Sat, 27 Sep 2008) | 1 line
make use of enumerate
........
r66651 | benjamin.peterson | 2008-09-27 14:24:13 -0500 (Sat, 27 Sep 2008) | 1 line
revert last revision; it breaks things
........
r66652 | benjamin.peterson | 2008-09-27 16:03:06 -0500 (Sat, 27 Sep 2008) | 1 line
add tests for lib2to3.refactor
........
2008-09-27 18:09:10 -03:00
|
|
|
return difflib.unified_diff(a, b, filename, filename,
|
|
|
|
"(original)", "(refactored)",
|
|
|
|
lineterm="")
|