Re: Constructor `String(String)` does not copy `hashIsZero`

2021-11-22 Thread Japris Pogrammer
't see that HotSpot/C2 is > actually intrinsifying this constructor. The signature is used for > some pattern matching in the legacy stringopts.cpp code, though. So > the trivial fix should help a bit. > > Thanks > Claes > > [1] https://bugs.openjdk.java.net/browse/JDK-8

Constructor `String(String)` does not copy `hashIsZero`

2021-11-22 Thread Japris Pogrammer
According to openjdk/jdk [1] current copy-constructor of String class does not copy the value of hashIsZero field which may lead to 0-hash being recalculated on copy even if it is known to be 0. Could you please tell me if I am right with this point or if this behaviour is intentional? If this is a

AbstractProcessor overrides process(..) for no reason

2021-10-16 Thread Japris Pogrammer
Hi there, I've seen that javax.annotation.processing.AbstractProcessor overrides `Processor#process(..)` method without any changes to it [1]: - it still is abstract - signature is untouched - javadoc is untouched Is this done intentionally or this redundant override can be eliminated? [1]: https: