malloc_extension_c.h

[Google Code Exporter]

Hello. In both 32 and 64 bit Linux builds of release 1.4 I have noticed
some problems with the header file malloc_extension_c.h

It is missing the definition:

static const int kMallocHistogramSize = 64;

which is present in malloc_extension.h

Also a number of exported function prototypes include the C++ bool type.
For example:

PERFTOOLS_DLL_DECL bool MallocExtension_VerifyAllMemory();

This is also missing a void argument declaration (as with a number of other
functions) but that's not very important.

I worked around this by defining:

typedef int bool;

Best wishes, Mark

Original issue reported on code.google.com by [e-mail] on 20 Sep 2009 at 3:14

[Google Code Exporter]

These are all good points!  Sounds like we need to beef up our tests of the
malloc_extension_c.h -- pehraps compile with -ansi or -pedantic or some such to 
make
sure we don't depend on any gcc extensions.

I'll have this fixed up for the next release.

Original comment by [e-mail] on 21 Sep 2009 at 1:37

• Changed state: Started

[Google Code Exporter]

Original comment by [e-mail] on 21 Sep 2009 at 1:37

• Added labels: Priority-Medium, Type-Defect

[Google Code Exporter]

This should be fixed in perftools 1.5, just released.

Original comment by [e-mail] on 20 Jan 2010 at 11:11

• Changed state: Fixed