emms-help
[Top][All Lists]
Advanced

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

Re: New emms-info-native


From: Petteri Hintsanen
Subject: Re: New emms-info-native
Date: Mon, 22 Feb 2021 23:38:13 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux)

Yoni Rabkin <yoni@rabkins.net> writes:

> (Without actually checking) my guess is that it is the fact that:
>
>    (cdr '("composer")) => nil
>
> ...and so `string-trim-right' errors on getting nil as an input.
>
> So perhaps the call to `string-trim-right' can look like:
>
>     (string-trim-right (or (cdr field) ""))
>
> ...to protect from that. But I'll leave that to you to decide.

You're right.  I fixed it to set info-.... field only if the value is
not empty.  Now it works like emms-info-libtag and few others.  Going to
push changes soon.

Thanks,
Petteri



reply via email to

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