Author: markt Date: Tue Mar 25 23:28:43 2014 New Revision: 1581597 URL: http://svn.apache.org/r1581597 Log: Code review - lock outside try/catch/finally
Modified: tomcat/trunk/java/javax/el/Util.java Modified: tomcat/trunk/java/javax/el/Util.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/javax/el/Util.java?rev=1581597&r1=1581596&r2=1581597&view=diff ============================================================================== --- tomcat/trunk/java/javax/el/Util.java (original) +++ tomcat/trunk/java/javax/el/Util.java Tue Mar 25 23:28:43 2014 @@ -118,8 +118,8 @@ class Util { if (factory == null) { final Lock writeLock = cacheValue.getLock().writeLock(); + writeLock.lock(); try { - writeLock.lock(); factory = cacheValue.getExpressionFactory(); if (factory == null) { factory = ExpressionFactory.newInstance(); --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org