-
Notifications
You must be signed in to change notification settings - Fork 21
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
Comments
Why closed? Is the report incorrect?
…On 16 Mar 2018 6:56 am, "torum" ***@***.***> wrote:
Closed mopidy/mopidy#1662 <https://github.com/mopidy/mopidy/issues/1662>.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<https://github.com/mopidy/mopidy/issues/1662#event-1524889498>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AA5DqGs3ziiRupr7SlQ20Ky1LHAKwtfzks5te2IxgaJpZM4Senx6>
.
|
Hello. I'm sorry. I just thought this wasn't much of an "issue". And I did not want to trouble you...so. |
No, all issues are important and the simpler ones make good starters for people new to the project. |
OK. I'll try to write again. Again sorry for the inconvenience.
|
Please, never edit posts and delete what you wrote on any forum unless you really have to. And just reporting is fine. Thanks |
I'll try to keep it in mind. I'm glad that I didn't offend or anything at least. |
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.
The text was updated successfully, but these errors were encountered: