Changed my mind on replace().

It's now replace(str, old, new, maxsplit=0).
Note new ordering of parameters (string first);
this is more consistent with translate().
This commit is contained in:
Guido van Rossum 1997-04-02 05:49:46 +00:00
parent aa925a5efd
commit 21aa0ef351
2 changed files with 4 additions and 24 deletions

View File

@ -321,18 +321,8 @@ def maketrans(fromstr, tostr):
return joinfields(L, "") return joinfields(L, "")
# Substring replacement (global) # Substring replacement (global)
def replace(old, new, str): def replace(str, old, new, maxsplit=0):
return joinfields(splitfields(str, old), new) return joinfields(splitfields(str, old, maxsplit), new)
# Substring replacement (1st substring only)
def replace1(old, new, str, i=0, last=None):
if last is None:
i = find(str, old, i)
else:
i = find(str, old, i, last)
if i >= 0:
str = str[:i] + new + str[i+len(old):]
return str
# Try importing optional built-in module "strop" -- if it exists, # Try importing optional built-in module "strop" -- if it exists,

View File

@ -321,18 +321,8 @@ def maketrans(fromstr, tostr):
return joinfields(L, "") return joinfields(L, "")
# Substring replacement (global) # Substring replacement (global)
def replace(old, new, str): def replace(str, old, new, maxsplit=0):
return joinfields(splitfields(str, old), new) return joinfields(splitfields(str, old, maxsplit), new)
# Substring replacement (1st substring only)
def replace1(old, new, str, i=0, last=None):
if last is None:
i = find(str, old, i)
else:
i = find(str, old, i, last)
if i >= 0:
str = str[:i] + new + str[i+len(old):]
return str
# Try importing optional built-in module "strop" -- if it exists, # Try importing optional built-in module "strop" -- if it exists,