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

Reply via email to