Re: RFR [13] 8153508: ContentHandler API contains link to private contentPathProp

2019-03-19 Thread Daniel Fuchs
Hi Chris, Looks good to me. @value: this is an interesting tag to know! Powerful, useful, maybe dangerous too? In this case the value in question is held in a *private* static final field. So {@link } was clearly wrong. But because the field is private some future maintainer might well be tempt

Re: RFR [13] 8153508: ContentHandler API contains link to private contentPathProp

2019-03-19 Thread Vyom Tiwari
Hi Chris, Change looks good to me. Thanks, Vyom On Tue, Mar 19, 2019 at 4:12 PM Chris Hegarty wrote: > This review request is to resolve a documentation regression in Java 9, > resulting from part of the changes for 8132478, where {@value > java.net.URLConnection#contentPathProp} was inadverten

RFR [13] 8153508: ContentHandler API contains link to private contentPathProp

2019-03-19 Thread Chris Hegarty
This review request is to resolve a documentation regression in Java 9, resulting from part of the changes for 8132478, where {@value java.net.URLConnection#contentPathProp} was inadvertently replaced with {@link java.net.URLConnection#contentPathProp}. The former will inline with the constant fiel