Prefer not operator
Nobody has claimed this yet.
Assessment
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Newbie friendliness
- 72/100
- Issue type
- Refactor
- Clarity
- Clearly specified
- Activity status
- Stale
- Domain
- build-system
Research direction
Open gpr/src/gpr-nmsc.adb at the linked line around 743 and inspect the existing condition. Replace the condition with the proposed not operator form, then review the surrounding Ada formatting and behavior to confirm the change is equivalent.
Written by the indexing model from the issue text.
Description
Dear Gprbuild developers,
Why not replace
https://github.com/AdaCore/gprbuild/blob/1e13ed644c71509a0a6b8cd3102d88cedb59cf88/gpr/src/gpr-nmsc.adb#L743
with
if not (Add_Src) then
Greetings,
Pierre
Problem detected and solved by Rejuvenation-Ada crate
- Dominant language
- Ada
- Stars
- 81
- Forks
- 24
- PR merge metrics
- No merged PRs in 30d
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 AdaCore/gprbuild
-
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
-
Difficulty 4/5 3-5 days Newbie friendliness 45/100
-
Difficulty 4/5 3-5 days Newbie friendliness 30/100
-
Difficulty 5/5 Over a week Newbie friendliness 25/100
-
Difficulty 3/5 1-2 days Newbie friendliness 45/100
All issues in AdaCore/gprbuild
Similar issues
-
type/automation type/tech-debt
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
priority: p3
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
googleapis/librarian#7636 ·
-
package-update
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
oSoWoSo/vOid_Community_repOsitory#147 · 1 comment ·
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
-
agentic-workflows cascade-suspected
Difficulty 1/5 Under an hour Newbie friendliness 88/100