[FixedVector/FixedDeque] Fix double-destroy bug in `erase`

teslamotors / fixed-containers

C++ Fixed Containers

MIT License

352 stars 31 forks source link

[FixedVector/FixedDeque] Fix double-destroy bug in `erase` #109

Closed Quuxplusone closed 2 months ago

Quuxplusone commented 2 months ago

Also rename emplace_move_backward to uninitialized_relocate_backward, introduce (forward) uninitialized_relocate, and complete the circle by resuming the use of uninitialized_relocate in erase.

alexkaratarakis commented 2 months ago

Sorry for the delay getting to this. Thank you for fix! I amended to also apply the same fix and add the new tests for FixedDeque/FixedCircularDeque/FixedList