beyond-all-reason / beyond-all-reason/RecoilEngine
ProjectileCreated/Destroyed: use nil instead of -1
Open
area: Lua API
- Dominant language
- C++
- Stars
- 679
- Forks
- 290
- Avg merge
- 3d 2h
- Merged PRs (30d)
- 40
Description
Currently they pass a magic -1 value when no valid value exists. Use nil instead.
Contributor guide
Research direction
Start by locating the implementations and callers of ProjectileCreated and ProjectileDestroyed. Check how an absent value is represented, replace the -1 sentinel with nil, and verify that both creation and destruction paths consistently handle the absent value.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- game-dev
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100