Casey Marshall wrote: > Jeroen Frijters wrote: > > Hi Casey, > > > > I would like to remove the dependence on VMChannel from Socket and > > ServerSocket (as this breaks the layering). Do you agree with this > > patch? > > > > I think this should be OK. Refresh my memory, though: in ServerSocket > and Socket `impl' is created in a constructor, right? And it > is only set to null in close, right? Because otherwise testing > `impl == null' won't make a correct definition of isClosed().
Yes, impl is always set to a non-null value in the constructor. Thanks. I will check this in. Regards, Jeroen