cpplint checks fail for services and actions with buffers in the response/result
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 56/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Stack tecnologico
- c
- Ambito
- build-system, testing-qa
Direzione di ricerca
Inizia con il file rosidl_generator_py generato build/.../rosidl_generator_py/srv/_get_buffer_s.c e con il fallimento build/include_order di cpplint_rosidl_generated_py. Confronta la generazione di service/action descritta in #263 con l’iniezione di stdint.h di #250, incluso il trattamento di response/result/feedback rispetto a request/goal. Il lavoro è concluso quando i controlli cpplint per il codice generato interessato hanno esito positivo senza introdurre errori negli altri casi.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.
- Lingua principale
- EmberScript
- Stelle
- 26
- Fork
- 68
- Merge medio
- 2g 13h
- PR unite (30g)
- 1
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di ros2/rosidl_python
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
ros2/rosidl_python#255 · 1 reazione ·
-
bug
Difficoltà 4/5 3-5 giorni Idoneità per principianti 55/100
ros2/rosidl_python#257 · 8 commenti ·
-
enhancement
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
ros2/rosidl_python#242 ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 32/100
ros2/rosidl_python#219 · 2 commenti ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
ros2/rosidl_python#217 · 2 commenti ·
Tutte le issue di ros2/rosidl_python
Issue simili
-
comp/dashboard P3 type/bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
NousResearch/hermes-agent#117722 ·
-
Use zstd compression? ApertaNeeds Design Priority: Wishlist
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
elementary/flatpak-platform#253 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
objectionary/hone-maven-plugin#1060 ·
-
0.kind: build failure
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
-
Update Vish to 1.1.5 Apertapackage-update
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
oSoWoSo/vOid_Community_repOsitory#144 · 1 commento ·