[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Emms-patches] [PATCH 1/2] * emms-browser.el (emms-browser-make-hash-by)
From: |
jao |
Subject: |
[Emms-patches] [PATCH 1/2] * emms-browser.el (emms-browser-make-hash-by): Check that field is present before attempting to add entries. |
Date: |
Sun, 8 Mar 2009 21:50:43 +0100 |
From: Jose Antonio Ortega Ruiz <address@hidden>
Hi,
These two patches address an issue i'm encountering when using lastfm
integration
and mpd as the player. After requesting streaming from last.fm, a new track gets
added to EMMS' database, which is named after the streaming URL but has no other
fields. Thus, when adding entries or making searches, checking the the field
at hand is non-nil is needed. This patch checks that for
emms-browser-make-hash-by.
Thanks!
jao
---
lisp/emms-browser.el | 9 +++++----
1 files changed, 5 insertions(+), 4 deletions(-)
diff --git a/lisp/emms-browser.el b/lisp/emms-browser.el
index dd1d316..1c0b699 100644
--- a/lisp/emms-browser.el
+++ b/lisp/emms-browser.el
@@ -656,10 +656,11 @@ compilations, etc."
'emms-browser-filter-tracks-hook track)
(setq field
(emms-browser-get-track-field track type))
- (setq existing-entry (gethash field hash))
- (if existing-entry
- (puthash field (cons track existing-entry) hash)
- (puthash field (list track) hash))))
+ (when field
+ (setq existing-entry (gethash field hash))
+ (if existing-entry
+ (puthash field (cons track existing-entry) hash)
+ (puthash field (list track) hash)))))
emms-cache-db)
hash))
--
1.6.2
- [Emms-patches] [PATCH 1/2] * emms-browser.el (emms-browser-make-hash-by): Check that field is present before attempting to add entries.,
jao <=