• Max Kellermann's avatar
    Revert "player/Thread: remove unnecessary "pipe" check" · 304d45b5
    Max Kellermann authored
    This reverts commit ff3e2c05.  The
    check was necessary, after all, because this is what checked whether
    the decoder had finished the current or the next song.
    
    > The "queued" flag can only possibly be set if the decoder is still
    > decoding the current song or if the decoder is stopped.
    
    That was wrong because ProcessCommand() sets `queued=true` and also
    starts the decoder (if it was idle).
    
    > This is also what the following assert() checks.
    
    That was also wrong, because the assert() has two conditions.
    
    Closes https://github.com/MusicPlayerDaemon/MPD/issues/566
    304d45b5
Thread.cxx 27.1 KB