Skip to content

Commit

Permalink
Improve hq.rs
Browse files Browse the repository at this point in the history
Attention: there is clearly a bug in the code when it uses `v.wrapping_add(1) > 0` when `v` is a `usize`-- because the only way for this condition to be false is for `v` to be `usize::MAX` -- and this will not be consistent between x32 and x64 versions. Moreover, there is even a case of `wrapping_add(2)` -- which would only pass if the original is `usize::MAX-1`.

Given all this, this expression does not make any sense, and should be replaced with `m.alloc_cell(num_bytes + 2)`:

```rust
if num_bytes + 2 > 0 {
    m.alloc_cell(num_bytes + 2)
} else {
    AllocF::AllocatedMemory::default()
}
```
  • Loading branch information
nyurik committed May 21, 2024
1 parent 8606c1f commit 0ec80e0
Showing 1 changed file with 233 additions and 288 deletions.
Loading

0 comments on commit 0ec80e0

Please sign in to comment.