Need reviewer for 6894633: NetHooks hould not require provider to be present (sol)

Jean-Christophe Collet Jean-Christophe.Collet at Sun.COM
Fri Oct 23 07:21:11 PDT 2009


Christopher Hegarty - Sun Microsystems Ireland wrote:
> The changes look fine.
>

Same here. Looks fine.

> -Chris.
>
> On 23/10/2009 14:57, Alan Bateman wrote:
>>
>> I need a reviewer for a small fix to sun.net.NetHooks. On Solaris, 
>> the code incorrectly assumes that the sdp provider is always present. 
>> The loadProvider method should return null if not present. The diffs 
>> are attached.
>>
>> Thanks,
>> Alan.
>>
>>
>> --- a/src/solaris/classes/sun/net/NetHooks.java
>> +++ b/src/solaris/classes/sun/net/NetHooks.java
>> @@ -81,7 +81,7 @@ public final class NetHooks {
>>                     try {
>>                         c = (Class<Provider>)Class.forName(cn, true, 
>> null);
>>                     } catch (ClassNotFoundException x) {
>> -                        throw new AssertionError(x);
>> +                        return null;
>>                     }
>>                     try {
>>                         return c.newInstance();
>>




More information about the net-dev mailing list