[dependabot]

Bumps com.puppycrawl.tools:checkstyle from 13.3.0 to 14.4.0.

Release notes

Sourced from com.puppycrawl.tools:checkstyle's releases.

checkstyle-14.4.0

Checkstyle 14.4.0 - https://checkstyle.org/releasenotes.html#Release_14.4.0

New:

#21890 - New property for JavadocUtilizingTrailingSpace validateOnlyJoinableLines to report a line only when the whole next line fits on it. #20970 - JavadocLeadingAsteriskAlign: Add indentation property. #12226 - LeftCurly: new option nl_or_singleline to allow single-line blocks in otherwise Allman-style code. #21813 - New property for JavadocUtilizingTrailingSpace ignoreTooShortPattern to allow skip utilization if line ends with punctuation symbol. #18437 - EmptyLineSeparatorCheck does not report violations when more than one empty line appears with enum constants. #20496 - New check: RedundantModifierCompactSource. #18571 - Google-style: New Check BlockCommentEndPositionCheck. #21359 - New Check: FinalPatternVariable for Enforcing Final Pattern Variables. #18971 - New Check NoCauseGetMessageInExceptionConstructor: to not use caught exception message in wrapper exception constructor. #20595 - new Check: ClassHeaderWrapOpenjdk. #20638 - new Check: OpenjdkMethodParameterAlignmentCheck. #21802 - New Check ExceptionThrowShouldUseContext: Exception wrapping should always use context variables. #7895 - new Check: RedundantThis. #5313 - New Check: EmptyLineWrappingInBlock. #16748 - New check JavadocUtilizingTrailingSpace: Javadoc comments check to enforce compact content to full usage of line length.

Bug fixes:

#21941 - escape line breaks in property keys echoed in violation messages. #20266 - RedundantModifierCheck: redundant public modifier on top-level method not flagged in compact source files (JEP 512). #21828 - UnnecessaryNullCheckWithInstanceOf: false positive when variable is dereferenced inside arithmetic or comparison sub-expression. #22058 - False Positive: AvoidNoArgumentSuperConstructorCall reports super() when statements precede it in constructor (Flexible Constructor Bodies). #22173 - MethodCountCheck does not clear counters on beginTree. #21816 - Add checks for Sun Style [8.2 - Blank Spaces]. #22153 - NoCauseGetMessageInExceptionConstructorCheck does not clear catchScopes state on beginTree. #22092 - VariableDeclarationUsageDistance: NullPointerException on method in enum, record or interface. #21761 - EmptyLineSeparator: false positive duplicate violation when multiple enum constants share a line. #22045 - DescendantToken: StackOverflowError on deeply nested string concatenation. #21694 - add OpenjdkMethodParameterAlignment check in openjdk_checks.xml. #21952 - CLI exits with code 0 when the number of errors is a multiple of 256. #21864 - malformed unicode escape in UniqueProperties. #21883 - UnnecessaryNullCheckWithInstanceOf: NullPointerException when Boolean variable is used as condition in logical AND expression. #21908 - fix NullPointerException and memory leak in RedundantThis. #15193 - Conflict in Indentation Check Between if Parentheses and Record Pattern Parentheses. #21600 - EmptyLineSeparator with allowMultipleEmptyLines as false does not report a violation if there is Javadoc then an annotation before a method. #18590 - False Negative in EmptyStatement Check. #21785 - Add checks for Sun Style [7.5 - for Statements]. #21755 - MissingJavadocMethod: false-positive on getters containing block comments when allowMissingPropertyJavadoc is enabled. #5353 - RequireThisCheck: false positive in anonymous class with same name as field in top level class. #21721 - SarifLogger: artifactLocation.uri is not properly encoded for file names containing '#', '?', or '%'. #21519 - RegexpSinglelineJava hangs on a suppressed zero-width match. #4225 - FinalLocalVariable: "throw" false negative. #21751 - RequireThis: false negative for a field shadowed by a resource in finally. #20811 - Update Coverage for Sun's Java Style.

... (truncated)

Commits

• 7d9e948 [maven-release-plugin] prepare release checkstyle-14.4.0
• 7304bca doc: release notes for 14.4.0
• e1b949f infra: restore git cred to let release pass for 14.4.0; release notes push wa...
• 9706aad Issue #20590: Add EqualsAvoidNullCheck compact source coverage
• ac4900c Issue #20590: Add HideUtilityClassConstructorCheck compact source coverage
• cecf68d Issue #20590: Add FinalClassCheck compact source coverage
• 532ffa6 Issue #20590: Add DefaultComesLastCheck compact source coverage
• a046994 Issue #20590: Add EmptyBlockCheck compact source coverage
• ef9378a Pull #21941: escape line breaks in property keys echoed in violation messages
• 0ffcc88 Issue #20590: Add AvoidEscapedUnicodeCharactersCheck compact source coverage
• Additional commits viewable in compare view

Most Recent Ignore Conditions Applied to This Pull Request

Dependency Name	Ignore Conditions
com.puppycrawl.tools:checkstyle	[>= 10.12.a, < 10.13]

[Fokko]

Pushed a fix commit for the Checkstyle 14.4.0 strict HTML javadoc parsing: replaced the malformed </p>/<p/> constructs with <p>, wrapped the <pre> code samples with generics in {@code}, repaired the malformed <a href> in SpecificData, removed the stray </li> in Conversion, removed 2 genuinely unused imports, and dropped the empty statements after enum bodies. mvn checkstyle:check now passes with 0 violations locally.

[Fokko]

Follow-up: also fixed the Spotless violation — the eclipse formatter reformats <pre>{@code blocks, so the readMapStart example in Decoder is now wrapped in spotless:off/on (the pom already enables <toggleOffOn/> for exactly this). mvn spotless:check and mvn checkstyle:check both pass locally.