search

foxbow / mixplay

mpg123 based player frontend supporting demon mode, random play, stream play and remote access. Designed to act as a replacement for my Squeezebox should it cease to work one day.
MIT License
1 stars 1 forks source link

Treat search as update #30

Closed foxbow closed 5 years ago

foxbow commented 5 years ago

treating search asynchronously begs for problems with a stateless protocol. Implement search as an update message so the search results must be in the matching reply and reach the proper client.

foxbow commented 5 years ago

In mpc_search, remove progressMsg and set message in ->argument instead. Then set found->state to search_error to enable returning the message immediately. Or do an explicit lock/progress/send in server.

foxbow commented 5 years ago

Looks good so far.