Skip to content

fix(jdk-codemodel, jdk-annotation-processor): complete constructor trait coverage across all three population paths#91

Open
deer wants to merge 1 commit into
mainfrom
model_fixes
Open

fix(jdk-codemodel, jdk-annotation-processor): complete constructor trait coverage across all three population paths#91
deer wants to merge 1 commit into
mainfrom
model_fixes

Conversation

@deer
Copy link
Copy Markdown
Collaborator

@deer deer commented Jun 3, 2026

  • JdkInitializer (processConstructor): replaced the manual two-line wiring with resolver.modifyConstructor, which adds type parameters, throws declarations, access modifier, and annotations — bringing the source-based constructor path to parity with modifyMethod.
  • TypeMirrorResolver: introduced modifyConstructor and widened addThrowables to accept any Traitable so it can serve both constructors and methods.
  • JDKCodeModel (reflection path): fixed ThrowableDescriptor being attached to the type descriptor instead of the owning ConstructorDescriptor / MethodDescriptor; fixed getFormalParameters to attach Varargs and Final traits from java.lang.reflect.Parameter.
  • AnnotationProcessor (createTypeDescriptor): added the missing CONSTRUCTOR element loop, matching the field and method loops that were already present.

…ait coverage across all three population paths
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant