search

DevShiftTeam / AppShift-MemoryPool

A very fast cross-platform memory pool mechanism for C++ built using a data-oriented approach (3 to 24 times faster than regular new or delete, depending on operating system & compiler)
Apache License 2.0
215 stars 25 forks source link

free using in destructor #13

Closed xiaoye74 closed 2 years ago

xiaoye74 commented 2 years ago

When using the destructor, your implementation will call the class member function free instead of std:: free. Here I think std:: free should be used to release the memory correctly.

LessComplexity commented 2 years ago

Indeed, thanks for noticing! Will upload a fix :) More changes are coming, I added a garbage collection feature and currently working on making a thread-safe mode of the pool.