Re: [PATCH -v2] Add #includes for C functions to jvmpi_oprofile

[Maynard J. <may...@us...>]

Suravee Suthikulpanit wrote:
> The patch is good :)
Thanks.  Patch applied.

-Maynard
> 
> Suravee
> 
> On Wed, 2009-10-14 at 15:46 -0500, Maynard Johnson wrote:
>> Newer libstdc++ header files no longer #include standard C header files.  So, for example, C++ source files should now include <cstring> if using any standard C string manipulation functions.  We had done this fix already for other files in the oprofile project, but missed agents/jvmpi/jvmpi_oprofile.cpp since it only gets compiled if configured to use a JDK that's 1.5 or older.
>>
>> *Suravee*, thanks again for catching the need for stdint.h.  Can you double-check that the attached patch is OK?
>>
>> -Maynard
>>
> 
>