This fixes #1616

This fixes a std::terminate() crash in the CURL storage plugin when
PropfindOperation::OnHeaders() throws an exception after receiving a
non-207 status.

Closes https://github.com/MusicPlayerDaemon/MPD/issues/1559

Signed-off-by: Rosen Penev <rosenp@gmail.com>

Fixes building with GCC 12.

Signed-off-by: Rosen Penev <rosenp@gmail.com>

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>

Necessary since commit 6acf81d5

Found with cppcoreguidelines-special-member-functions
Signed-off-by: Rosen Penev <rosenp@gmail.com>

Shorter.
Signed-off-by: Rosen Penev <rosenp@gmail.com>

Preparing for a variant of TimerEvent with coarse 1-second
granularity, but cheaper (with a timer wheel).

This eliminates some complexity from class CurlRequest.

Shorter.
Signed-off-by: Rosen Penev <rosenp@gmail.com>

Keep the logging library out of this low-level libcurl wrapper.

Similar to commits 1686f4e8 and
30a5dd26

Our Windows build is built with `--disable-netrc`, and that makes
CURLOPT_NETRC fail, causing failures with all streams.  D'oh!

Closes https://github.com/MusicPlayerDaemon/MPD/issues/886

Apparently, it's not possible to change CURLOPT_NETRC on Android.

Found with readability-uppercase-literal-suffix
Signed-off-by: Rosen Penev <rosenp@gmail.com>