efegokdemir opened a new pull request, #377: URL: https://github.com/apache/maven-jxr/pull/377
Fixes #317 ## Summary `JavaFileImpl` could stop parsing a source file after a `@SuppressWarnings` annotation with multiple values, causing the generated Source Xref class indexes to omit the annotated class. ## Changes - Track annotation argument parentheses so array braces are not mistaken for Java type nesting. - Preserve token context for punctuation so adjacent annotation string literals do not enter the text-block state. - Add a regression fixture and `DirectoryIndexerTest` coverage for a class with multiple `@SuppressWarnings` values. ## Testing - `JAVA_HOME=/opt/homebrew/opt/openjdk@21/libexec/openjdk.jdk/Contents/Home PATH="$JAVA_HOME/bin:$PATH" mvn -pl maven-jxr -Dspotless.skip=true -Dtest=DirectoryIndexerTest test -q` ✅ - `JAVA_HOME=/opt/homebrew/opt/openjdk@21/libexec/openjdk.jdk/Contents/Home PATH="$JAVA_HOME/bin:$PATH" mvn verify -q` ✅ - `JAVA_HOME=/opt/homebrew/opt/openjdk@21/libexec/openjdk.jdk/Contents/Home PATH="$JAVA_HOME/bin:$PATH" mvn -Prun-its verify -q` ✅ - `git diff --check` ✅ ## Notes The change is limited to parser state handling and regression coverage. The commit is SSH-signed and includes a DCO sign-off. Codex assisted with analysis and implementation; the changed lines and validation were reviewed locally. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
