On Mon, 19 Jan 2026 17:23:10 GMT, David Beaumont <[email protected]> wrote:

> Allow replacement string to contain embedded newlines to split entries.
> This turns output like:
> 
> 
>     <content url="file:///home/david/dev/test/jdk">
>       <sourceFolder 
> url="file:///home/david/dev/test/jdk/open/src/java.base/linux/classes" 
> isTestSource="false" /> <sourceFolder 
> url="file:///home/david/dev/test/jdk/open/src/java.base/unix/classes" 
> isTestSource="false" /> <sourceFolder 
> url="file:///home/david/dev/test/jdk/closed/src/java.base/share/classes" 
> isTestSource="false" /> ... another 10k chars on the end of this line ...
> 
> 
> into:
> 
>     <content url="file:///home/david/dev/test/jdk">
>       <sourceFolder 
> url="file:///home/david/dev/test/jdk/open/src/java.base/linux/classes" 
> isTestSource="false" />
>       <sourceFolder 
> url="file:///home/david/dev/test/jdk/open/src/java.base/unix/classes" 
> isTestSource="false" />
>       <sourceFolder 
> url="file:///home/david/dev/test/jdk/closed/src/java.base/share/classes" 
> isTestSource="false" />
>       ... one entry per line ...

In addition, I see extra spaces presumably for indent after the `\n` - are we 
sure the number of spaces is correct? Is there a reason we don't need this 
indent for the 1st line?

-------------

PR Comment: https://git.openjdk.org/jdk/pull/29305#issuecomment-3779447248

Reply via email to