On Mon, 3 Oct 2022 05:54:31 GMT, Markus KARG <d...@openjdk.org> wrote:

>> This PR implements JDK-8294696.
>
> Markus KARG has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Checking explicitly -1 instead of < 0

> > Isn't a test needed here which fails without but passes with the proposed 
> > change?
> I do not see such a _need_, as this is solely a performance improvement, not 
> a fixed bug or added functionality. If the majority of this team wants to 
> have such a test, then please clearly instruct me to write one. Thanks.

@AlanBateman WDYT? Is such a test mandatory to include this rather simple PR?

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

PR: https://git.openjdk.org/jdk/pull/10525

Reply via email to