Use check_symbol_exists() to check if cfree() is available (v2)

Authored by beischer on Aug 30 2017, 8:07 PM.



This is a follow-up of including the requested changes. Thanks to Daniel Vratil for the initial work.

Test Plan

heaptrack compiles on Arch Linux x86_64 with glibc 2.26.

Diff Detail

R45 Heaptrack
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.
beischer created this revision.Aug 30 2017, 8:07 PM
beischer updated this revision to Diff 18985.Aug 31 2017, 6:18 AM

Added link to bug 383889.

mwolff accepted this revision.Aug 31 2017, 7:51 AM

thanks, I'll fix this up and merge this


you could use cmakedefine01 here and then #if ... in the c++ code, to ensure we don't typo anything (since #if defined(HAVE_CFRE) e.g. would compile but be wrong

This revision is now accepted and ready to land.Aug 31 2017, 7:51 AM
This revision was automatically updated to reflect the committed changes.