[sctp-dev] Request for review: 6916890
Alan Bateman
Alan.Bateman at Sun.COM
Mon Jan 18 06:03:54 PST 2010
Christopher Hegarty -Sun Microsystems Ireland wrote:
>
>
> Alan Bateman wrote:
>> .....
>> In SctpChannelImpl it looks like you can move lines 985-988 to after
>> you've checked if the user's buffer is direct or not - that would
>> avoid fetching the position and limit twice for the direct buffer
>> case. Same comment for SctpMultiChannelImpl. Otherwise the
>> implementation change looks right to me. For the tests, one
>> suggestion is to go a step further and verify the received message.
>> Without that it only checks the return value and that the buffer
>> position has been updated.
>
> Thanks Alan for the review and the comments. I made the changes and
> updated the webrev:
> http://cr.openjdk.java.net/~chegar/6916890/webrev.1/
Looks OK to me.
-Alan.
More information about the sctp-dev
mailing list