micro-ROS / micro-ROS/micro_ros_diagnostics
Translation indexes should be unsigned
Offen
enhancement
help wanted
- Vorherrschende Sprache
- C
- Sterne
- 10
- Forks
- 10
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
Currently, all indexes
- `HW_ID`
- `TASK_ID`
- `UPDATER_ID`
- `KEY`
are signed int16_t values, one shouldn't be indexing anything on a negative value (although could be possible). Thus, I'd suggest changing these indexes to be uint16_t instead.
Beitragsleitfaden
Rechercherichtung
Durchsuche das Repository nach den Deklarationen und Verwendungen von HW_ID, TASK_ID, UPDATER_ID und KEY und untersuche anschließend den umgebenden Übersetzungsindex-Code sowie alle relevanten Prüfungen. Erledigt bedeutet, dass alle vier Indizes konsistent uint16_t verwenden, ohne ihre bestehenden Verwendungen zu beeinträchtigen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- c
- Bereich
- embedded-iot
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 48/100