elharo opened a new pull request, #382: URL: https://github.com/apache/maven-shared-utils/pull/382
Constructors accepting a `String lineIndent` parameter document it "can be null" but call `lineIndent.toCharArray()` without a null check, throwing `NullPointerException`. Fixed constructors now fall back to the default indentation (two spaces) when `lineIndent` is null, and to `"\n"` when `lineSeparator` is null — matching the behavior of the constructors that don't take these parameters. Fixes https://github.com/apache/maven-shared-utils/issues/381 Commits: - 581ca4a — Add failing test: PrettyPrintXMLWriter with null lineIndent - b38df77 — PrettyPrintXMLWriter: handle null lineIndent/lineSeparator in constructors -- 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]
