On Wed, 11 May 2022 16:20:00 GMT, Alexander Zuev <kiz...@openjdk.org> wrote:

>> src/java.desktop/macosx/native/libawt_lwawt/awt/a11y/TableAccessibility.m 
>> line 141:
>> 
>>> 139:             [rowCache removeObjectForKey:key];
>>> 140:         }
>>> 141:         cacheValid = YES;
>> 
>> It seems to me that here it is necessary to release the cache.
>> 
>>     [rowCache release];
>>     rowCache = nil;
>
>> It seems to me that here it is necessary to release the cache.
> 
> I am not so sure. Why would we do that? I mean - we removed all the records 
> from the cache and even if we would release it we would have to immediately 
> re-create it otherwise the very next operation will crash since we will be 
> trying to call objectForKey selector upon nil. So what is the point in 
> releasing the now empty dictionary?

Ok.

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

PR: https://git.openjdk.java.net/jdk/pull/8636

Reply via email to