This eliminates some overhead, because the compiler doesn't need to
consider these functions throwing.

An ino_t is usually a 64 bit integer, and some file systems (such as
Linux's kernel NFS client) really uses the upper 32 bit.  This can
lead to false positives in the directory loop detection in
FindAncestorLoop().  Increasing these two attributes (in
StorageFileInfo and Directory) to 64 bit adds little overhead, but
makes the check a lot safer.

If the base class is not accessible, the "catching" the base class
won't work.  This caused the fatal error:

 terminate called after throwing an instance of 'LibmpdclientError'

SimpleDatabase::Visit() always throws exception when visiting a song.
Adding a song to the queue always results in a "No such directory"
error response, although the song is successfully added.  This
behaviour was introduced by commit
3ff728ab.  This patch fixes this.

https://bugs.musicpd.org/view.php?id=4601

Implementations shall use exceptions instead.

Use this function instead of CheckError() when we already know an
error has occurred.

Obsoleted by commit d4d4d621