emms-help
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [emms-help] mpv player alternative with json-ipc


From: Yoni Rabkin
Subject: Re: [emms-help] mpv player alternative with json-ipc
Date: Fri, 25 May 2018 18:59:06 -0400
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux)

Mike Kazantsev <address@hidden> writes:

> On Fri, 25 May 2018 16:02:10 -0400
> Yoni Rabkin <address@hidden> wrote:
>
>> Before you do, please change the dependence on the cl package to the
>> newer cl-lib (clean namespace), and make sure that emms-mpv-ipc-proc is
>> defined before it is referenced.
>
> Oh, right, forgot to check it for warnings after updates, fixed both of
> these, thanks.
>
> Second one by adding dummy define for emms-mpv-ipc-proc to keep
> defcustoms in the head of the file before any internals for readability
> (as I tend to skim code to find tunables myself, not customize via ui).
>
>
>> Please feel free to merge this into master so that more people can
>> hammer on it.
>
> Done.
>
> I did squash all commits of the initial implementation into one, as they
> don't seem to be very relevant to overall history, same as it was with
> previous implementation.
>
> Indeed, more exposure should hopefully find more interesting uses and
> tweaks for many things that mpv offers via its APIs, as well as weed
> out bugs faster.

Another change needed is to add a regexp to the player so that
(emms-player-get emms-player-mpv 'regex) will return a useful value that
emms-source-file-regex can use.

Among other things this breaks is that the various
emms-add-... functions will add files that the player doesn't know how
to play. For instance, jpegs that litter album directories will start
showing up in the playlists, etc.

Adding this:

(apply #'emms-player-simple-regexp emms-player-base-format-list)

...is typically what we start with.

-- 
   "Cut your own wood and it will warm you twice"



reply via email to

[Prev in Thread] Current Thread [Next in Thread]