[FR] Support RE2 regular expression engine

Abierto
#1,650 0 comentarios 2 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
5/5
Tiempo estimado
Más de una semana
Aptitud para principiantes
35/100
Tipo de issue
Nueva funcionalidad
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
cpp

Línea de trabajo

No se nombran archivos, pruebas ni puntos de entrada. Empieza por localizar el manejo actual de std::regex y la API orientada a los embedders; después determina cómo se podría proporcionar un motor RE2 alternativo. Se considera terminado cuando las expresiones no válidas informan de un error e impiden que se ejecuten los benchmarks.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Is your feature request related to a problem? Please describe.
When given an invalid regular expression, Chromium perftests using this library crash.

This is because Chromium compiles google_benchmark with std::regex and with exceptions disabled, and so there's no way for it to recover from a regular expression parsing error.

Describe the solution you'd like
When given an invalid regular expression, an error should be reported and no benchmarks should be run.

The Chromium project already incorporates the re2 regular expression library, so if it was possible for the embedder to provide the re2 engine then this problem could be solved.

Describe alternatives you've considered
Chromium could use the POSIX regex library, but it wouldn't work on Windows. Chromium could use the POSIX regex library on non-Windows platforms, but then behaviour would differ between platforms, which causes a lot of problems.

Additional context
N/A

Lenguaje dominante
C++
Estrellas
10.5k
Forks
1.8k
Merge medio
1 d 17 h
PR fusionados (30 d)
30

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de google/benchmark

Todos los issues de google/benchmark

Issues similares

Más issues de C++

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.