Looks fine to me Chris.

- Michael

On 09/04/14 18:21, Chris Hegarty wrote:
Michael, Alan

There is/was an additional reference to the implementation specific package name in ContentHandler. It is now removed. I think this is the last one!

Latest specdiff:

http://cr.openjdk.java.net/~chegar/8039470/02/java/net/package-summary.html

On 09/04/14 14:33, Alan Bateman wrote:
On 09/04/2014 14:13, Chris Hegarty wrote:
Flip-flop!

Thinking again about this, you're right.

http://cr.openjdk.java.net/~chegar/8039470/URLConnection-report.html
http://cr.openjdk.java.net/~chegar/8039470/URLConnection-report_01.html

I could go further with this change, but I just want to remove the
reference to the sun.* package, that should not be in the public API.

Would it make sense to just remove the bulk of step 2 from the javadoc
and just specify that it uses a built-in implementation default if is
exists for the content type?

Step 2 is important for user defined handlers, but it may not be correctly placed. I moved it to ContentHandler

-Chris.


-Alan

Reply via email to