c++17 is offered by default with Xcode 10
https://developer.apple.com/documentation/xcode_release_notes/xcode_10_release_notes

The ones in std have overloads for const char/char.
Signed-off-by: Rosen Penev <rosenp@gmail.com>

Passing `length+1` to `MultiByteToWideChar()` means the function may
fill the whole buffer with output data, and could theoretically
overwrite the null terminator.  In practice, this will never happen,
but this way, it's slightly more correct.

Also, null-terminate after `MultiByteToWideChar()`, after we got the
real output length.  Again, this would never have been a problem, but
who knows...

Needed for std::wmemchr under libcxx
Signed-off-by: Rosen Penev <rosenp@gmail.com>

Fixes `-Wpedantic`.

libavutil's macro definition is a compound literal, which is illegal
in C++.  Fixes yet another -Wpedantic warning.

Some more `-Wpedantic` fixups.

Enables `-Wpedantic`.

Fixes regression from commit 69f09648

This branch isn't yet ready for level 3 (`-Wpedantic`) due to several
C++ violations (e.g. variable length arrays).  These are already
cleaned up in the master branch (0.22).

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

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