RFR [15/java.xml] 8235368 : Update BCEL to Version 6.4.1

Daniel Fuchs daniel.fuchs at oracle.com
Thu Jan 16 10:35:04 UTC 2020


Hi Joe,

Looks OK to me as well.
I am a bit surprised by the number of methods that are no longer
`final` though. Do you know what was the motivation for those
changes?

best regards,

-- daniel

On 14/01/2020 20:08, Joe Wang wrote:
> Hi,
> 
> Please review an update to BCEL 6.4.1.
> 
> JBS: https://bugs.openjdk.java.net/browse/JDK-8235368
> webrev: http://cr.openjdk.java.net/~joehw/jdk15/8235368/webrev/index.html
> 
> Similar approach as the last update:
> 1. Format
>      All format changes are kept as they are in the source in order to 
> reduce the amount of changes in future updates, the exceptions are 
> extreme long lines.
> 
> 2. Exclusions
>      Contents that were not in the JDK or unnecessary for java.xml are 
> excluded. This includes: the ability to load arbitrary classes and 
> classes related to ClassLoader, ClassPath and JavaWrapper, and relevant 
> methods and references in other classes; System Properties used to set 
> cache sizes and track certain statistics (caches are set as in previous 
> version); Deprecated classes and related contents.
> 
> 3. Warnings
>      Warnings were the main reason for the changes made to the original 
> source. It has been done in the previous update. They are re-applied for 
> this update. The LastModified field indicates such changes to the 
> original source.
> 
> 4. Deprecated fields to private and references to deprecated methods
>     Deprecated fields in the original source were changed to private 
> ones in previous update. References to deprecated methods were modified 
> to use proper methods. These changes are inherited in this update.
> 
> 5. Test
>      Since the update does not affect java.xml's usage of the BCEL 
> component, it is essential to pass all of the existing tests. I've run 
> the tests multiple times and noted that all of the XML functional and 
> unit tests passed, so were JCK XML tests. A performance test is running.
> 
> 
> Thanks,
> Joe
> 
> 



More information about the core-libs-dev mailing list