RFR: 8286266: [macos] Voice over moving JTable column to be the first column JVM crashes

[Artem Semenov]

On Wed, 11 May 2022 16:20:00 GMT, Alexander Zuev <kizune at 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