Hi Jan,
Is the revised historical data also available for review?
Thanks,
-Joe
On 2/7/2019 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