Re: fix for UI interface issues in elms-streams.el

[Gerard Vermeulen]

On Apr 22, 2022, at 9:50 AM, Omar Polo <op@omarpolo.com> wrote:

Gerard Vermeulen <gerard.vermeulen@neel.cnrs.fr> wrote:

Hello,

the attached patch moves two misplaced documentation strings
to their intended positions and adds a magic autoload comment
to the emms-stream function.

Best regards — Gerard Vermeulen

replying off-list: you forgot to attach the patch ;)

Thanks Omar, I see the patch on the web interface, but here it is:

diff --git a/emms-streams.el b/emms-streams.el

index bb8a29c..c223b95 100644

--- a/emms-streams.el

+++ b/emms-streams.el

@@ -150,13 +150,14 @@

 ;;; ------------------------------------------------------------------

 (defun emms-streams-install ()

-  (interactive)

   "Install the built-in streams file."

+  (interactive)

   (emms-streams-install-file emms-streams-file))

+;;;###autoload

 (defun emms-streams ()

-  (interactive)

   "Create or switch to the built-in streaming audio playlist."

+  (interactive)

   (when (and (not (file-exists-p emms-streams-file))

      (y-or-n-p "Emms' built-in streams file hasn't been installed yet. Install it now?"))

     (emms-streams-install))