Subject: Re: bug#73891: [PATCH] Add ability to crossfade between songs in 'mpc'

From: john muhl <[email-masked]>
Date: Sat, 19 Oct 2024 16:40:27 -0500

Err. I guess it should have a NEWS entry too.

Sorry for the flood.

>From 984eccb981127b417fd883e03e4cd7ab38baa06c Mon Sep 17 00:00:00 2001
From: john muhl <[email-masked]>
Date: Sat, 28 Sep 2024 19:11:03 -0500
Subject: [PATCH] Add ability to crossfade between songs in 'mpc'

* lisp/mpc.el (mpc-cmd-crossfade):
(mpc-toggle-crossfade): New function.
(mpc-crossfade-time): New option.
(mpc-mode-menu): Add menu item to toggle crossfade.  (Bug#73891)
---
 etc/NEWS    |  5 +++++
 lisp/mpc.el | 18 ++++++++++++++++++
 2 files changed, 23 insertions(+)

diff --git a/etc/NEWS b/etc/NEWS
index a889528ab6a..cfc5a8e1785 100644
--- a/etc/NEWS
+++ b/etc/NEWS
@@ -582,6 +582,11 @@ a desktop notification when the song changes, using
 customized using the new user options 'mpc-notifications-title' and
 'mpc-notifications-body'.
 
+*** New user option 'mpc-crossfade-time'.
+When non-nil, MPC will crossfade between songs for the specified number
+of seconds.  Crossfading can be toggled using the command
+'mpc-toggle-crossfade' or from the MPC menu.
+
 
 * New Modes and Packages in Emacs 31.1
 
diff --git a/lisp/mpc.el b/lisp/mpc.el
index 9905e3aa554..01876bf230b 100644
--- a/lisp/mpc.el
+++ b/lisp/mpc.el
@@ -883,6 +883,11 @@ mpc-cmd-update
 (defun mpc-cmd-tagtypes ()
   (mapcar #'cdr (mpc-proc-cmd-to-alist "tagtypes")))
 
+(defun mpc-cmd-crossfade (&optional arg)
+  "Set duration of crossfade to `mpc-crossfade-time' or ARG seconds."
+  (mpc-proc-cmd (list "crossfade" (or arg mpc-crossfade-time))
+                #'mpc-status-refresh))
+
 ;; This was never integrated into MPD.
 ;; (defun mpc-cmd-download (file)
 ;;   (with-current-buffer (generate-new-buffer " *mpc download*")
@@ -918,6 +923,11 @@ mpc-data-directory
   "Directory where MPC.el stores auxiliary data."
   :type 'directory)
 
+(defcustom mpc-crossfade-time 3
+  "Number of seconds to crossfade between songs."
+  :version "31.1"
+  :type 'natnum)
+
 (defun mpc-data-directory ()
   (unless (file-directory-p mpc-data-directory)
     (make-directory mpc-data-directory))
@@ -1188,6 +1198,8 @@ mpc-mode-menu
      :selected (member '(single . "1") mpc-status)]
     ["Consume Mode" mpc-toggle-consume :style toggle
      :selected (member '(consume . "1") mpc-status)]
+    ["Crossfade Songs" mpc-toggle-crossfade :style toggle
+     :selected (alist-get 'xfade mpc-status)]
     "--"
     ["Add new browser" mpc-tagbrowser]
     ["Update DB" mpc-update]
@@ -2428,6 +2440,12 @@ mpc-toggle-shuffle
   (mpc-cmd-random
    (if (string= "0" (cdr (assq 'random (mpc-cmd-status)))) "1" "0")))
 
+(defun mpc-toggle-crossfade ()
+  "Toggle crossfading between songs."
+  (interactive)
+  (mpc-cmd-crossfade
+   (if (alist-get 'xfade mpc-status) "0" mpc-crossfade-time)))
+
 (defun mpc-stop ()
   "Stop playing the current queue of songs."
   (interactive)
-- 
2.46.2

john muhl <[email-masked]> writes:

> And the patch...
>
> [2. text/x-patch; 0001-Add-ability-to-crossfade-between-songs-in-mpc.patch]...
>
>
>
> john muhl <[email-masked]> writes:
>
>> Tags: patch
>>
>> Another small addition for mpc so that you can toggle the
>> crossfader.
>>
>> p.s. I noticed the use of (cdr (assq 'etc ...)) throughout and
>> guessed that was because mpc predates alist-get but if there is a
>> preference I can change it to match the other mpc-toggle-*
>> functions.