The `db->close()` call was a `nullptr` dereference because the `db`
variable had already been moved.

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

This method gets called a lot during MPD startup, via FindChild() and
directory_load_subdir(), so this is worth optimizing at the expense of
code readability.

This speeds up MPD startup by 10%.

Instead of passing tag and group, pass an array of tags.  To support a
nested return value, return a nested std::map of std::maps.  Each key
specifies the tag value, and each value may be another nesting level.

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

Fixes a memory leak, or an assertion failure in the debug build.

Fixes assertion failure.

`light_song.Get()` could cause an assertion failure because the
`Manual<>` object must not be used if uninitialized.

Regression by commit ebc006ab

Since we switched from autotools to Meson in commit
94592c14, we don't need to include
`config.h` early to properly enable large file support.  Meson passes
the required macros on the compiler command line instead of defining
them in `config.h`.

This means we can include `config.h` at any time, whenever we want to
check its macros, and there are no ordering constraints.

Grouping in the "list" command was completely broken from the start,
unlike "count group".  I have no idea what I have been thinking when I
wrote commit ae178c77, but it didn't
make any sense.

This commit is a rewrite of the feature.

For clients to be able to detect this feature, this commit also
increments the protocol version.

That way, each plugin can decide to implement it better.

This passes the new `sort` and `window` attributes to the mounted
database.

Modeled after std::filesystem::operator/() from C++17.

This enforces the "not nullptr" rule.

Prepare to make LightSong non-assignable.

Fixes search within mount points, resulting in error "No such
directory".

Closes #190

No longer a problem with NDK r16.

If a directory is a mount point, omit the "directory: " as well.

This bug is years old, but has become more visible now that mount
points are persistent in the state file.

If `SimpleDatabase::Visit` is called on a database that contains a mounted directry the URIs of the elements passed to the callbacks are not prefixed by the mountpoint path. This leads to lsinfo and add not working because they use the wrong URI. This pull request is using the `WalkMount` helper function to create prefixed versions of `VisitDirectory`, `VisitSong` and `VisitPlaylist` to add the correct prefix to the parameters of the callback functions.

Only _WIN32 is defined by the compiler, and WIN32 is not standardized
and may be missing.

Closes #169

assignment of `mnt->mounted_database` in `SimpleDatabase::Mount`.

Signed-off-by: FlashSystems <developer@flashsystems.de>

Allows database plugins to use the EventThread, e.g. for CURL
integration.