Review Request for 8005428: Update jdeps to read the same profile information as by javac

Alan Bateman Alan.Bateman at oracle.com
Thu Mar 14 15:20:38 UTC 2013


On 13/03/2013 20:23, Mandy Chung wrote:
> This fix removes the temporary workaround for jdeps to print the 
> profile information.  The profiles information is embedded in ct.sym 
> generated during the jdk build.  jdeps will read ct.sym and parse the 
> annotation to determine which profile a class belongs to.   I have 
> extended the test to  test the profile information.
>
> Webrev at:
>    http://cr.openjdk.java.net/~mchung/jdk8/webrevs/8005428/webrev.00/
I went through the webrev and it looks good to me (educational too as I 
only had a superficial understanding of the ct.sym before this).

In initProfiles then this might be less code to get the right path:

     Path home = Paths.get(System.getProperty("java.home"));
     if (home.endsWith("jre"))
         home = home.getParent();
     Path ctsym  = home.resolve("lib").resolve("ct.sym");

For consistency then the constants with the annotation types should 
probably be uppercase.

You've moved profiles.properties into the test tree to be used by the 
test. I think that is okay for the types that are tested but one thing 
to mention is that these properties file is out of date in a few places now.

-Alan.



More information about the core-libs-dev mailing list