RFR: 8301269: Update Commons BCEL to Version 6.7.0 [v2]

Lance Andersen lancea at openjdk.org
Fri Feb 10 23:03:26 UTC 2023


On Fri, 10 Feb 2023 22:48:24 GMT, Joe Wang <joehw at openjdk.org> wrote:

>> Update Commons BCEL in the JDK from 6.5.0 to 6.7.0. 
>> 
>> 1. Improvement
>>    Code modernization (using relatively more modern language features), normalization (such as naming), javadocs, cleanup and minor improvements made up most of the changes. These changes do not change how BCEL works for the XSLT function in the JDK. Many of the changes unfortunately created long lines, for example merging multiple lines into a single long line, not what we would like to have in the JDK, but I have to leave them as they are so that they don't show up as changes in future updates. 
>> 
>> 2. Exclusions
>>    As it has always been, the focus of the BCEL component is to satisfy the needs for the XSLT functionality. Classes not needed for the JDK are excluded as it has been done previously, this is especially true in the util package. The verifier package is also excluded. Methods and sources that have dependencies on the excluded classes are either removed or modified.
>> 
>> 3. Deprecation and Warnings
>>    
>>    Deprecated classes and contents relying on them are removed or modified to use proper methods without changing the functionality. Deprecated fields in the original source were changed to private ones in previous update in the JDK, however, since the update process applies sources from the upstream again, they were reverted back. In order to not have to go through them on every update, I'll keep the original source this time. 
>> 
>>     Warnings were one of the main reasons for the changes made to the original source. It has been done in the previous update. In this update, many of the previous JDK fixes on warnings have already been addressed in the upstream source as the result of item 1, thus lessoned the needs to reapply the warning patches. Fixed the rest of warnings as they showed up during build.
>> 
>> 4. Note that JRocket specific patches are not in the current code base.
>> 
>> 5. Test
>>     Tier2 tests passed.
>>     JCK XML tests passed.
>
> Joe Wang has updated the pull request incrementally with one additional commit since the last revision:
> 
>   update the date of md file

Marked as reviewed by lancea (Reviewer).

-------------

PR: https://git.openjdk.org/jdk/pull/12464


More information about the core-libs-dev mailing list