Integrated: 8301269: Update Commons BCEL to Version 6.7.0

Joe Wang joehw at openjdk.org
Sat Feb 11 22:36:37 UTC 2023


On Wed, 8 Feb 2023 00:15:27 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.

This pull request has now been integrated.

Changeset: 6a44120a
Author:    Joe Wang <joehw at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/6a44120a16d0f06b4ed9f0ebf6b0919da7070287
Stats:     30511 lines in 334 files changed: 9313 ins; 10828 del; 10370 mod

8301269: Update Commons BCEL to Version 6.7.0

Reviewed-by: lancea

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

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


More information about the core-libs-dev mailing list