AlisterT / openjazz

OpenJazz
GNU General Public License v2.0
275 stars 49 forks source link

fix: remove usage of malloc.h #100

Closed SMillerDev closed 11 months ago

SMillerDev commented 11 months ago

As identified in https://github.com/Homebrew/homebrew-core/pull/152682, OpenJazz doesn't build on macOS with these malloc references.

https://stackoverflow.com/a/56463133 says they're not standards compliant, so it should probably be removed.

carstene1ns commented 11 months ago

Good catch, however I am not sure if we should remove it from the Windows code path as well, since MSVC seems to ship this header. Hopefully they include it from stdlib.h.