maniksurtani/infinispan — comment on commit 682eedd9bb6c69aeacd159899cefb0d617661cdd
File: core/src/main/java/org/infinispan/transaction/AbstractCacheTransaction.java line 126
By Sanne:

Like you said, most of my changes are also "logically equivalent" because I'm missing the intended design.
So unless you know what you're doing (i.e. have more information on the design) - I'm pointing out that this change is not logically equivalent:
as you can see above, the original code is both 1) checking and 2) waiting on the condition, both while the lock is owned.

Just making sure you've noticed the difference ;)