RFR: 8009428 and 8009429 - Profile related fixes and clean ups

Alan Bateman Alan.Bateman at oracle.com
Mon Mar 11 08:54:41 UTC 2013

On 11/03/2013 01:24, David Holmes wrote:
> I had overlooked the need to update the ct.sym creation tool to 
> recognize the new syntax in the profile spec file. That process also 
> uncovered a few bugs in the listing that needed correcting.
> The javadoc generation of compact profile information is not quite 
> right, but that will be handled in a seperate bug.
> Updated webrevs at:
> http://cr.openjdk.java.net/~dholmes/8009428_8009429.v2/
The changes to the top-level and jdk repo look good to me. As per my 
original comments, I'm happy to see profile-rtjar-includes.txt changed 
to be much easier to maintain. The assumption that jdk.** is in compact1 
is okay for now but I'm sure that will need to be changed when that 
namespace is used more.

For the intro comment in profile-rtjar-includes.txt then it might be 
useful to beef up the comment to explain what happens when an API 
package does not match one of the rules, ie: does it go into compact1, 
only the full JRE, or none. Also make it explicit that the form 
DialogCallbackHandler*.class should not be used. I suggest this for the 
benefit of someone needing to tweak this in the future.

I have played with com.sun.tools.javac.sym.Profiles and so the changes 
to MakefileProfiles look okay to me but Jon should really be the 
reviewer for this. One thing about maxProfiles is that it should match 
Profile.values.length maybe maxProfiles should not be hardcoded to 4. 
Another thing is whether to add a test or beef up an existing test 
(ProfileOptionTest.java in particular).


More information about the core-libs-dev mailing list