AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

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

Offen
#2,258 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
C++
Sterne
2.1k
Forks
505
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

Same goes for the std::lock_guard. Should this be replaced with Mutex and AutoMutex ?

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginnen Sie in ColorSpaceHelpers.cpp und prüfen Sie, wie std::mutex und std::lock_guard verwendet werden. Vergleichen Sie anschließend den umgebenden Code mit der Verwendung der Mutex- und AutoMutex-Wrapper des Projekts. Bestätigen Sie die vorgesehene Konvention anhand der bestehenden Verwendung der Wrapper. Die Aufgabe gilt als abgeschlossen, wenn die direkten Standard-Mutex-Typen gegebenenfalls ersetzt wurden und die Änderung mit der umgebenden Implementierung konsistent bleibt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cpp
Bereich
tooling
Issue-Typ
Refactoring
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Ruhig
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
56/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.