micro-ROS / micro-ROS/micro_ros_diagnostics
Translation indexes should be unsigned
Đang mở
Chưa có ai nhận issue này.
enhancement
help wanted
- Ngôn ngữ chính
- C
- Star
- 10
- Fork
- 10
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Mô tả
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.
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Hướng nghiên cứu
Tìm trong repository các khai báo và nơi sử dụng của HW_ID, TASK_ID, UPDATER_ID và KEY, sau đó kiểm tra phần code chỉ mục bản dịch xung quanh cùng mọi kiểm tra liên quan. Hoàn thành nghĩa là cả bốn chỉ mục đều sử dụng uint16_t một cách nhất quán mà không làm hỏng các cách sử dụng hiện có của chúng.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- c
- Lĩnh vực
- embedded-iot
- Loại issue
- Lỗi
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 48/100