cpplint checks fail for services and actions with buffers in the response/result
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 56/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- c
- Área
- build-system, testing-qa
Línea de trabajo
Comienza con el archivo rosidl_generator_py generado build/.../rosidl_generator_py/srv/_get_buffer_s.c y el fallo build/include_order de cpplint_rosidl_generated_py. Compara la generación de service/action descrita en #263 con la inyección de stdint.h de #250, incluyendo el tratamiento de response/result/feedback frente a request/goal. Se considera terminado cuando las comprobaciones de cpplint del código generado afectado pasan sin introducir fallos en los demás casos.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Generated by Generative AI
No
Operating System:
any
ROS version or commit hash:
rolling, lyrical
RMW implementation (if applicable):
No response
RMW Configuration (if applicable):
No response
Client library (if applicable):
No response
'ros2 doctor --report' output
No response
Steps to reproduce issue
See #263
Expected behavior
Tests pass
Actual behavior
Tests fail
- rosidl_generator_py.cpplint_rosidl_generated_py build/include_order [4] (/ws/build/rosidl_generator_py/rosidl_generator_py/rosidl_generator_py/srv/_get_buffer_s.c:83)
<<< failure message
Found C system header after other header. Should be: _get_buffer_s.h, c system, c++ system, other.
>>>
Additional information
#250 injects <stdint.h> into the headers, which might appear in the generated code after non-C headers, if the code is for services and actions, which have buffer type in their response/result/feedback, but not in the response/goal.
There are two simple fixes:
- silence build/include_order in the cpplint checks
- always add <stdint.h>
A more complicated fix would add stdint.h as in the request/goal part, if the parts use buffers.
- Lenguaje dominante
- EmberScript
- Estrellas
- 26
- Forks
- 68
- Merge medio
- 1 d 9 h
- PR fusionados (30 d)
- 3
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 ros2/rosidl_python
-
Eliminate array resetting loop in _msg_support.c.emPosiblemente ocupada @Lidang-Jiang la tomó hace 167 días. Abiertoenhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
ros2/rosidl_python#255 · 1 reacción ·
-
Message named pkg/Duration with field builtin_interfaces/Duration is invalidPosiblemente ocupada @Lidang-Jiang la tomó hace 166 días. Abiertobug
Dificultad 4/5 3-5 días Aptitud para principiantes 55/100
ros2/rosidl_python#257 · 8 comentarios ·
-
enhancement
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
ros2/rosidl_python#242 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 32/100
ros2/rosidl_python#219 · 2 comentarios ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
ros2/rosidl_python#217 · 2 comentarios ·
Todos los issues de ros2/rosidl_python
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
atlassian-labs/compiled#1960 ·
Los mantenedores suelen responder en 1 día
-
Upgrade to Commons Codec 1.23.0Abiertostatus: team-only type: dependency-upgrade
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
spring-projects/spring-boot#52143 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
Los mantenedores suelen responder en 2 días
-
fails to compile with Qt6Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
Los mantenedores suelen responder en 1 día