`state_reset()` does not clear `save_marks` after an aborted match
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Python
- Estrellas
- 77.2k
- Forks
- 35.9k
- Métricas de merge de PR
- Métricas de PR pendientes
Descripción
Bug description:
state_reset() in Modules/_sre/sre.c clears the per-match state at the start of each match in a scanner loop, including state->repeat, but not state->save_marks, which is zeroed only in state_init(). A match aborted by a signal returns straight out of SRE(match), so that counter stays above zero for the rest of the iterator and every later match saves and restores marks on backtracking that it would otherwise skip. Only finditer() and pattern.scanner() can observe it, because the findall/sub/split loops abandon the state on error. Results are unchanged, only the cost.
$ cat repro.py
import re, signal, time
class Alarm(Exception): pass
def boom(*args): raise Alarm
signal.signal(signal.SIGALRM, boom)
PAT = r"(z)*" + "(a)" * 200 + r"(?:xy|x)" * 300
SUBJECT = "z" * 400000 + ("a" * 200 + "x" * 300) * 400
def run(interrupt):
it = re.compile(PAT).finditer(SUBJECT)
if interrupt:
signal.setitimer(signal.ITIMER_REAL, 0.02)
try:
next(it)
except Alarm:
pass
signal.setitimer(signal.ITIMER_REAL, 0)
next(it)
t = time.perf_counter()
spans = [m.span() for m in it]
return time.perf_counter() - t, spans
clean, clean_spans = run(False)
dirty, dirty_spans = run(True)
print("clean iterator : %.4fs, %d matches" % (clean, len(clean_spans)))
print("resumed after one abort : %.4fs, %d matches" % (dirty, len(dirty_spans)))
print("same results : %s" % (clean_spans == dirty_spans,))
$ ./python repro.py
clean iterator : 0.0087s, 399 matches
resumed after one abort : 0.0238s, 399 matches
same results : True
Expected: an iterator resumed after an aborted match costs the same as a clean one.
CPython versions tested on:
CPython main branch
Operating systems tested on:
Linux
Linked PRs
- gh-157707
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza en Modules/_sre/sre.c, en state_reset y en las rutas del scanner-loop utilizadas por finditer() y pattern.scanner(). Ejecuta primero la comparación con repro.py y, después, verifica que un match abortado no ralentice la iteración posterior y que los spans reanudados permanezcan sin cambios.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- backend
- Tipo de issue
- Error
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bien especificado
- Aptitud para principiantes
- 30/100