casseopea2 / gperftools

Automatically exported from code.google.com/p/gperftools
BSD 3-Clause "New" or "Revised" License
1 stars 0 forks source link

malloc_extension_c.h #170

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
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 mark.sof...@gmail.com on 20 Sep 2009 at 3:14

GoogleCodeExporter commented 9 years ago
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 csilv...@gmail.com on 21 Sep 2009 at 1:37

GoogleCodeExporter commented 9 years ago

Original comment by csilv...@gmail.com on 21 Sep 2009 at 1:37

GoogleCodeExporter commented 9 years ago
This should be fixed in perftools 1.5, just released.

Original comment by csilv...@gmail.com on 20 Jan 2010 at 11:11