[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