RFR: 8301269: Update Commons BCEL to Version 6.7.0
Joe Wang
joehw at openjdk.org
Wed Feb 8 00:23:44 UTC 2023
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.
-------------
Commit messages:
- 8301269: Update Commons BCEL to Version 6.7.0
Changes: https://git.openjdk.org/jdk/pull/12464/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=12464&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8301269
Stats: 30510 lines in 334 files changed: 9313 ins; 10828 del; 10369 mod
Patch: https://git.openjdk.org/jdk/pull/12464.diff
Fetch: git fetch https://git.openjdk.org/jdk pull/12464/head:pull/12464
PR: https://git.openjdk.org/jdk/pull/12464
More information about the core-libs-dev
mailing list