eisenwave / std-big-int / 24277854036

96%

Build Type

push

github

Committed by web-flow

Commit Message

Fix various allocation-related problems (#21)

This fixes a bunch of problems:
- Various places use raw pointers instead of the allocator's fancy `pointer`.
- `big_int` does rebinding, but allocator-aware containers like `vector` typically don't do that.
- `allocate_at_least` isn't used, which is wasteful.
- Unnecessary allocation in copy constructor.
- Unnecessary and weird special casing in `constexpr` which bypasses `allocate` via `new` expression.
- Undefined behavior due to `std::copy_n` to uninitialized storage (must be `uninitialized_copy_n`)

Coverage Stats

30 of 38 new or added lines in 1 file covered. (78.95%)

3 existing lines in 1 file now uncovered.

192 of 229 relevant lines covered (83.84%)

12.34 hits per line