ltalloc icon indicating copy to clipboard operation
ltalloc copied to clipboard

Continuing development

Open jlaumon opened this issue 6 years ago • 1 comments

Hello,

ltalloc is great: it's only one file, it's very fast when using lots of threads and it's relatively simple.

But it's not perfect, notably I find it annoying to port to new platforms because all the macros have to be modified inside ltalloc.cc and there are ifdefs _WIN32 scattered in the code. It's also lacking tests (there's a little bit of code here, but it's not much), and there's no benchmark to check for regressions (but maybe some code from malloc-survey could be reused).

I'd like to improve some of that by:

  • making all the macros overridable (ie. only define if not already defined) and adding an optional config include (like ImGui does)
  • adding macros to abstract the remaining ifdefed features (eg. ptrie vs VirtualQuery)
  • adding at least some tests with a Travis or Appveyor integration

And maybe in the future making actual changes to the initial design (for example, I would love to have an API that takes heap objects, like nedpools in nedmalloc).

This seems to be the most recent, most active ltalloc repo, so I'm asking here:

  • Is there anybody interested in those changes? (and in discussing them, or even participating)
  • Are you (@r-lyeh / @r-lyeh-archived) interested in getting pull requests here?

jlaumon avatar Jul 08 '18 09:07 jlaumon

Hey thanks for the heads up. PRs are always welcome. I merge often as much as I can. Dont hesitate to work as needed :)

r-lyeh avatar Jul 09 '18 13:07 r-lyeh