diff options
author | Scott Murray <scott.murray@konsulko.com> | 2023-02-19 17:39:28 -0500 |
---|---|---|
committer | Scott Murray <scott.murray@konsulko.com> | 2023-02-21 17:41:22 +0000 |
commit | eb2f0cdd984307cec1df7ab3821f8af2248211df (patch) | |
tree | 0eaed6fb57fa9548a6652cbbef39dc42db888740 /.gitreview | |
parent | 22ad8cbaad00d89361cbd92023dd20a807bcf5cb (diff) |
mediaplayer: Improve MPD failure handling
Changes:
- Avoid crashing in the MPD event handler code when our connection
times out because it has hung. Additionally, attempt to update
the front end UI to a sane state with respect to metadata when
this happens.
- Lower the MPD connection timeout and keepalive timeouts to try to
catch MPD failures faster. There are still problems in this area
with respect to the front end UI getting stuck if it does an
operation that would attempt to use the MPD API while it is hung.
Fixing that completely would require a significant refactoring to
move all the MPD API code to a separate thread so that the QML
rendering thread does not get stuck. It's unclear at present if
such a refactoring is justified, as the MPD hang in this case is
fixed by changes in MPD 0.23.8.
Bug-AGL: SPEC-4661
Signed-off-by: Scott Murray <scott.murray@konsulko.com>
Change-Id: Ia5e74d00acdc6b3b28cdf87b26486dbddfa02a98
(cherry picked from commit 2cbbf914242d0766ff4e97ee3c0d02505d13e422)
Diffstat (limited to '.gitreview')
0 files changed, 0 insertions, 0 deletions