AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

ColorSpaceHelpers.cpp directly uses std::mutex rather than our Mutex wrappers

Open
#2,258 1 comment 0 reactions 0 assignees View on GitHub
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.