[MERGE][0.13] Dynamic fulltext
John Arbash Meinel
john at arbash-meinel.com
Sun Nov 26 15:10:42 GMT 2006
Wouter van Heyst wrote:
...
>
> +1 to merging for 0.13. How hard would it be to check against the
> current fulltext instead of the original one?
Well, there are 2 code paths one for 'add_delta' and one for
'add_lines'. If you are using 'add_delta', you don't actually have a
full text to compare against. (It actually expands the delta back into a
fulltext if it has to, and then applies it).
However, there aren't any actual code paths that use 'add_delta'. So we
might pass an optional 'fulltext_size' parameter, which can be None.
In the short-term, though, I'll just merge it as is, and we can revisit
it later.
I submitted it with a NEWS entry, but otherwise unchanged.
John
=:->
>
> going to look up the progress bar changes again,
> Wouter van Heyst
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 254 bytes
Desc: OpenPGP digital signature
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20061126/6ac49acd/attachment.pgp
More information about the bazaar
mailing list