Hi Christoph,

It looks good.
Thank you taking care about it!

Yes, the libdt_shmem is for Windows only.

Thanks,
Serguei


On 12/7/17 10:13, Chris Plummer wrote:
Looks good to me. I can sponsor the push for you.

thanks,

Chris

On 12/7/17 7:21 AM, Langer, Christoph wrote:

Hi,

 

please review the first extracted patch from 8192978. This one is about correcting the jlong format string in src/jdk.jdi/share/native/libdt_shmem/shmemBase.c.

 

Bug: https://bugs.openjdk.java.net/browse/JDK-8193183

WebRev: http://cr.openjdk.java.net/~clanger/webrevs/8193183.0/

 

I believe the libdt_shmem is only built on windows.

 

Thanks,

Christoph

 



Reply via email to