On 12/10/2015 01:14 PM, Mandy Chung wrote:
On Dec 10, 2015, at 7:53 AM, Sean Mullan <[email protected]>
wrote:
Max asked me to add the com.sun.jarsigner package to this bug,
which also has an unnecessary @Deprecated annotation. I also made a
few tweaks to the deprecation wording here and there, so a webrev
is probably easier to review than diffs now:
http://cr.openjdk.java.net/~mullan/webrevs/8144784/webrev.00/
Is there any other package-info.java beside these three packages? It
worths to have a quick scan on the source.
To me, it’s fine to keep the existing @deprecated comment in
com.sun.jarsigner.* classes since I assume you want to discourage new
classes to be added in that package.
Well, we are trying to "deprecate" use of the @Deprecated annotation on
packages, so I would prefer not to leave this in (also, I heard there
will eventually be a javac warning for this). Instead I will add a
sentence or comment that says new classes should not be added to the
package. Is that ok?
--Sean
