[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
- Re: New emms-info-native, (continued)
- Re: New emms-info-native, Fran Burstall (Gmail), 2021/02/20
- Re: New emms-info-native, Petteri Hintsanen, 2021/02/21
- Re: New emms-info-native, Fran Burstall (Gmail), 2021/02/21
- Re: New emms-info-native, Petteri Hintsanen, 2021/02/21
- Re: New emms-info-native, Fran Burstall (Gmail), 2021/02/21
- Re: New emms-info-native, Fran Burstall (Gmail), 2021/02/21
- Re: New emms-info-native, Petteri Hintsanen, 2021/02/22
- Re: New emms-info-native, Fran Burstall (Gmail), 2021/02/23
- Re: New emms-info-native, Petteri Hintsanen, 2021/02/23
- Re: New emms-info-native, Yoni Rabkin, 2021/02/21
- Re: New emms-info-native,
Petteri Hintsanen <=
- Re: New emms-info-native, Petteri Hintsanen, 2021/02/19
- Re:Re: New emms-info-native, tumashu, 2021/02/18
- Re:Re:Re: New emms-info-native, tumashu, 2021/02/18
- Re: New emms-info-native, Yoni Rabkin, 2021/02/18
- Re:Re: New emms-info-native, tumashu, 2021/02/18
- Re: New emms-info-native, Petteri Hintsanen, 2021/02/19
- Re:Re: New emms-info-native, tumashu, 2021/02/17
- Re: New emms-info-native, Yoni Rabkin, 2021/02/18
- Re:Re:Re: New emms-info-native, tumashu, 2021/02/18
- Re: New emms-info-native, Yoni Rabkin, 2021/02/18