AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO
ColorSpaceHelpers.cpp directly uses std::mutex rather than our Mutex wrappers
Open
- Dominant language
- C++
- Stars
- 2.1k
- Forks
- 503
- PR merge metrics
- No merged PRs in 30d
Description
Same goes for the std::lock_guard. Should this be replaced with Mutex and AutoMutex ?
Contributor guide
Research direction
Start in ColorSpaceHelpers.cpp and inspect how std::mutex and std::lock_guard are used, then compare nearby code for the project's Mutex and AutoMutex wrappers. Confirm the intended convention with existing wrapper usage; done means the direct standard mutex types are replaced if appropriate and the change remains consistent with the surrounding implementation.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- tooling
- Issue type
- Refactor
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 56/100