FYI, I plan to fix this by way of:
https://bugs.openjdk.java.net/browse/JDK-8186217
Remove erroneous @hidden JavaDoc tag from
java.util.Properties.replace(Object, Object, Object)
-Brent
On 8/14/17 10:02 AM, Brent Christian wrote:
Hi, Max
This tag snuck in by mistake. Before pushing the fix for JDK-8029891,
we looked into ways to avoid addition of JavaDoc for all the
trivial method overrides. I tried adding @hidden tags,
but they didn't do what we wanted, so they were taken out. Except one -
sorry about that.
-Brent
On 8/14/17 3:36 AM, Weijun Wang wrote:
FWIW, there is one remaining issue to be addressed: it is a broken
link in Provider.html.
It comes from Provider.replace(Object, Object, Object) which overrides
Properties.replace(Object, Object, Object), but for some reason, that
method is
marked @hidden in Properties, causing a broken link in Provider. That
needs
to be investigated, but I don't want to block these accessibility fixes.
I don't understand why there is a @hidden in Properties.java. There
are many other methods nearby with a similar structure and I cannot
see how this one is different.
The tag was added at
http://hg.openjdk.java.net/jdk9/dev/jdk/rev/01a8615439f0#l2.349
in the fix for
8029891 : Deadlock detected in
java/lang/ClassLoader/deadlock/GetResource.java
+
+ /** @hidden */
+ @Override
+ public synchronized boolean replace(Object key, Object oldValue,
Object newValue) {
+ return map.replace(key, oldValue, newValue);
+ }
*Brent*: Is this method any special?
Thanks
Max
-- Jon
