[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: |
Sun, 21 Feb 2021 20:02:30 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux) |
I just pushed a new revision without emms-info-native--max-peek-size
checks. It still does a couple of other checks, but you shouldn’t see
excessive size errors anymore.
> My personal take is that trimming the whitespace is a good idea, if only
> because other info sources do it.
I added trailing whitespace trimming to all info-fields, including
Vorbis comments. They are text anyway.
> and similarly for emms-info-taglib. The native took 200 seconds and taglib
> 300 for the 14000 or so files! Looks like not shelling out 14000 times
> trumps the speed of C++ (at least on my setup where I suspect a lot of the
> time is spent reading the mp3's from the ntfs filesystem they live on).
That’s true, shelling incurs a heavy overhead.
I have compiled taglib shim as Emacs module so that it doesn’t need to
do any execs. It is, depending on caching conditions (I suppose), about
2-10x faster than emms-info-native.
Petteri
- Re: New emms-info-native, (continued)
- Message not available
- Re: New emms-info-native, Fran Burstall (Gmail), 2021/02/17
- Re: New emms-info-native, Yoni Rabkin, 2021/02/17
- Re: New emms-info-native, Petteri Hintsanen, 2021/02/18
- Re: New emms-info-native, Yoni Rabkin, 2021/02/18
- Re: New emms-info-native, Fran Burstall (Gmail), 2021/02/18
- Re: New emms-info-native, Yoni Rabkin, 2021/02/18
- Re: New emms-info-native, Petteri Hintsanen, 2021/02/19
- 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 <=
- 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, 2021/02/22
- 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