[NEW BUG]: Avoid allocations in String.replace(CharSequence, CharSequence) in case no replacement happened

[Vitaly Davidovich]

On Thu, Mar 2, 2017 at 6:45 AM Claes Redestad <claes.redestad at oracle.com>
wrote:

> On 03/02/2017 11:38 AM, Ulf Zibis wrote:
> > Hi Vitaly,
> >
> > I don't see any contract to throw an NPE:
> >
> https://docs.oracle.com/javase/8/docs/api/java/lang/String.html#replace-java.lang.CharSequence-java.lang.CharSequence-
> >
>
> "Unless otherwise noted, passing anullargument to a constructor or
> method in this class will cause a|NullPointerException|
> <
> https://docs.oracle.com/javase/8/docs/api/java/lang/NullPointerException.html
> >to
> be thrown."
>
> /Claes
>
Ah, it was moved to class level docs in 8 - thanks Claes.

As a side comment, I certainly understand the desire to not repeat yourself
for each method's javadoc, but at the same time it's a bit less user
friendly as most people don't read class level docs nearly as frequently as
method level.
-- 
Sent from my phone