On 7/17/2014 2:41 AM, Daniel Fuchs wrote:
Hi Mandy,
minor:
JdepsTask.java:977 could be refactored to use putIfAbsent
ok.
I wonder whether the summary table at the end should contain a warning
for all the usage of jdk internals that have no replacement?
It's intentional. The list shown by jdeps is not an excessive list and only
includes the classes that I found are used in several products. When we find
more jDK internal APIs that are used and have a replacement (or RFE), I'll
update the wiki.
Otherwise looks good! I like the output much better :-)
Thanks
Mandy
best regards,
-- daniel
On 7/17/14 12:38 AM, Mandy Chung wrote:
Updated webrev:
http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8050804/webrev.01/
I plan to backport this to 8u40.
Mandy
On 7/16/14 8:09 AM, Mandy Chung wrote:
My first version prints a separate table of the replacements following
the new warning message at the end of the jdeps output. I like the
first version better than this version replacing rt.jar with "Use..."
message. I might be too anxious getting developers to take action
using the supported APIs when there is one and hence this version.
I'll update the webrev to print as a separate table.
thanks
Mandy
On 7/16/2014 6:10 AM, Daniel Fuchs wrote:
Hi Mandy,
here is a typical output - with names mangled to save
space:
s.u.l.p.LPA (rt.jar)
-> s.s.a.GP JDK internal API (Use j.s.PA @since 1.1)
-> s.u.c.CLDRLPA JDK internal API (rt.jar)
In the first dependency line, the archive name has been replaced
by the 'Use ...' message.
I wonder whether it would be better to keep the archive name
and print the use message as additional information, in order
to be 'script friendly' so that you could use things like
| grep 'rt.jar' on the output.
best regards,
-- daniel
On 7/16/14 1:35 AM, Mandy Chung wrote:
jdeps -jdkinternals flags use of JDK internal APIs. We have
created a
wiki page to keep track of the JDK internal APIs and its replacement:
https://wiki.openjdk.java.net/display/JDK8/Java+Dependency+Analysis+Tool
While this page will be updated when we identify any new ones
worthnoting, it'd still be useful for jdeps to suggest the
replacement
APIs of the known ones.
webrev at:
http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8050804/webrev.00/
Mandy
