Current code only supports one player
Details
Details
- Reviewers
apol - Group Reviewers
KDE Connect - Commits
- R224:1b622de1f179: Support multiple players in MprisRemote
Run two media sessions. Dbus inspection works fine, UI has some issues with values not updated, but out of scope for this diff
Diff Detail
Diff Detail
- Repository
- R224 KDE Connect
- Branch
- titleartistalbum
- Lint
No Linters Available - Unit
No Unit Test Coverage
Comment Actions
Other than that looks good to me. Thanks!
plugins/mprisremote/mprisremoteplugin.cpp | ||
---|---|---|
205 ↗ | (On Diff #33168) | While this will work, if the map ever gets out of sync we could get a crash. Maybe it's just safer to do something like: auto player = m_players.value(m_currentPlayer); return player ? player->album() : QString(); |