JDK-8295058 : test/langtools/tools/javac 116 test classes uses com.sun.tools.classfile library
  • Type: Sub-task
  • Component: tools
  • Sub-Component: javac
  • Priority: P4
  • Status: Resolved
  • Resolution: Fixed
  • Submitted: 2022-10-10
  • Updated: 2023-10-03
  • Resolved: 2023-08-07
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
JDK 22
22 b10Fixed
Related Reports
Relates :  
JDK-8317415 - [lworld] Preload_attribute needs moving to jdk.internal.classfile.attribute
Relates :  
JDK-8313258 - RuntimeInvisibleTypeAnnotationsAttribute.annotations() API Index out of Bound error
Description
There are 263 tests to convert to the new Class-File API.   Here's the list of test in 

find test/langtools/tools/javac -type f -name '*.java' | xargs grep -w -l 'com.sun.tools.classfile'

Modified 116 tests with new Class-File API, and migrate helper methods to test/langtools/tools/javac/annotations/typeAnnotations/classfile/CombinationsTargetTest3.java while keep using old Class-File API. Here's the list:

	test/langtools/tools/javac/annotations/typeAnnotations/classfile/AnonymousClassTest.java	test/langtools/tools/javac/annotations/typeAnnotations/classfile/CombinationsTargetTest3.java	test/langtools/tools/javac/classfiles/attributes/lib/TestBase.java	test/langtools/tools/javac/classfiles/attributes/LineNumberTable/LineNumberTestBase.java	test/langtools/tools/javac/classfiles/attributes/Module/ModuleTestBase.java	test/langtools/tools/javac/classfiles/attributes/innerclasses/InnerClassesTestBase.java	test/langtools/tools/javac/classfiles/attributes/SourceFile/SourceFileTestBase.java	test/langtools/tools/javac/classfiles/attributes/annotations/RuntimeParameterAnnotationsTestBase.java	test/langtools/tools/javac/classfiles/attributes/annotations/RuntimeAnnotationsTestBase.java	test/langtools/tools/javac/classfiles/attributes/annotations/AnnotationsTestBase.java	test/langtools/tools/javac/classfiles/attributes/LocalVariableTable/LocalVariableTestBase.java	test/langtools/tools/javac/classfiles/attributes/Synthetic/PackageInfoTest.java	test/langtools/tools/javac/classfiles/attributes/SourceFile/AnonymousClassTest.java	test/langtools/tools/javac/classfiles/attributes/SourceFile/LocalClassTest.java	test/langtools/tools/javac/classfiles/attributes/Signature/EnumTest.java	test/langtools/tools/javac/classfiles/attributes/SourceFile/InnerClassTest.java	test/langtools/tools/javac/classfiles/attributes/Synthetic/AssertFieldTest.java	test/langtools/tools/javac/classfiles/attributes/Synthetic/EnumTest.java	test/langtools/tools/javac/classfiles/attributes/innerclasses/InnerEnumsInInnerClassTest.java	test/langtools/tools/javac/classfiles/attributes/innerclasses/InnerClassesInInnerInterfaceTest.java	test/langtools/tools/javac/classfiles/attributes/innerclasses/InnerInterfacesInInnerClassTest.java	test/langtools/tools/javac/classfiles/attributes/innerclasses/InnerAnnotationsInInnerClassTest.java	test/langtools/tools/javac/classfiles/attributes/innerclasses/InnerClassesInInnerAnnotationTest.java	test/langtools/tools/javac/constDebug/ConstDebugTest.java	test/langtools/tools/javac/classfiles/attributes/Synthetic/BridgeMethodForGenericMethodTest.java	test/langtools/tools/javac/classfiles/attributes/innerclasses/InnerEnumInInnerInterfaceTest.java	test/langtools/tools/javac/classfiles/attributes/innerclasses/InnerEnumInInnerAnnotationTest.java	test/langtools/tools/javac/classfiles/attributes/innerclasses/InnerAnnotationsInInnerInterfaceTest.java	test/langtools/tools/javac/classfiles/attributes/innerclasses/InnerInterfacesInInnerAnnotationTest.java	test/langtools/tools/javac/classfiles/attributes/innerclasses/InnerAnnotationsInInnerAnnotationTest.java	test/langtools/tools/javac/classfiles/attributes/innerclasses/NoInnerClassesTest.java	test/langtools/tools/javac/classfiles/attributes/LineNumberTable/T8050993.java	test/langtools/tools/javac/classfiles/attributes/innerclasses/InnerEnumInInnerEnumTest.java	test/langtools/tools/javac/classfiles/attributes/innerclasses/InnerInterfacesInInnerEnumTest.java	test/langtools/tools/javac/classfiles/attributes/innerclasses/InnerAnnotationsInInnerEnumTest.java	test/langtools/tools/javac/classfiles/attributes/innerclasses/InnerInterfacesInInnerInterfaceTest.java	test/langtools/tools/javac/classfiles/attributes/SourceFile/NoSourceFileAttribute.java	test/langtools/tools/javac/classfiles/attributes/SourceFile/ModuleInfoTest.java	test/langtools/tools/javac/annotations/typeAnnotations/classfile/Scopes.java	test/langtools/tools/javac/classfiles/attributes/SourceFile/SyntheticClassTest.java	test/langtools/tools/javac/classfiles/attributes/Module/ModuleFlagTest.java	test/langtools/tools/javac/annotations/typeAnnotations/classfile/T8008769.java	test/langtools/tools/javac/classfiles/attributes/Synthetic/ThisFieldTest.java	test/langtools/tools/javac/classfiles/attributes/LineNumberTable/MultipleRecordPatterns.java	test/langtools/tools/javac/classfiles/attributes/innerclasses/InnerClassesInInnerEnumTest.java	test/langtools/tools/javac/T7165659/InnerClassAttrMustNotHaveStrictFPFlagTest.java	test/langtools/tools/javac/classfiles/attributes/LineNumberTable/StringSwitchBreaks.java	test/langtools/tools/javac/8009170/RedundantByteCodeInArrayTest.java	test/langtools/tools/javac/annotations/typeAnnotations/classfile/NewTypeArguments.java	test/langtools/tools/javac/annotations/typeAnnotations/classfile/Wildcards.java	test/langtools/tools/javac/classfiles/attributes/SourceFile/TopLevelClassesOneFileTest.java	test/langtools/tools/javac/T8028504/DontGenerateLVTForGNoneOpTest.java	test/langtools/tools/javac/T8210435/NoLocalsMustBeReservedForDCEedVarsTest.java	test/langtools/tools/javac/annotations/typeAnnotations/classfile/InstanceInitializer.java	test/langtools/tools/javac/annotations/typeAnnotations/classfile/StaticInitializer.java	test/langtools/tools/javac/classfiles/attributes/Synthetic/AccessToPrivateInnerClassConstructorsTest.java	test/langtools/tools/javac/annotations/typeAnnotations/classfile/T8008762.java	test/langtools/tools/javac/classreader/BadClass.java	test/langtools/tools/javac/classfiles/attributes/annotations/RuntimeAnnotationsForInnerAnnotationTest.java	test/langtools/tools/javac/lambda/LambdaTestStrictFPFlag.java	test/langtools/tools/javac/resolve/NoObjectToString.java	test/langtools/tools/javac/T8011181/EmptyUTF8ForInnerClassNameTest.java	test/langtools/tools/javac/annotations/typeAnnotations/classfile/T8010015.java	test/langtools/tools/javac/classfiles/attributes/innerclasses/InnerClassesInInnerClassTest.java	test/langtools/tools/javac/linenumbers/ConditionalLineNumberTest.java	test/langtools/tools/javac/StringConcat/WellKnownTypes.java	test/langtools/tools/javac/classfiles/attributes/innerclasses/InnerClassesInAnonymousClassTest.java	test/langtools/tools/javac/classfiles/attributes/Signature/MethodTypeBoundTest.java	test/langtools/tools/javac/annotations/typeAnnotations/classfile/DeadCode.java	test/langtools/tools/javac/classfiles/attributes/annotations/RuntimeParameterAnnotationsForGenericMethodTest.java	test/langtools/tools/javac/classfiles/attributes/LineNumberTable/LineNumberTest.java	test/langtools/tools/javac/classfiles/attributes/SourceFile/MixTest.java	test/langtools/tools/javac/classfiles/attributes/annotations/RuntimeAnnotationsForGenericMethodTest.java	test/langtools/tools/javac/annotations/typeAnnotations/classfile/TypeCasts.java	test/langtools/tools/javac/classfiles/attributes/LineNumberTable/RuleSwitchBreaks.java	test/langtools/tools/javac/classfiles/attributes/Synthetic/AccessToPrivateInnerClassMembersTest.java	test/langtools/tools/javac/classfiles/attributes/annotations/RuntimeAnnotationsForInnerClassTest.java	test/langtools/tools/javac/classfiles/attributes/annotations/RuntimeAnnotationsForInnerInterfaceTest.java	test/langtools/tools/javac/annotations/typeAnnotations/classfile/SyntheticParameters.java	test/langtools/tools/javac/classfiles/attributes/deprecated/DeprecatedPackageTest.java	test/langtools/tools/javac/classfiles/attributes/annotations/RuntimeAnnotationsForInnerEnumTest.java	test/langtools/tools/javac/classfiles/attributes/annotations/RuntimeAnnotationsForTopLevelClassTest.java	test/langtools/tools/javac/classfiles/attributes/annotations/RuntimeParameterAnnotationsTest.java	test/langtools/tools/javac/classfiles/attributes/innerclasses/InnerClassesTest.java	test/langtools/tools/javac/classfiles/attributes/Signature/ExceptionTest.java	test/langtools/tools/javac/annotations/typeAnnotations/classfile/TypeAnnotationPropagationTest.java	test/langtools/tools/javac/classfiles/attributes/Synthetic/AccessToPrivateSiblingsTest.java	test/langtools/tools/javac/classfiles/attributes/Synthetic/BridgeMethodsForLambdaTest.java	test/langtools/tools/javac/sealed/CheckSubtypesOfSealedTest.java	test/langtools/tools/javac/classfiles/attributes/annotations/RuntimeParameterAnnotationsForLambdaTest.java	test/langtools/tools/javac/classfiles/attributes/Synthetic/BridgeMethodsForLambdaTargetRelease14Test.java	test/langtools/tools/javac/annotations/SyntheticParameters.java	test/langtools/tools/javac/classfiles/attributes/innerclasses/InnerClassesIndexTest.java	test/langtools/tools/javac/classfiles/attributes/AnnotationDefault/AnnotationDefaultTest.java	test/langtools/tools/javac/classfiles/attributes/innerclasses/InnerClassesInLocalClassTest.java	test/langtools/tools/javac/classfiles/attributes/Signature/ConstructorTest.java	test/langtools/tools/javac/classfiles/attributes/LocalVariableTable/LocalVariableTypeTableTest.java	test/langtools/tools/javac/annotations/typeAnnotations/classfile/NoTargetAnnotations.java	test/langtools/tools/javac/classfiles/attributes/Signature/MethodParameterTest.java	test/langtools/tools/javac/classfiles/attributes/Signature/FieldTest.java	test/langtools/tools/javac/classfiles/attributes/Synthetic/SyntheticTestDriver.java	test/langtools/tools/javac/classfiles/attributes/Signature/Driver.java	test/langtools/tools/javac/classfiles/attributes/LocalVariableTable/LocalVariableTableTest.java	test/langtools/tools/javac/annotations/typeAnnotations/classfile/ClassfileTestHelper.java	test/langtools/tools/javac/classfiles/attributes/AnnotationDefault/AnnotationDefaultVerifier.java	test/langtools/tools/javac/classfiles/attributes/deprecated/DeprecatedTest.java	test/langtools/tools/javac/classfiles/attributes/Module/ModuleTest.java	test/langtools/tools/javac/classfiles/attributes/Signature/InnerClassTest.java	test/langtools/tools/javac/annotations/typeAnnotations/classfile/TestNewCastArray.java	test/langtools/tools/javac/annotations/typeAnnotations/classfile/CombinationsTargetTest1.java	test/langtools/tools/javac/classfiles/attributes/innerclasses/InnerClassesHierarchyTest.java	test/langtools/tools/javac/classfiles/attributes/annotations/TestAnnotationInfo.java	test/langtools/tools/javac/annotations/typeAnnotations/classfile/TestAnonInnerClasses.java	test/langtools/tools/javac/annotations/typeAnnotations/classfile/CombinationsTargetTest2.java	test/langtools/tools/javac/classfiles/attributes/EnclosingMethod/EnclosingMethodTest.java	test/langtools/tools/javac/classfiles/attributes/Signature/ReturnTypeTest.java	test/langtools/lib/annotations/annotations/classfile/ClassfileInspector.java
Comments
Changeset: 380418fa Author: Qing Xiao <xqoasis@gmail.com> Committer: Adam Sotona <asotona@openjdk.org> Date: 2023-08-07 15:49:11 +0000 URL: https://git.openjdk.org/jdk/commit/380418fad07c0526bb698b4bfcbacbd65a8615be
07-08-2023
A pull request was submitted for review. URL: https://git.openjdk.org/jdk/pull/14874 Date: 2023-07-13 17:29:44 +0000
31-07-2023