beyond-all-reason / beyond-all-reason/RecoilEngine
Consider `VFS.RecursiveDirList`
- Dominant language
- C++
- Stars
- 683
- Forks
- 293
- Avg merge
- 3d 2h
- Merged PRs (30d)
- 40
Description
`VFS.RecursiveDirList(path, filter, mode)` would be equivalent to current `VFS.DirList(path, filter, mode, true)` since the 4th bool arg is whether the search is recursive. Named functions are clearer (for both reading and writing) than bool args.
Contributor guide
Research direction
Start by locating the declaration and uses of VFS.DirList, especially calls that pass true as the recursive argument. Check how the VFS API is exposed and tested, then determine whether the named RecursiveDirList equivalent can be added without changing existing behavior. Done means the clearer named entry point is available and its recursive behavior is covered or verified.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- backend-api-design
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100