RFR 9: 8169556 : Wrap FileInputStream's native skip and available methods (original) (raw)

Chris Hegarty chris.hegarty at oracle.com
Thu Nov 10 19:17:04 UTC 2016


On 10 Nov 2016, at 19:09, Roger Riggs <Roger.Riggs at oracle.com> wrote:

Hi, Please review this request to enable instrumentation on the FileInputStream skip and available methods by wrapping native methods with java methods as proposed and contributed by Sunny Chan. webrev: http://cr.openjdk.java.net/~rriggs/webrev-fis-native-wrap-8169556/

Reviewed. Thanks Roger.

-Chris.

Issue: https://bugs.openjdk.java.net/browse/JDK-8169556

Regards, Roger p.s. Sorry for the long lapse Sonny

On 11/9/2016 10:02 PM, Chan, Sunny wrote: Hello all,

I proposed a patch to provide wrapping for some native methods in FileInputStream a while ago in May, and the patch has been reviewed but it has not been integrated. I have checked the patch again with the latest JDK9 dev builds and it still works correctly passing java.io regression tests so could I have someone to sponsor this and integrate this patch? Here is the previous threads from email archive: http://mail.openjdk.java.net/pipermail/core-libs-dev/2016-May/040901.html I have attached the patch again in this email for reference. Sunny Chan Executive Director Technology Goldman Sachs (Asia) L.L.C. | 39th Floor | The Center | 99 Queens Road Central | Hong Kong Email: sunny.chan at gs.com | Tel: +852 2978 6481 | Fax: +852 2978 0633 Learn more about Goldman Sachs GS.com<http://www.goldmansachs.com/> | Blog<http://www.goldmansachs.com/careers/blog/index.html> | LinkedIn<http://www.linkedin.com/company/goldman-sachs/careers> | YouTube<http://www.youtube.com/goldmansachs> | Twitter<http://www.twitter.com/goldmansachs> This message may contain information that is confidential or privileged. If you are not the intended recipient, please advise the sender immediately and delete this message. See http://www.gs.com/disclaimer/email for further information on confidentiality and the risks inherent in electronic communication.



More information about the core-libs-dev mailing list