plasma-umass / Mesh

A memory allocator that automatically reduces the memory footprint of C/C++ applications.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Miniheap pairs are not properly checked for isMeshingCandidate()

Corillian opened this issue · comments

While meshing all size classes the meshFound callback only checks one of the miniheaps for each miniheap pair if it is a meshing candidate.

if (std::get<0>(miniheaps)->isMeshingCandidate() && std::get<0>(miniheaps)->isMeshingCandidate())

should be:

if (std::get<0>(miniheaps)->isMeshingCandidate() && std::get<1>(miniheaps)->isMeshingCandidate())

wow, great find. It looks like I fat-fingered this in a refactor at the end of 2017:
816ba1f#diff-5e53b77d66403f5bb04fab8563dc8e80L394-L396

thanks @Corillian ! amazing find