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

2013-03-13 Thread Jonathan Gibbons
The langtools changes look OK to me. -- Jon On 03/10/2013 06:24 PM, 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

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

2013-03-13 Thread David Holmes
Hi Alan, On 12/03/2013 8:05 PM, Alan Bateman wrote: On 12/03/2013 07:10, David Holmes wrote: : 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

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

2013-03-11 Thread Alan Bateman
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

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

2013-03-10 Thread David Holmes
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

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

2013-03-08 Thread David Holmes
Thanks Erik! David On 8/03/2013 9:25 PM, Erik Joelsson wrote: On 2013-03-08 10:19, Alan Bateman wrote: On 08/03/2013 01:48, David Holmes wrote: Not sure which is best list for this given Alan will likely be the only reviewer anyway :) Webrevs under:

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

2013-03-07 Thread David Holmes
Not sure which is best list for this given Alan will likely be the only reviewer anyway :) Webrevs under: http://cr.openjdk.java.net/~dholmes/8009428_8009429/ Two related sets of fixes here: JDK-8009428: Revert changes to $ substitution performed as part of nashorn integration This