[PATCH][MERGE] Improvements to is_ignored
John A Meinel
john at arbash-meinel.com
Wed Jan 11 04:35:07 GMT 2006
Martin Pool wrote:
> On 10 Jan 2006, John A Meinel <john at arbash-meinel.com> wrote:
>> Jan Hudec wrote:
>
>> Here are my results:
>> time bzr status
>> real 0m8.374s
>> user 0m7.840s
>> sys 0m0.404s
>>
>> time bzr.ignore/bzr status
>> real 0m5.497s
>> user 0m4.988s
>> sys 0m0.412s
>
> Very nice.
>
>> So it was able to shave a lot of the time off. This is with 71 ignore
>> patterns, and 805 ignored files.
>>
>> I also found an interesting problem if you don't use (?:), specifically:
>> bzr: ERROR: exceptions.AssertionError: sorry, but this version only
>> supports 100 named groups
>> at /usr/lib/python2.4/sre_compile.py line 506
>> in compile
>
> So we should add a test with enough patterns to provoke this, calling
> both bzr status and bzr ignored.
>
I agree.
Also, I found a small advantage to using the pattern (a$)|(b$)|(c$)
versus doing ((a)|(b)|(c))$.
Which is nice, because it is easier to do.
John
=:->
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 249 bytes
Desc: OpenPGP digital signature
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20060110/11f96307/attachment.pgp
More information about the bazaar
mailing list