emms-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Emms-patches] [COMMIT] (emms-browser-mmatches-p): When performing searc


From: Jose Antonio Ortega Ruiz
Subject: [Emms-patches] [COMMIT] (emms-browser-mmatches-p): When performing searches, check whether
Date: Sun, 8 Mar 2009 21:50:44 +0100

This is the second patch addressing the null track fields issue.  In
this case, before trying to STRING-MATCH in a search, we check that
the track's field is non-null.
---
 lisp/emms-browser.el |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/lisp/emms-browser.el b/lisp/emms-browser.el
index d1c85d7..5286d75 100644
--- a/lisp/emms-browser.el
+++ b/lisp/emms-browser.el
@@ -1509,9 +1509,9 @@ included."
     (dolist (item search-list)
       (setq matched nil)
       (dolist (field (car item))
-        (when (string-match (cadr item)
-                            (emms-track-get track field ""))
-          (setq matched t)))
+        (let ((track-field (emms-track-get track field "")))
+          (when (and track-field (string-match (cadr item) track-field))
+            (setq matched t))))
       (unless matched
         (setq no-match t)))
     (not no-match)))
-- 
debian.1.5.6.1.19.ge6b2





reply via email to

[Prev in Thread] Current Thread [Next in Thread]