RFR: 8323659: LinkedTransferQueue add and put methods call overridable offer [v3]

Alan Bateman alanb at openjdk.org
Mon Jan 15 09:52:20 UTC 2024


On Mon, 15 Jan 2024 01:55:12 GMT, David Holmes <dholmes at openjdk.org> wrote:

> With my CSR hat on, JDK-8301341 should never have made the changes it did without going through a CSR request. We have been bitten by this kind of problem many times. Unless a public method is specified to utilise another public method, we should never implement one public method in terms of another (non-final one) due to the overriding problem.

JDK-8301341 was a big update, a lot of refactoring to hollow out SQ and it was just an oversight that we didn't spot the methods now use an overridable method. It's something to always look out for in the collections area, just missed here.

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

PR Comment: https://git.openjdk.org/jdk/pull/17393#issuecomment-1891752374


More information about the core-libs-dev mailing list