Hi Chris,

Thanks for review, while my testing i discovered issue in the way we handle extended  socket options and standard socket options. I fixed it and updated one test as well.

I removed one redundant "if check" which i think not required. JPRT is clean with the changed code.

Please find the latest webrev(http://cr.openjdk.java.net/~vtewari/8190843/webrev0.2/index.html).

Thanks,

Vyom


On Thursday 23 November 2017 06:04 PM, Chris Hegarty wrote:
On 23 Nov 2017, at 09:20, vyom tewari <vyom.tew...@oracle.com> wrote:

Hi Chris,

Thanks for the review, please find the latest 
webrev(http://cr.openjdk.java.net/~vtewari/8190843/webrev0.1/index.html) where 
i modified the code to take care of SO_FLOW for Solaris.
I think this is fine Vyom, thanks.

-Chris.


Reply via email to