Add `-Winclude-shadow` to warn about multiple candidates for `INCLUDE`/`INCBIN`/`READFILE`
Open
enhancement
rgbasm
- Dominant language
- C++
- Stars
- 1.6k
- Forks
- 193
- Avg merge
- 22h 17m
- Merged PRs (30d)
- 26
Description
`rgbasm -I/--include` can specify multiple possible include paths, and the same file might exist in more than one. We have an unambiguous resolution sequence, but that doesn't mean users always intentionally expect it.
Contributor guide
Research direction
Start with rgbasm's -I/--include handling and the resolution paths for INCLUDE, INCBIN, and READFILE. Review the existing rgbasm tests, then consider the work complete when the new -Winclude-shadow warning consistently identifies cases where multiple include paths contain a candidate file.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- cli, compilers
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 57/100