eqfix.py 5.29 KB
Newer Older
1
#! /usr/bin/env python
Guido van Rossum's avatar
Guido van Rossum committed
2

3
# Fix Python source files to use the new equality test operator, i.e.,
Guido van Rossum's avatar
Guido van Rossum committed
4
#	if x = y: ...
5 6 7 8
# is changed to
#	if x == y: ...
# The script correctly tokenizes the Python program to reliably
# distinguish between assignments and equality tests.
Guido van Rossum's avatar
Guido van Rossum committed
9 10 11 12 13 14 15
#
# Command line arguments are files or directories to be processed.
# Directories are searched recursively for files whose name looks
# like a python module.
# Symbolic links are always ignored (except as explicit directory
# arguments).  Of course, the original file is kept as a back-up
# (with a "~" attached to its name).
16 17 18
# It complains about binaries (files containing null bytes)
# and about files that are ostensibly not Python files: if the first
# line starts with '#!' and does not contain the string 'python'.
Guido van Rossum's avatar
Guido van Rossum committed
19 20 21 22 23 24 25 26 27 28 29 30 31 32
#
# Changes made are reported to stdout in a diff-like format.
#
# Undoubtedly you can do this using find and sed or perl, but this is
# a nice example of Python code that recurses down a directory tree
# and uses regular expressions.  Also note several subtleties like
# preserving the file's mode and avoiding to even write a temp file
# when no changes are needed for a file.
#
# NB: by changing only the function fixline() you can turn this
# into a program for a different change to Python programs...

import sys
import regex
33
import os
Guido van Rossum's avatar
Guido van Rossum committed
34
from stat import *
35
import string
Guido van Rossum's avatar
Guido van Rossum committed
36 37 38 39 40 41 42 43

err = sys.stderr.write
dbg = err
rep = sys.stdout.write

def main():
	bad = 0
	if not sys.argv[1:]: # No arguments
Guido van Rossum's avatar
Guido van Rossum committed
44
		err('usage: ' + sys.argv[0] + ' file-or-directory ...\n')
Guido van Rossum's avatar
Guido van Rossum committed
45 46
		sys.exit(2)
	for arg in sys.argv[1:]:
47
		if os.path.isdir(arg):
Guido van Rossum's avatar
Guido van Rossum committed
48
			if recursedown(arg): bad = 1
49
		elif os.path.islink(arg):
Guido van Rossum's avatar
Guido van Rossum committed
50 51 52 53 54 55 56 57 58 59 60 61 62 63
			err(arg + ': will not process symbolic links\n')
			bad = 1
		else:
			if fix(arg): bad = 1
	sys.exit(bad)

ispythonprog = regex.compile('^[a-zA-Z0-9_]+\.py$')
def ispython(name):
	return ispythonprog.match(name) >= 0

def recursedown(dirname):
	dbg('recursedown(' + `dirname` + ')\n')
	bad = 0
	try:
64 65
		names = os.listdir(dirname)
	except os.error, msg:
Guido van Rossum's avatar
Guido van Rossum committed
66 67 68 69 70
		err(dirname + ': cannot list directory: ' + `msg` + '\n')
		return 1
	names.sort()
	subdirs = []
	for name in names:
71 72 73 74
		if name in (os.curdir, os.pardir): continue
		fullname = os.path.join(dirname, name)
		if os.path.islink(fullname): pass
		elif os.path.isdir(fullname):
Guido van Rossum's avatar
Guido van Rossum committed
75 76 77 78 79 80 81 82
			subdirs.append(fullname)
		elif ispython(name):
			if fix(fullname): bad = 1
	for fullname in subdirs:
		if recursedown(fullname): bad = 1
	return bad

def fix(filename):
83
##	dbg('fix(' + `filename` + ')\n')
Guido van Rossum's avatar
Guido van Rossum committed
84 85 86 87 88
	try:
		f = open(filename, 'r')
	except IOError, msg:
		err(filename + ': cannot open: ' + `msg` + '\n')
		return 1
