cpp: audit include statements
Nobody has claimed this yet.
Assessment
- Difficulty
- 5/5
- Estimated time
- Over a week
- Newbie friendliness
- 25/100
Research direction
Start by reviewing include warnings in the C++ code across LizardByte/Sunshine, LizardByte/libdisplaydevice, and LizardByte/tray. Compare cppclean and include-what-you-use against the CLion findings, then determine the CI scope. Done means the includes have been audited and an agreed automated check is integrated into CI.
Written by the indexing model from the issue text.
Description
Is there an existing issue for this item?
- I have searched the existing issues
Repositories
LizardByte/Sunshine, LizardByte/libdisplaydevice, LizardByte/tray
Languages/Skills/Technologies
C++
Description
Many includes are shown as not used by CLion. I haven't investigated if these are legitimate warnings, but we should audit the includes and probably automate it as part of CI.
I found the following that might be able to help:
https://stackoverflow.com/questions/614794/detecting-superfluous-includes-in-c-c
Estimated Effort
effort:Large
Priority
priority:High
Target Milestone
1-3 months
Dependencies
No response
- Dominant language
- No language data
- Stars
- 2
- Forks
- 0
- Avg merge
- 25m
- Merged PRs (30d)
- 5
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from LizardByte/roadmap
-
1-3 months effort:Large LizardByte/Sunshine packaging planned priority:Medium
Difficulty 5/5 Over a week Newbie friendliness 35/100
LizardByte/roadmap#178 ·
-
1-3 months C++ CSS effort:Medium HTML JavaScript LizardByte/Sunshine planned priority:High Vue
Difficulty 5/5 Over a week Newbie friendliness 35/100
LizardByte/roadmap#153 · 2 reactions ·
-
Dependency Dashboard Opendependencies
LizardByte/roadmap#151 · 1 assignee ·
-
1-3 months Bash/Shell/Scripting effort:Medium GitHub actions LizardByte/actions planned priority:High
LizardByte/roadmap#134 · 1 assignee ·
-
1-3 months C++ effort:Medium LizardByte/Sunshine planned priority:Medium Rest API
Difficulty 5/5 Over a week Newbie friendliness 35/100
LizardByte/roadmap#101 ·
All issues in LizardByte/roadmap
Similar issues
-
kind/bug needs-triage
Difficulty 1/5 Under an hour Newbie friendliness 72/100
matrixorigin/matrixone#29223 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
copse-dev/agent-pane#2953 ·
-
bug ci-failure high priority
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
vllm-project/vllm-omni#7972 · 1 comment ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 90/100
simonw/sqlite-utils#872 ·
-
bug good first issue
Difficulty 1/5 Under an hour Newbie friendliness 88/100
amponce/archive-movie-browser#166 ·