Mike Kazantsev <mk.frag...@gmail.com> writes:

> On Fri, 25 May 2018 16:02:10 -0400
> Yoni Rabkin <y...@rabkins.net> 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"

_______________________________________________
Emms-help mailing list
Emms-help@gnu.org
https://lists.gnu.org/mailman/listinfo/emms-help

Reply via email to