Consider using assertions to validate locking assumptions --------------------------------------------------------- Key: JBTM-2855 URL: https://issues.jboss.org/browse/JBTM-2855 Project: JBoss Transaction Manager Issue Type: Enhancement Components: Transaction Core Affects Versions: 5.5.2.Final Reporter: Michael Musgrove Assignee: Michael Musgrove Priority: Optional Fix For: 5.next Some methods in BasicAction assume that other methods will acquire monitors before calling them; it would aid both maintenance/readability and testing if we such methods have assert Thread.holdsLock(this); at their beginning point.