89 90
	head, tail = os.path.split(filename)
	tempname = os.path.join(head, '@' + tail)
Guido van Rossum's avatar
Guido van Rossum committed
91 92 93 94 95 96 97 98
	g = None
	# If we find a match, we rewind the file and start over but
	# now copy everything to a temp file.
	lineno = 0
	while 1:
		line = f.readline()
		if not line: break
		lineno = lineno + 1
99 100 101 102 103 104 105 106 107 108 109 110 111 112
		if g is None and '\0' in line:
			# Check for binary files
			err(filename + ': contains null bytes; not fixed\n')
			f.close()
			return 1
		if lineno == 1 and g is None and line[:2] == '#!':
			# Check for non-Python scripts
			words = string.split(line[2:])
			if words and regex.search('[pP]ython', words[0]) < 0:
				msg = filename + ': ' + words[0]
				msg = msg + ' script; not fixed\n'
				err(msg)
				f.close()
				return 1
Guido van Rossum's avatar
Guido van Rossum committed
113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 145
		while line[-2:] == '\\\n':
			nextline = f.readline()
			if not nextline: break
			line = line + nextline
			lineno = lineno + 1
		newline = fixline(line)
		if newline != line:
			if g is None:
				try:
					g = open(tempname, 'w')
				except IOError, msg:
					f.close()
					err(tempname+': cannot create: '+\
					    `msg`+'\n')
					return 1
				f.seek(0)
				lineno = 0
				rep(filename + ':\n')
				continue # restart from the beginning
			rep(`lineno` + '\n')
			rep('< ' + line)
			rep('> ' + newline)
		if g is not None:
			g.write(newline)

	# End of file
	f.close()
	if not g: return 0 # No changes
	
	# Finishing touch -- move files

	# First copy the file's mode to the temp file
	try:
146 147 148
		statbuf = os.stat(filename)
		os.chmod(tempname, statbuf[ST_MODE] & 07777)
	except os.error, msg:
Guido van Rossum's avatar
Guido van Rossum committed
149 150 151
		err(tempname + ': warning: chmod failed (' + `msg` + ')\n')
	# Then make a backup of the original file as filename~
	try:
152 153
		os.rename(filename, filename + '~')
	except os.error, msg:
Guido van Rossum's avatar
Guido van Rossum committed
154 155 156
		err(filename + ': warning: backup failed (' + `msg` + ')\n')
	# Now move the temp file to the original file
	try:
157 158
		os.rename(tempname, filename)
	except os.error, msg:
Guido van Rossum's avatar
Guido van Rossum committed
159 160 161 162 163
		err(filename + ': rename failed (' + `msg` + ')\n')
		return 1
	# Return succes
	return 0

164 165 166 167 168

from tokenize import tokenprog

match = {'if':':', 'elif':':', 'while':':', 'return':'\n', \
	 '(':')', '[':']', '{':'}', '`':'`'}
Guido van Rossum's avatar
Guido van Rossum committed
169 170

def fixline(line):
171 172 173 174 175 176 177 178 179 180 181 182 183 184 185 186 187 188 189 190 191 192 193 194 195
	# Quick check for easy case
	if '=' not in line: return line
	
	i, n = 0, len(line)
	stack = []
	while i < n:
		j = tokenprog.match(line, i)
		if j < 0:
			# A bad token; forget about the rest of this line
			print '(Syntax error:)'
			print line,
			return line
		a, b = tokenprog.regs[3] # Location of the token proper
		token = line[a:b]
		i = i+j
		if stack and token == stack[-1]:
			del stack[-1]
		elif match.has_key(token):
			stack.append(match[token])
		elif token == '=' and stack:
			line = line[:a] + '==' + line[b:]
			i, n = a + len('=='), len(line)
		elif token == '==' and not stack:
			print '(Warning: \'==\' at top level:)'
			print line,
Guido van Rossum's avatar
Guido van Rossum committed
196 197
	return line

198

Guido van Rossum's avatar
Guido van Rossum committed
199
main()