chore: enable -Wthread-safety and add LockGuard

romange / helio

A modern framework for backend development based on io_uring Linux interface

Apache License 2.0

447 stars 50 forks source link

chore: enable -Wthread-safety and add LockGuard #306

Closed kostasrim closed 3 months ago

kostasrim commented 3 months ago

std::lock_guard and std::unique_lock do not work with ABSL_GUARD_XXX variants and for that we need our own wrappers. Furthermore, even if helio doesn't use those guards right now, they might be useful in the future so I add them here and enable -Wthread-safety for clang

enable -Wthread-safety
add LockGuard as replacement of std::LockGuard

kostasrim commented 3 months ago

@romange FYI :)