[emms-help] Re: MusicPD player, EMMS, and a customization option

[Daniel Brockman]

Michael Olson <address@hidden> writes:

> Also, could someone please change the defcustom for
> emms-player-list to:
>
> (defcustom emms-player-list nil
>   "*List of players that EMMS can use.  You need to set this!"
>   :group 'emms
>   :type '(repeat (sexp :tag "Player")))
>
> If :type is 'function, an error will be thrown when using
> the Customize interface to save changes to this option,
> since no function definition (i.e. emms-player-mplayer) is
> provided for any of the players.

Thanks for catching this.  I've commited the following patch:

Index: emms.el
===================================================================
RCS file: /cvsroot/emms/emms/emms.el,v
retrieving revision 1.58
diff -u -r1.58 emms.el
--- emms.el     14 Aug 2005 16:02:20 -0000      1.58
+++ emms.el     16 Aug 2005 11:31:54 -0000
@@ -85,7 +85,7 @@
 (defcustom emms-player-list nil
   "*List of players that EMMS can use.  You need to set this!"
   :group 'emms
-  :type '(repeat function))
+  :type '(repeat (symbol :tag "Player")))
 
 (defcustom emms-show-format "Currently playing: %s"
   "*The format to use for `emms-show'.
<#/patch>

--
Daniel Brockman <address@hidden>