zephyrproject-rtos/zephyr

remove -Wno-pointer-sign

Open

#8,921 opened on Jul 13, 2018

View on GitHub
 (4 comments) (0 reactions) (0 assignees)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.

Contributor guide

Tech stack
c
Domain
developer experience
Issue type
refactor
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
3-5 days
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
mostly clear
Prerequisites
C programmingunderstanding of pointer types
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
35
Research direction
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.