zephyrproject-rtos/zephyr

remove -Wno-pointer-sign

Open

#8 921 ouverte le 13 juil. 2018

Voir sur GitHub
 (4 commentaires) (0 réactions) (0 assignés)C (15 249 stars) (9 144 forks)batch import
EnhancementGood first issueNeeds reviewpriority: low

Description

Remove zephyr_cc_option(-Wno-pointer-sign) and fix all code that mixes signed/unsigned pointers.

Guide contributeur

Stack technique
c
Domaine
developer experience
Type d'issue
refactor
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
3-5 days
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
mostly clear
Prérequis
C programmingunderstanding of pointer types
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
35
Direction de recherche
Locate all occurrences of `zephyr cc option( Wno pointer sign)` in the Zephyr CMake files. Remove this option, then compile the project to identify warnings about signed/unsigned pointer mismatches. Fix each warning by adding appropriate casts or changing variable types. Comments on the issue (4 comments) may provide hints. Since no PRs are linked, a good approach is to start with one subsystem and submit a PR with incremental fixes. The Zephyr codebase is large, so focus on a specific area like kernel or drivers.