From: john muhl <jm@pub.pink>
To: 73891@debbugs.gnu.org
Cc: Stefan Monnier <monnier@iro.umontreal.ca>
Subject: bug#73891: [PATCH] Add ability to crossfade between songs in 'mpc'
Date: Sat, 19 Oct 2024 16:27:51 -0500 [thread overview]
Message-ID: <87plnvvl48.fsf@pub.pink> (raw)
In-Reply-To: <87y12jvl9n.fsf@pub.pink> (john muhl's message of "Sat, 19 Oct 2024 16:24:36 -0500")
[-- Attachment #1: Type: text/plain, Size: 18 bytes --]
And the patch...
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-Add-ability-to-crossfade-between-songs-in-mpc.patch --]
[-- Type: text/x-patch, Size: 2193 bytes --]
From e2855c6493d8d0ab06a97452b567e87701754601 Mon Sep 17 00:00:00 2001
From: john muhl <jm@pub.pink>
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)
---
lisp/mpc.el | 18 ++++++++++++++++++
1 file changed, 18 insertions(+)
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
[-- Attachment #3: Type: text/plain, Size: 334 bytes --]
john muhl <jm@pub.pink> 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.
next prev parent reply other threads:[~2024-10-19 21:27 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-10-19 21:24 bug#73891: [PATCH] Add ability to crossfade between songs in 'mpc' john muhl
2024-10-19 21:27 ` john muhl [this message]
2024-10-19 21:40 ` john muhl
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87plnvvl48.fsf@pub.pink \
--to=jm@pub.pink \
--cc=73891@debbugs.gnu.org \
--cc=monnier@iro.umontreal.ca \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).