Sorry for the long delay, I've finally spent some time review the mpv code. It's pretty neat I think, I like it. I've fixed a few nits already.
Some questions:
- In `emms-player-mpv-proc-stop': (setq emms-player-mpv-proc nil) should
be done at the end I think, or in case of interrupt ("C-g") at the
wrong time the variable will be nil while the player will not be
stopped. Further calls to emms-player-mpv-proc-stop won't be able to
stop the player.
- In `emms-player-mpv-ipc-connect-fifo':
--8<---------------cut here---------------start------------->8---
(format "cat > '%s'"
(replace-regexp-in-string "'" "'\"'\"'" emms-player-mpv-ipc-socket t t))
--8<---------------cut here---------------end--------------->8---
That looks like proper ugly shell! :p I haven't tried, but couldn't we
use `shell-quote-argument' instead?
--8<---------------cut here---------------start------------->8---
(concat "cat > "
(shell-quote-argument emms-player-mpv-ipc-socket))
--8<---------------cut here---------------end--------------->8---
Thank you very much for your hard work, Mike!
--
Pierre Neidhardt
signature.asc
Description: PGP signature
_______________________________________________ Emms-help mailing list [email protected] https://lists.gnu.org/mailman/listinfo/emms-help
