We overloaded enum mpd_state to add an “offline” state. Don’t do
this. We can detect the offline state by checking if our connection
object is NULL.
Closes#16
When we detect an error in yml_parser_parse(), we handle that
specifically, and then jump to the generic error handler.
The generic error handler overwrites the previously formatted error
before cleaning up and returning.
This meant that a) the actual error message was lost, and replaced
with a generic “unknown error”, and b) the dynamically allocated error
string buffer was leaked.
