<AWT Dev> [9] Review request for 8068886: IDEA IntelliJ crashes in objc_msgSend when an accessibility tool is enabled

[Sergey Bylokhov]

Hi, Anton.
The fix looks fine, but I suggest to document the new resources 
deallocation policy. I suppose documentation of loops, 
CFRetainedResource, etc should be updated.
Also please confirm that all our code follow this rule.

On 08.05.15 19:52, Anton Nashatyrev wrote:
> Hello,
>     could you please review the following fix:
>
> fix: http://cr.openjdk.java.net/~anashaty/8068886/webrev.00/ 
> <http://cr.openjdk.java.net/%7Eanashaty/8068886/webrev.00/>
> bug: https://bugs.openjdk.java.net/browse/JDK-8068886
>
>     Problem: the native window pointer is disposed in the inner AppKit 
> loop, while it is still referenced on the stack in the native Cocoa 
> method which caused the mentioned inner loop. When the inner loop is 
> exited Cocoa crashes while dereferencing this window pointer
>
>     Fix: dispose resources in the main AppKit loop only. For this a 
> 'dispose message' posted which is processed only on the main loop.
>
> Thanks!
> Anton.


-- 
Best regards, Sergey.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20150512/cfe79fec/attachment.html>