SonarLint reports the latter to be better:

std::scoped_lock basically provides the same feature as std::lock_guard,
but is more generic: It can lock several mutexes at the same time, with a
deadlock prevention mechanism (see {rule:cpp:S5524}). The equivalent code
to perform simultaneous locking with std::lock_guard is significantly more
complex. Therefore, it is simpler to use std::scoped_lock all the time,
even when locking only one mutex (there will be no performance impact).
Signed-off-by: Rosen Penev <rosenp@gmail.com>

This adds a bit of overhead, but also adds flexibility to the API,
because arbitrary triggers may be invoked from that virtual method
implementation, not just Cond::signal().

The motivation for this is to make the handlers more dynamic, for the
upcoming buffering class utilizing ProxyInputStream.

Eliminate dependency on io_thread_get().

Obsolete class CloseSongEnumerator, which was a kludge.

Since version 2.32, g_thread_init() is deprecated and a no-op.  Let's
upgrade that (optional) dependency so we can remove all those
g_thread_init() calls.

Auto-stop the IO thread in all error handlers.

Revert to the old API before commit e9e55b08, removing unnecessary
bloat.

Replaces the method Close().

Merge some duplicate code.

Avoid GLib.

Obsolete, our logging library doesn't ues GLib anymore.

Allows forward-declaration.

Prepare to migrate away from GLib.  Currently, we're still using GLib
as a backend.