On 15/05/2013 12:46, Michael McMahon wrote:

updated here:

http://cr.openjdk.java.net/~michaelm/8010464/webrev.3/

Michael.

This looks much better.

In HttpURLConnection then odd formatting in the declaration of followRedirect0 (which probably needs a better name, maybe implFollowRedirect?) A couple of indentation issues that probably should be fixed too, L980 and L2951 for example.

I don't see any other issues.

-Alan

Reply via email to