RFR: 8323659: LinkedTransferQueue add and put methods call overridable offer
Viktor Klang
vklang at openjdk.org
Fri Jan 12 12:25:18 UTC 2024
On Fri, 12 Jan 2024 12:10:46 GMT, Chris Hegarty <chegar at openjdk.org> wrote:
>>> this PR is against mainline, but the issue's Fix Version/s is 22.
>>
>> I updated the fix version in JIRA, and followed the process as outlined in https://mail.openjdk.org/pipermail/jdk-dev/2023-December/008560.html
>
>> @ChrisHegarty My 2¢—this change could impact existing classes which have extended LTQ and knowingly, or unknowingly, presume that add and put delegate to offer, which would (silently) break those classes when running on a newer JDK (presuming this change gets merged).
>>
>> I think the change is in principle correct, with that said, I want to make sure that the change is worth the potential compatibility risk (if any).
>
> Hi @viktorklang-ora , I agree with your comment regarding the potential impact, and the assumption of the implementation, however, you got it the wrong way round! ;-) The change I am proposing restores previous behaviour to pre-JDK 22. It is JDK 22 that has changed this. This is exactly why I am proposing the change
@ChrisHegarty 😅 Well then I agree
-------------
PR Comment: https://git.openjdk.org/jdk/pull/17393#issuecomment-1889041351
More information about the core-libs-dev
mailing list