Rev 3906: Remove an invalid assert. in http://bzr.arbash-meinel.com/branches/bzr/brisbane/gc_delta_index_room
John Arbash Meinel
john at arbash-meinel.com
Thu Mar 19 14:50:11 GMT 2009
At http://bzr.arbash-meinel.com/branches/bzr/brisbane/gc_delta_index_room
------------------------------------------------------------
revno: 3906
revision-id: john at arbash-meinel.com-20090319145002-8owsqmvodgkng272
parent: john at arbash-meinel.com-20090319144240-d5czr7b37zv010ip
committer: John Arbash Meinel <john at arbash-meinel.com>
branch nick: gc_delta_index_room
timestamp: Thu 2009-03-19 09:50:02 -0500
message:
Remove an invalid assert.
The assert is only valid if we grow at most by 1 level,
with the new limitation of forcing the hsize, we can
grow by a lot more than that. (f=>ff).
Rather than writing a new assert that fits all cases, just
remove it.
-------------- next part --------------
=== modified file 'bzrlib/diff-delta.c'
--- a/bzrlib/diff-delta.c 2009-03-19 14:41:53 +0000
+++ b/bzrlib/diff-delta.c 2009-03-19 14:50:02 +0000
@@ -591,13 +591,6 @@
entry < old_index->hash[j+1] && entry->ptr != NULL;
++entry) {
assert((entry->val & old_index->hash_mask) == j);
- // if (!(j == i || j + old_index->hash_mask + 1 == i)) {
- // fprintf(stderr, "Old hash entry %x"
- // " doesn't fit with new %x\n"
- // "old_mask: %x new_mask: %x\n",
- // j, i, old_index->hash_mask, hmask);
- // }
- assert(j == i || j + old_index->hash_mask + 1 == i);
if ((entry->val & hmask) == i) {
/* Any entries not picked up here will be picked up on the
* next pass.
More information about the bazaar-commits
mailing list