micro-ROS / micro-ROS/micro_ros_diagnostics

Translation indexes should be unsigned

オープン
#33 コメント 0 件 リアクション 2 件 担当者 0 名 GitHub で見る
enhancement help wanted
主要言語
C
スター
10
フォーク
10
PR マージ指標
30日以内にマージされた PR はありません

説明

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.

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

リポジトリ内で HW_ID、TASK_ID、UPDATER_ID、KEY の宣言と使用箇所を検索し、その周辺の翻訳インデックスコードと関連するチェックを調査します。完了とは、既存の使用箇所を壊すことなく、4 つすべてのインデックスが一貫して uint16_t を使用している状態です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
c
領域
embedded-iot
issue の種類
バグ
難易度
2/5
見積もり時間
1〜3時間
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
48/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。