Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MPD: Unnecessary multiple idle subsystem events #16

Open
torumyax opened this issue Mar 6, 2018 · 6 comments
Open

MPD: Unnecessary multiple idle subsystem events #16

torumyax opened this issue Mar 6, 2018 · 6 comments
Labels
C-enhancement Category: A PR with an enhancement or an issue with an enhancement proposal

Comments

@torumyax
Copy link

torumyax commented Mar 6, 2018

This isn't much of an issue, but it seems,

Sending this MPD command

command_list_begin
clear
load someplaylist
play
command_list_end

returns

changed: player
OK

changed: player
changed: playlist
changed: mixer
OK

It would be nice to have fewer nortifications for clients since it is sometimes costly to update UI thread in the GUI apps and especially for mobile clients.

Also, sending

next

returns

changed: mixer
OK

changed: player
OK

It would be nice if it is just,

changed: player
changed: mixer
OK

or I'm not sure for other cases but, in my case at least,

changed: player
OK

is enough since I'd be issuing "status" command and it would return volumes anyways.

I'm using Mopidy 2.1.0 from apt repo on Ubuntu 17.10.
And sorry for my English. English isn't my language.

@kingosticks
Copy link
Member

kingosticks commented Mar 16, 2018 via email

@torumyax
Copy link
Author

Hello. I'm sorry. I just thought this wasn't much of an "issue". And I did not want to trouble you...so.

@kingosticks
Copy link
Member

No, all issues are important and the simpler ones make good starters for people new to the project.

@kingosticks kingosticks reopened this Mar 16, 2018
@torumyax
Copy link
Author

OK. I'll try to write again. Again sorry for the inconvenience.

  • If I know Python, I would contribute to the project, but ... all I can is to report. I did write an introduction to Mopidy in Japanese in my blog though.

@kingosticks
Copy link
Member

kingosticks commented Mar 16, 2018

Please, never edit posts and delete what you wrote on any forum unless you really have to.

And just reporting is fine. Thanks

@torumyax
Copy link
Author

I'll try to keep it in mind. I'm glad that I didn't offend or anything at least.

@jodal jodal transferred this issue from mopidy/mopidy Dec 20, 2019
@jodal jodal added the C-enhancement Category: A PR with an enhancement or an issue with an enhancement proposal label Dec 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Category: A PR with an enhancement or an issue with an enhancement proposal
Projects
None yet
Development

No branches or pull requests

3 participants