Re: [PR] Remove concatenation in String.format() calls [lucene]

2024-01-29 Thread via GitHub
uschindler merged PR #13038: URL: https://github.com/apache/lucene/pull/13038 -- 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: issues-unsubscr...@lucene.

Re: [PR] Remove concatenation in String.format() calls [lucene]

2024-01-29 Thread via GitHub
uschindler commented on PR #13038: URL: https://github.com/apache/lucene/pull/13038#issuecomment-1915649179 Thanks for this. The code also has safety problems. If the concatted text para contain percent signs it would break. So format strings should never ever be constructed with variables

[PR] Remove concatenation in String.format() calls [lucene]

2024-01-28 Thread via GitHub
sabi0 opened a new pull request, #13038: URL: https://github.com/apache/lucene/pull/13038 (no comment) -- 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-mai