views:

192

answers:

2

Are there any unclear side effects to throwing an exception from within a synchronized clause? What happens to the lock?

private void doSomething() throws Exception {...}

synchronized (lock) {   
    doSomething();       
}
+8  A: 

I see no side-effect.

The lock is guaranteed to be terminated in all cases, and an exception is no exception (pun intended).

KLE
+1 for giving evidence, that sometimes an exception is not an exception :-))
Andreas_D
@Andreas Good one! :-)
KLE
+2  A: 

As you would hope, the lock is released normally.

For reference, the appropriate section of the JLS which guarantees this behaviour is § 14.19:

If execution of the Block completes normally, then the lock is unlocked and the synchronized statement completes normally. If execution of the Block completes abruptly for any reason, then the lock is unlocked and the synchronized statement then completes abruptly for the same reason.

('abrupt completion' is defined elsewhere in the JLS to include exceptions from JVM, exceptions raised by throw, and use of the break, continue, or return statements to transfer outside the block.)

Cowan