DamienCassou

DamienCassou pushed to master at mpdel/libmpdel

0ccf76c860 Merge pull request 'Add libmpdel-get-state to run handler after state is get' (#31) from vonfry/libmpdel:fix/play-pause into master
928857b394 Add libmpdel-get-state to run handler after state is get
Compare 2 commits »

6 months ago

DamienCassou closed issue mpdel/libmpdel#30

First run libmpdel-playback-play-pause with error `apply: Wrong number of arguments: #<subr format>, 0`

6 months ago

DamienCassou merged pull request mpdel/libmpdel#31

Add libmpdel-get-state to run handler after state is get

6 months ago

DamienCassou approved mpdel/libmpdel#31

Add libmpdel-get-state to run handler after state is get

Perfect, thank you very much!

6 months ago

DamienCassou commented on issue mpdel/mpdel#35

Connection refused

I'm lost, I'm sorry.

6 months ago

DamienCassou approved mpdel/libmpdel#31

Add libmpdel-get-state to run handler after state is get

Here are some more suggestions. Thank you for your great work!

6 months ago

DamienCassou commented on pull request mpdel/libmpdel#31

Add libmpdel-get-state to run handler after state is get

I suggest only calling `libmpdel-send-command` once:

6 months ago

DamienCassou commented on pull request mpdel/libmpdel#31

Add libmpdel-get-state to run handler after state is get

if you apply my suggestion above, you get the `play-state` as argument here so you don't have to call `(libmpdel-play-state)` below.

6 months ago

DamienCassou commented on pull request mpdel/libmpdel#31

Add libmpdel-get-state to run handler after state is get

I suggest passing the variable value as argument to `handler`. Something like (untested):

6 months ago

DamienCassou approved mpdel/libmpdel#31

Add libmpdel-get-state to run handler after state is get

Thank you very much. I suggested some changes.

6 months ago

DamienCassou commented on pull request mpdel/libmpdel#31

Add libmpdel-get-state to run handler after state is get

I suggest adding an optional callback to `libmpdel-refresh-status`. This callback would be called when refreshing is finished.

6 months ago

DamienCassou commented on issue mpdel/libmpdel#30

First run libmpdel-playback-play-pause with error `apply: Wrong number of arguments: #<subr format>, 0`

Thank you very much for your the bug report. I confirm the bug exists and has always existed. It never bothered me though because it fixes itself immediately :-). What happens is that some command…

6 months ago

DamienCassou pushed to master at mpdel/libmpdel