<!DOCTYPE html>
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body>
<p>Hello Eirik,</p>
<p>Like Claes noted, this is an internal class and the use of either
of @Deprecated annotation or the @deprecated javadoc taglet is
unncessary. Furthermore, like you note, it looks like this method
is not used anywhere within the repo (not even in tests). So I
think we can remove it. This method was the only way to set the
private "defaultAuth" field (to a non-null value). This then means
that the "defaultAuth" is currently not set by any code and while
we are removing that setDefaultAuthenticator method, we should
also remove this field and references to this field within the
same internal class. This field and its usage have already been
marked with a code comment which says that it should be removed in
JDK2.0.</p>
<p>-Jaikiran<br>
</p>
<div class="moz-cite-prefix">On 11/03/24 2:14 am, Eirik Bjørsnøs
wrote:<br>
</div>
<blockquote type="cite"
cite="mid:CA+pBWhte+_HNociZqYco35cq=zhUSevB545T=daeCsWT-G5rSQ@mail.gmail.com">
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
<div dir="ltr">
<div>Hi,</div>
<div><br>
</div>
<div>While investigating methods annotated as @Deprecated, but
with missing @deprecated javadoc tags, I discovered that
sun.net.w.p.h.HttpURLConnection.setDefaultAuthenticator has a
trailing period after its @deprecated tag, causing javac to
not recognize it as such.</div>
<div><br>
</div>
<div>Some observations:</div>
<div><br>
</div>
<div>o The method has had this invalid <a
class="gmail_plusreply"
id="m_6761717428841858628gmail-plusReplyChip-3"
moz-do-not-send="true">@deprecated</a> note since the
initial load </div>
<div>o It was marked @Deprecated in Java 8</div>
<div>o It seems to be unused across the OpenJDK code base,
including tests</div>
<div>o The class seems internal, given that its package is
exported only to java.security.jgss</div>
<div><br>
</div>
<div>What is the right step forward here?</div>
<div><br>
</div>
<div>0: Do nothing, nobody complained so far</div>
<div>1: Fix the @deprecated tag issue</div>
<div>2: Mark the @Deprecated forRemoval=true</div>
<div>3: Remove the method and the private static field now</div>
<div>4: Something else</div>
<div><br>
</div>
<div>Thanks,</div>
<div>Eirik.<br>
<a class="gmail_plusreply" moz-do-not-send="true"><br>
</a></div>
</div>
</blockquote>
</body>
</html>