RFR: JDK-8218630: CreateSymbols includes class and module headers unnecessarily.

Jonathan Gibbons jonathan.gibbons at oracle.com
Thu Feb 7 18:27:31 UTC 2019


Looks good to me.

-- Jon

On 2/7/19 9:15 AM, Jan Lahoda wrote:
> Hi,
>
> Joe noted that the proposed historical data for JDK 12:
> http://cr.openjdk.java.net/~jlahoda/8216263/webrev.00/
> may contain unnecessary items, and it indeed is the case.
>
> The main problems appear to be:
> -when the class headers are being compared, the nestMembers for the 
> current version may be null, and the nestMembers for the previous 
> version may be an empty list. So, currently, such headers won't match 
> and the header will be generated again to the historical record. 
> listMatch can be used to consider null and empty list to be the same
>
> -module header descriptions may refer to 
> RequiresDescription/ProvidesDescription, which are missing 
> hashCode/equals, which means that headers that have either requires or 
> provides won't match any of the existing headers. The fix is to add 
> hashCode and equals
>
> I generated the ct.sym for the original JDK 12 historical data from
> http://cr.openjdk.java.net/~jlahoda/8216263/webrev.00/
>
> and the historical data generated by this patch, and the resulting 
> content appears to be equivalent.
>
> Proposed webrev: http://cr.openjdk.java.net/~jlahoda/8218630/webrev.00/
> Bug: https://bugs.openjdk.java.net/browse/JDK-8218630
>
> How does this look?
>
> Thanks,
>     Jan



More information about the build-dev mailing list