Chris Hegarty wrote: > : >> The webrev is here: >> http://cr.openjdk.java.net/~alanb/6431344/webrev > > This change mainly looks fine. One minor comment; do we really need > the nested try statements in transferFromFileChannel? Can the catch > not simply match the first try and remove the second? That would be better - thanks! I've updated the webrev. -Alan