Dmitry Chuyko
Dmitry Chuyko
+1. Not OS specific (e.g. happens on Linux).
Are there any high severity problems caused by the original PR? Especially not in the new functionality. Minor issues could be probably addressed without backing out the entire functionality.
> > Are there any high severity problems caused by the original PR? Especially not in the new functionality. Minor issues could be probably addressed without backing out the entire...
> I agree with this backout. Thank you @eastig for explaining your point. We have about 3 weeks before RDP1 and it is better we have less issues before that....
Just a trivial note: this change also improves the calculation of String.hashCode(). For instance, on V1 ``` Benchmark size Improvement StringHashCode.Algorithm.defaultLatin1 1 -2.86% StringHashCode.Algorithm.defaultLatin1 10 45.84% StringHashCode.Algorithm.defaultLatin1 100 79.43% StringHashCode.Algorithm.defaultLatin1...
> @dchuyko ^^^ 17? 21 is what makes practical sense for us as well at the time.
/approval request This a tiny bugfix in ForkJoinPool. At one of code paths the TC subfield of ctl field is decremented and the result is not masked correctly, so the...
Yes. Merged recent master.