Apply const to generated parser code
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
Línea de trabajo
Comienza inspeccionando las declaraciones de contexto del parser generado del target de C++ y cómo el código AST navega por ellas. Aclara si el recorrido calificado con const, noexcept y la limpieza de virtual/override están todos dentro del alcance; se considera terminado cuando la salida generada admite los calificadores acordados y compila sin errores ni advertencias relacionados.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Hi,
I'm using references to the generated parser contexts in my AST implementation. I find that I have a tendency to want to return const pointers from getter functions.
class AstExpression
{
public:
const MyParser::ExpressionContext* getContext() const;
};
However calling any function on that return value to further navigate the parse-tree complains about the const because these functions are not declared as const (not changing the instance).
Would it be possible to generate const aware code?
This would make it more correct (and strict).
Another one to consider is noexcept.
Also seen virtual and override being used together. MSVC 'complains' that only one is needed. Not sure if other compilers feel the same.
- Lenguaje dominante
- C++
- Estrellas
- 80
- Forks
- 33
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la 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.
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 2/5 1-3 horas Aptitud para principiantes 35/100
antlr/antlr4-cpp#10 ·
Todos los issues de antlr/antlr4-cpp
Issues similares
-
Broken links in the docsAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
microsoft/onnxruntime#33018 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
DataLakeFileSystemClient::ListPaths() throws JSON exception due to accessing undefined fieldsAbiertocustomer-reported needs-triage question
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Azure/azure-sdk-for-cpp#7435 ·
Los mantenedores suelen responder en 1 día
-
ChromieCraft Generic Confirmed World Event
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
azerothcore/azerothcore-wotlk#27882 ·
Los mantenedores suelen responder en 1 día
-
MacOS build failureAbiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
aristocratos/btop#1874 ·
Los mantenedores suelen responder en 1 día