zephyrproject-rtos/zephyr

remove -Wno-pointer-sign

Open

#8921 aperta il 13 lug 2018

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)C (9144 fork)batch import
EnhancementGood first issueNeeds reviewpriority: low

Metriche repository

Star
 (15.249 star)
Metriche merge PR
 (Merge medio 5g 20h) (1000 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Individua tutte le occorrenze di `zephyr cc option( Wno pointer sign)` nei file CMake di Zephyr. Rimuovi questa opzione, quindi compila il progetto per identificare gli avvisi relativi alle incongruenze tra puntatori con e senza segno. Correggi ogni avviso aggiungendo cast appropriati o modificando i tipi di variabile. I commenti sull'issue (4 commenti) potrebbero fornire suggerimenti. Poiché non sono collegati PR, un buon approccio è iniziare con un sottosistema e inviare una PR con correzioni incrementali. La codebase di Zephyr è ampia, quindi concentrati su un'area specifica come il kernel o i driver.
Tech stack
c
Dominio
developer experience
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
3-5 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
C programmingunderstanding of pointer types
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
35