gbdev / gbdev/rgbds

Add `-Winclude-shadow` to warn about multiple candidates for `INCLUDE`/`INCBIN`/`READFILE`

Open
#2,018 0 comments 0 reactions 0 assignees View on GitHub
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.