RFR (S) 8212178: Soft reference reclamation race in com.sun.xml.internal.stream.util.ThreadLocalBufferAllocator
Joe Wang
huizhe.wang at oracle.com
Mon Oct 15 19:24:26 UTC 2018
Thanks Joe for the reminder, and thanks all for doing this! The change
looks good to me as well. I assume you updated copyright locally, not
reflected in the webrev. With regards to the class documentation, it's
confusing probably due to a class refactoring during the development.
That part of the history was gone. But we know what's going on, so we
can leave it as is since it's in maintenance.
Best,
Joe
On 10/15/18, 11:20 AM, Joseph D. Darcy wrote:
> I think it would be reasonable to give engineers who work in the XML
> area some time to review the fix as well before it is pushed.
>
> Cheers,
>
> -Joe
>
>
> On 10/15/2018 11:13 AM, Aleksey Shipilev wrote:
>> On 10/15/2018 07:50 PM, Kim Barrett wrote:
>>>> Fix:
>>>> http://cr.openjdk.java.net/~shade/8212178/webrev.01/
>>> Change looks good. Update copyright?
>> Thanks! Updated. jdk-submit is clean, I am going to push it soon.
>>
>>> The class documentation here seems very confusing to me, like it’s
>>> not really about
>>> this class at all. But that’s an entirely different problem.
>> Right.
>>
>> -Aleksey
>>
>>
>
More information about the core-libs-dev
mailing list