micro-ROS / micro-ROS/micro_ros_diagnostics

Translation indexes should be unsigned

Ouverte
#33 0 commentaires 2 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

enhancement help wanted
Langage dominant
C
Étoiles
10
Forks
10
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

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.

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Recherchez dans le dépôt les déclarations et les utilisations de HW_ID, TASK_ID, UPDATER_ID et KEY, puis examinez le code environnant de l’index de traduction ainsi que toutes les vérifications pertinentes. Le travail est terminé lorsque les quatre index utilisent uint16_t de manière cohérente sans perturber leurs utilisations existantes.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
c
Domaine
embedded-iot
Type d'issue
Bug
Difficulté
2/5
Temps estimé
1-3 heures
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
48/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.