emms-help
[Top][All Lists]
Advanced

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

Re: [emms-help] Experiment with speeding up Emms


From: Yoni Rabkin Katzenell
Subject: Re: [emms-help] Experiment with speeding up Emms
Date: Thu, 19 Oct 2006 05:39:10 +0200
User-agent: Gnus/5.110006 (No Gnus v0.6) Emacs/22.0.50 (gnu/linux)

Jorgen Schaefer <address@hidden> writes:

> Yoni Rabkin Katzenell <address@hidden> writes:
>
>> If it helps anyone to locate where to make the changes:
>>
>> $:~/elisp/emms$ darcs whatsnew
>> {
>> hunk ./emms-info-mp3info.el 79
>> -             (string-match "\\.[Mm][Pp]3\\'" (emms-track-name track)))
>> +             (string-match "\.\(?:\(?:M[Pp]\|mp\)3\)" (emms-track-name 
>> track)))
>> hunk ./emms-info-ogginfo.el 50
>> -             (string-match "\\.[Oo][Gg][Gg]\\'" (emms-track-name track)))
>> +             (string-match "\.\(?:O\(?:GG\|gg\)\|ogg\)" (emms-track-name 
>> track)))
>> }
>
> These regular expressions won't match many file names, which
> explains why you notice the speedup. 

Makes sense!

> You need to escape the
> parentheses twice, once for the string, and once for the regular
> expression engine.
>
> I.e.
> "\\.\\(?:\\(?:M[Pp]\\|mp\\)3\\)"
> "\\.\\(?:O\\(?:GG\\|gg\\)\\|ogg\\)"
>
> Sorry for not noticing this when you asked on the channel.
>

Well... with the above and another change I've now managed to completely
bork my local emms, so I didn't even notice them not matching!

Good thing I did not try and make it into a patch. Or darcs emms would
have been borkened everywhere.

Thanks for trying it out.

-- 
   "Cut your own wood and it will warm you twice"




reply via email to

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