Re: Request for Review 6855335

2009-06-29 Thread Michael McMahon
Christopher Hegarty - Sun Microsystems Ireland wrote: On 29/06/2009 11:32, Michael McMahon wrote: Christopher Hegarty - Sun Microsystems Ireland wrote: Hi Michael, Jessie, I need a code review for some SCTP changes that were found during testing. CR 6855335: Several changes in the SCTP im

Re: Request for Review 6855335

2009-06-29 Thread Christopher Hegarty - Sun Microsystems Ireland
On 29/06/2009 11:32, Michael McMahon wrote: Christopher Hegarty - Sun Microsystems Ireland wrote: Hi Michael, Jessie, I need a code review for some SCTP changes that were found during testing. CR 6855335: Several changes in the SCTP implementation. Webrev: http://cr.openjdk.java.net/~ch

Re: Request for Review 6855335

2009-06-29 Thread Michael McMahon
Christopher Hegarty - Sun Microsystems Ireland wrote: Hi Michael, Jessie, I need a code review for some SCTP changes that were found during testing. CR 6855335: Several changes in the SCTP implementation. Webrev: http://cr.openjdk.java.net/~chegar/6855335/webrev.00/webrev/ 1) SctpMultiC

Request for Review 6855335

2009-06-29 Thread Christopher Hegarty - Sun Microsystems Ireland
Hi Michael, Jessie, I need a code review for some SCTP changes that were found during testing. CR 6855335: Several changes in the SCTP implementation. Webrev: http://cr.openjdk.java.net/~chegar/6855335/webrev.00/webrev/ 1) SctpMultiChannel.branch returns null. branch was not implemented