Simplify addWriteRange slightly
This commit is contained in:
@@ -2936,36 +2936,23 @@ void addWriteRange(Node *&root, std::span<const uint8_t> begin,
|
|||||||
Node **useAsRoot =
|
Node **useAsRoot =
|
||||||
insert(&root, begin.subspan(0, lcp), writeVersion, tls, impl);
|
insert(&root, begin.subspan(0, lcp), writeVersion, tls, impl);
|
||||||
|
|
||||||
int consumed = lcp;
|
begin = begin.subspan(lcp, begin.size() - lcp);
|
||||||
|
end = end.subspan(lcp, end.size() - lcp);
|
||||||
begin = begin.subspan(consumed, begin.size() - consumed);
|
|
||||||
end = end.subspan(consumed, end.size() - consumed);
|
|
||||||
|
|
||||||
auto *beginNode = *insert(useAsRoot, begin, writeVersion, tls, impl);
|
auto *beginNode = *insert(useAsRoot, begin, writeVersion, tls, impl);
|
||||||
|
|
||||||
const bool insertedBegin = !beginNode->entryPresent;
|
|
||||||
|
|
||||||
addKey(beginNode);
|
addKey(beginNode);
|
||||||
beginNode->entryPresent = true;
|
if (!beginNode->entryPresent) {
|
||||||
|
|
||||||
if (insertedBegin) {
|
|
||||||
++tls->accum.entries_inserted;
|
++tls->accum.entries_inserted;
|
||||||
auto *p = nextLogical(beginNode);
|
auto *p = nextLogical(beginNode);
|
||||||
beginNode->entry.rangeVersion =
|
beginNode->entry.rangeVersion =
|
||||||
p == nullptr ? tls->zero : std::max(p->entry.rangeVersion, tls->zero);
|
p == nullptr ? tls->zero : std::max(p->entry.rangeVersion, tls->zero);
|
||||||
beginNode->entry.pointVersion = writeVersion;
|
beginNode->entryPresent = true;
|
||||||
}
|
}
|
||||||
assert(writeVersion >= beginNode->entry.pointVersion);
|
|
||||||
beginNode->entry.pointVersion = writeVersion;
|
beginNode->entry.pointVersion = writeVersion;
|
||||||
|
|
||||||
auto *endNode = *insert(useAsRoot, end, writeVersion, tls, impl);
|
auto *endNode = *insert(useAsRoot, end, writeVersion, tls, impl);
|
||||||
|
|
||||||
const bool insertedEnd = !endNode->entryPresent;
|
|
||||||
|
|
||||||
addKey(endNode);
|
addKey(endNode);
|
||||||
endNode->entryPresent = true;
|
if (!endNode->entryPresent) {
|
||||||
|
|
||||||
if (insertedEnd) {
|
|
||||||
++tls->accum.entries_inserted;
|
++tls->accum.entries_inserted;
|
||||||
auto *p = nextLogical(endNode);
|
auto *p = nextLogical(endNode);
|
||||||
endNode->entry.pointVersion =
|
endNode->entry.pointVersion =
|
||||||
@@ -2975,6 +2962,7 @@ void addWriteRange(Node *&root, std::span<const uint8_t> begin,
|
|||||||
beginNode = *useAsRoot;
|
beginNode = *useAsRoot;
|
||||||
assert(beginNode->entryPresent);
|
assert(beginNode->entryPresent);
|
||||||
}
|
}
|
||||||
|
endNode->entryPresent = true;
|
||||||
}
|
}
|
||||||
endNode->entry.rangeVersion = writeVersion;
|
endNode->entry.rangeVersion = writeVersion;
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user