Unit Test expecting 57, 123; code review
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 35/100
Línea de trabajo
Comienza con antlr4cpp/antlr/test/test_interval_set.cpp y el método test_mixed_ranges_and_elements(). Compara su cadena de intervalo esperada con la implementación de Java mostrada y ejecuta la prueba unitaria de C++ relevante. Se considera terminado cuando la expectativa de la prueba o el comportamiento del intervalo se hayan confirmado y estén alineados con la semántica prevista de rangos inclusivos.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Hello,
Inspiring on C ++ test for my unit tests in C, I noticed an error in the expecting result. Maybe the +1 has no effect?
In https://github.com/antlr/antlr4-cpp/blob/master/antlr4cpp/antlr/test/test_interval_set.cpp
void test_mixed_ranges_and_elements()
{
interval_set s;
s.insert(1);
s.insert(std::make_pair(L'a',L'z'+1));
s.insert(std::make_pair(L'0',L'9'+1));
std::wstring expecting = L"{1, 48..57, 97..122}";
}
by comparison with Java implementation
IntervalSet interval_set = new IntervalSet();
interval_set.add(1);
interval_set.add('a', 'z'+1);
interval_set.add('0', '9'+1);
String str = interval_set.toString();
String expecting = "{1, 48..58, 97..123}";
PS:
C ++ operator overloading is friendly
interval_set s;
s << 1;// s+1 is ambigus
s << interval('a', 'z'+1);
s &= interval('0', '9'+1);
s |= interval('0', '9'+1);
std::cout << s << std::endl;
- Lenguaje dominante
- C++
- Estrellas
- 80
- Forks
- 33
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
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.
Más de antlr/antlr4-cpp
-
Dificultad 3/5 1-2 días Aptitud para principiantes 30/100
antlr/antlr4-cpp#14 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 20/100
antlr/antlr4-cpp#13 ·
-
About antlrcpp::Any derived.Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
antlr/antlr4-cpp#12 · 1 comentario ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
antlr/antlr4-cpp#11 ·
Todos los issues de antlr/antlr4-cpp
Issues similares
-
Unconfirmed bug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
luanti-org/luanti#17605 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
area: config area: firmware priority: P2 - medium size: S type: bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Mizithra/ActiveTerrain#16 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
grumpycoders/pcsx-redux#2171 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
bytedance/trae-agent#524 · 1 comentario ·
Los mantenedores suelen responder en 1 día