FW: RFR(XS): 8160565: AIX port: cleanup of libo4 wrapper stub

[Volker Simonis]

Hi Christoph,

overall the change looks good.
Can you please just remove the logic from
libo4::removeEscapeMessageFromJoblogByContext() in libo4.cpp. As this
is a stub anyway, I think it is perfectly fine to simply return false.
There's no need to post a new webrev for that change.

Thank you and best regards,
Volker


On Thu, Jun 30, 2016 at 2:31 PM, Langer, Christoph
<christoph.langer at sap.com> wrote:
> Forgot to include hotspot-runtime mailing list...
>
> -----Original Message-----
> From: Langer, Christoph
> Sent: Donnerstag, 30. Juni 2016 11:22
> To: Lindenmaier, Goetz <goetz.lindenmaier at sap.com>
> Cc: ppc-aix-port-dev at openjdk.java.net
> Subject: RE: RFR(XS): 8160565: AIX port: cleanup of libo4 wrapper stub
>
> Hi Goetz,
>
> I added this new structure, too: http://cr.openjdk.java.net/~clanger/webrevs/8160565.2/
>
> Thanks
> Christoph
>
>
>> -----Original Message-----
>> From: Lindenmaier, Goetz
>> Sent: Donnerstag, 30. Juni 2016 08:24
>> To: Langer, Christoph <christoph.langer at sap.com>; ppc-aix-port-
>> dev at openjdk.java.net
>> Subject: RE: RFR(XS): 8160565: AIX port: cleanup of libo4 wrapper stub
>>
>> Hi Christoph,
>>
>> thanks for doing these changes, they look good.
>> Could you please also include the fixes we did in
>> Libperfstat_aix.hpp?
>>
>> Thanks,
>>   Goetz.
>>
>> > -----Original Message-----
>> > From: ppc-aix-port-dev [mailto:ppc-aix-port-dev-
>> > bounces at openjdk.java.net] On Behalf Of Langer, Christoph
>> > Sent: Mittwoch, 29. Juni 2016 18:28
>> > To: ppc-aix-port-dev at openjdk.java.net
>> > Subject: RFR(XS): 8160565: AIX port: cleanup of libo4 wrapper stub
>> >
>> > Hi,
>> >
>> >
>> >
>> > I made a little cleanup in the libo4 wrapper stubs to ease merging.
>> >
>> >
>> >
>> > Webrev: http://cr.openjdk.java.net/~clanger/webrevs/8160565.1/
>> >
>> > Bug: https://bugs.openjdk.java.net/browse/JDK-8160565
>> > <https://bugs.openjdk.java.net/browse/JDK-8160565>
>> >
>> >
>> >
>> > Thanks
>> >
>> > Christoph
>> >
>> >
>