arduino / arduino/ArduinoCore-API

String object, len is of type unsigned int, however indexOf() and lastIndexOf() return int

Ouverte
#239 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub
bug
Langage dominant
C++
Étoiles
306
Forks
150
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

### Description

String Object:

len is of type unsigned int.
method that apply to position into the String or its size are expecting an unsigned int, this is the case with:
charAt()
substring()
remove()
reserve()

Oddly, indexOf() and lastIndexOf() are returning int and not unsigned int

### Current behavior

Oddly, indexOf() and lastIndexOf() are returning int and not unsigned int

### Expected behavior

indexOf() and lastIndexOf() ought to return unsigned int, to avoid unnecessary casting from int to unsigned int

### Additional information
It's frequent to pass the result of indexOf() to substring()

Thanks

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Piste de recherche

Recherchez dans l’implémentation de String de ArduinoCore-API les déclarations et les implémentations de indexOf() et lastIndexOf(), puis comparez leurs types de retour avec len, charAt(), substring(), remove() et reserve(). Vérifiez que les deux méthodes utilisent systématiquement le type unsigned prévu et que les appelants qui transmettent leurs résultats à substring() n’ont plus besoin d’un cast.

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

Évaluation

Stack technique
cpp
Domaine
api, embedded-iot
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Clairement spécifiée
Accessibilité débutants
52/100

Recevez les nouvelles issues par e-mail

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