splitfields(s, '') is illegal

diff --git a/Lib/string.py b/Lib/string.py
index 94e91570c5c70915556a6cb69c46402011b766e0..f358ac4778833052bd725b533adaa37b969925c4 100644 (file)
--- a/Lib/string.py
+++ b/Lib/string.py
@@ -63,10 +63,14 @@ def split(s):
 
 # Split a list into fields separated by a given string
 # NB: splitfields(s, ' ') is NOT the same as split(s)!
+# splitfields(s, '') is illegal
+splitfields_error = 'string.splitfields called with empty separator'
 def splitfields(s, sep):
        res = []
-       ns = len(s)
        nsep = len(sep)
+       if nsep == 0:
+               raise splitfields_error
+       ns = len(s)
        i = j = 0
        while j+nsep <= ns:
                if s[j:j+nsep] == sep:

diff --git a/Lib/stringold.py b/Lib/stringold.py
index 94e91570c5c70915556a6cb69c46402011b766e0..f358ac4778833052bd725b533adaa37b969925c4 100644 (file)
--- a/Lib/stringold.py
+++ b/Lib/stringold.py
@@ -63,10 +63,14 @@ def split(s):
 
 # Split a list into fields separated by a given string
 # NB: splitfields(s, ' ') is NOT the same as split(s)!
+# splitfields(s, '') is illegal
+splitfields_error = 'string.splitfields called with empty separator'
 def splitfields(s, sep):
        res = []
-       ns = len(s)
        nsep = len(sep)
+       if nsep == 0:
+               raise splitfields_error
+       ns = len(s)
        i = j = 0
        while j+nsep <= ns:
                if s[j:j+nsep] == sep: