On Thu, 20 Feb 2025 09:30:02 GMT, Andrey Turbanov <aturba...@openjdk.org> wrote:

> We have helpful NPE messages now - they are more user-friendly.
> And shorter methods are more likely to be inlined.

While having prerequirements checks often is good, I think not having it in 
String.join can be a good thing.
1. String.join is quite short method. 11 lines. And having 2 lines to have 
_excessive_ check don't improve reading experience much.
2. There are other overloads which don't have null check. I personally like 
them more - they are easier to read.
3. `String` is one of core classes, and having a bit shorter and more 
performant code will never be redundant in it.

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

PR Comment: https://git.openjdk.org/jdk/pull/23710#issuecomment-2704832981

Reply via email to