summaryrefslogtreecommitdiffstats
path: root/doc/TODO.txt
diff options
context:
space:
mode:
authorSven Gothel <[email protected]>2014-09-23 03:56:04 +0200
committerSven Gothel <[email protected]>2014-09-23 03:56:04 +0200
commit66ecb6c386d5c3d87d8be2600a0c7dd7d71a4329 (patch)
tree8a50d67fb0f3ea895d2522a9888f585c3737b470 /doc/TODO.txt
parentc0c5fac5301f7264cfdd69117c1f85adfdc7b604 (diff)
Fix synchronization issues in Animator* Exception case
Refines commit cef7ba607ad7e8eb1ff2a438d77710a29aa0bda6 - The animator monitor-lock was still hold in the post finally block issuing flushGLRunnables(), due to intrinsic monitor release (in finally): - <http://docs.oracle.com/javase/specs/jvms/se7/html/jvms-2.html#jvms-2.11.10> - <http://stackoverflow.com/questions/10743285/behavior-of-a-synchronized-method-with-try-and-finally> - Further: AnimatorBase.flushGLRunnables() acquired the lock itself (duh!) This commit removes the requirement for finally altogether by simply return a boolean from handleUncaughtException(caughtException), where false denotes the caller to propagate the exception itself (no handler). Post synchronized block then issues flushGLRunnables() and exceptation propagation as required. AnimatorBase.flushGLRunnables() 'synchronized' modifier is removed. Further, ThreadDeath is being propagated if caught. Here the finally block is also removed - redundant.
Diffstat (limited to 'doc/TODO.txt')
0 files changed, 0 insertions, 0 deletions