>>>>> "David" == David Daney <[EMAIL PROTECTED]> writes:

David> This patch is a partial rewrite of the
David> gnu/java/net/protocol/http package.

Awesome!

I gave this a quick read-through.  I want to read it again, and I'd
like it if Chris commented on it.

David>      Map ret = new LinkedHashMap();

Might as well declare locals with the most specific type.
Not a big deal, but in this case avoids some needless invokeinterface
calls.

David> -    ret.put("", Collections.singletonList(getStatusLine(response)));
David> +    ret.put(null, Collections.singletonList(getStatusLine(response)));

This, btw, is PR 24104.

Tom


_______________________________________________
Classpath-patches mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/classpath-patches

Reply via email to