Dfuds implementation#46
Conversation
| /** | ||
| * @brief Returns the size of the tree | ||
| */ | ||
| size_t size() const { return num_bits_ / 2; } |
There was a problem hiding this comment.
WARNING: size() returns an incorrect node count
num_bits_ is initialized as 2 * tree_size - 1, so num_bits_ / 2 evaluates to tree_size - 1 for all non-empty trees. This under-reports the actual tree size by one and can break callers relying on size() for bounds or traversal logic.
| "${BUILD_DIR}/unittests" | ||
| "${BUILD_DIR}/excess_positions_tests" | ||
| "${BUILD_DIR}/louds_tree_tests" | ||
| "${BUILD_DIR}/dfus_tree_tests" |
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Files Reviewed (1 files)
Reviewed by gpt-5.3-codex · 123,918 tokens |
a4a5921 to
a6575a7
Compare
a6575a7 to
c447002
Compare
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #46 +/- ##
==========================================
- Coverage 87.29% 87.11% -0.18%
==========================================
Files 12 14 +2
Lines 3117 3252 +135
Branches 612 626 +14
==========================================
+ Hits 2721 2833 +112
- Misses 260 263 +3
- Partials 136 156 +20
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
No description provided.