weaselab/conflict-set
2
0
Fork 0
Code Issues 4 Packages Releases 13 Activity

Remove some redundant nullptr checks
All checks were successful
Tests / Clang total: 825, passed: 825
Details
Clang |Total|New|Outstanding|Fixed|Trend |:-:|:-:|:-:|:-:|:-: |0|0|0|0|:clap:
Details
Tests / Release [gcc] total: 825, passed: 825
Details
GNU C Compiler (gcc) |Total|New|Outstanding|Fixed|Trend |:-:|:-:|:-:|:-:|:-: |0|0|0|0|:clap:
Details
Tests / Release [gcc,aarch64] total: 824, passed: 824
Details
Tests / Coverage total: 823, passed: 823
Details
weaselab/conflict-set/pipeline/head This commit looks good
Details

Browse Source
This commit is contained in:
Andrew Noyes
2024-03-18 16:22:24 -07:00
parent a5710b8282
commit 83c7f66d67
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
ConflictSet.cpp
View File

@@ -2349,6 +2349,9 @@ Iterator firstGeq(Node *n, const std::span<const uint8_t> key) {
goto downLeftSpine; goto downLeftSpine;
} else { } else {
n = nextSibling(n); n = nextSibling(n);
if (n == nullptr) {
return {nullptr, 1};
}
goto downLeftSpine; goto downLeftSpine;
} }
} }
@@ -2379,9 +2382,6 @@ Iterator firstGeq(Node *n, const std::span<const uint8_t> key) {
} }
} }
downLeftSpine: downLeftSpine:
if (n == nullptr) {
return {nullptr, 1};
}
for (;;) { for (;;) {
if (n->entryPresent) { if (n->entryPresent) {
return {n, 1}; return {n, 1};