Skip to content

Commit bb43b68

Browse files
committed
Fix match groups default argument
1 parent 4fc6597 commit bb43b68

File tree

2 files changed

+4
-3
lines changed
  • graalpython

2 files changed

+4
-3
lines changed

graalpython/com.oracle.graal.python.test/src/tests/unittest_tags/test_re.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -67,6 +67,7 @@
6767
*graalpython.lib-python.3.test.test_re.ReTests.test_re_escape_non_ascii_bytes
6868
*graalpython.lib-python.3.test.test_re.ReTests.test_re_findall
6969
*graalpython.lib-python.3.test.test_re.ReTests.test_re_groupref_exists
70+
*graalpython.lib-python.3.test.test_re.ReTests.test_re_match
7071
*graalpython.lib-python.3.test.test_re.ReTests.test_re_split
7172
*graalpython.lib-python.3.test.test_re.ReTests.test_re_subn
7273
*graalpython.lib-python.3.test.test_re.ReTests.test_repeat_minmax_overflow_maxrepeat

graalpython/lib-graalpython/_sre.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -166,7 +166,7 @@ def group(self, *args):
166166
def groups(self, default=None):
167167
lst = []
168168
for arg in range(1, self.compiled_regex.groupCount):
169-
lst.append(self.__group__(arg))
169+
lst.append(self.__group__(arg, default))
170170
return tuple(lst)
171171

172172
def __groupidx__(self, idx):
@@ -175,11 +175,11 @@ def __groupidx__(self, idx):
175175
else:
176176
return idx
177177

178-
def __group__(self, idx):
178+
def __group__(self, idx, default=None):
179179
idxarg = self.__groupidx__(idx)
180180
start = self.result.getStart(idxarg)
181181
if start < 0:
182-
return None
182+
return default
183183
else:
184184
return self.input_str[start:self.result.getEnd(idxarg)]
185185

0 commit comments

Comments
 (0)