You are viewing a plain text version of this content. The canonical link for it is here.
Posted to derby-commits@db.apache.org by kr...@apache.org on 2008/02/20 14:53:08 UTC

svn commit: r629457 - /db/derby/code/trunk/java/client/org/apache/derby/client/am/stmtcache/JDBCStatementCache.java

Author: kristwaa
Date: Wed Feb 20 05:53:05 2008
New Revision: 629457

URL: http://svn.apache.org/viewvc?rev=629457&view=rev
Log:
DERBY-3324: JDBC statement cache implementation.
Added a check for null reference when evicting an entry.
Patch file: n/a

Modified:
    db/derby/code/trunk/java/client/org/apache/derby/client/am/stmtcache/JDBCStatementCache.java

Modified: db/derby/code/trunk/java/client/org/apache/derby/client/am/stmtcache/JDBCStatementCache.java
URL: http://svn.apache.org/viewvc/db/derby/code/trunk/java/client/org/apache/derby/client/am/stmtcache/JDBCStatementCache.java?rev=629457&r1=629456&r2=629457&view=diff
==============================================================================
--- db/derby/code/trunk/java/client/org/apache/derby/client/am/stmtcache/JDBCStatementCache.java (original)
+++ db/derby/code/trunk/java/client/org/apache/derby/client/am/stmtcache/JDBCStatementCache.java Wed Feb 20 05:53:05 2008
@@ -148,7 +148,7 @@
          */
         protected boolean removeEldestEntry(Map.Entry eldest) {
             final boolean remove = size() > maxSize;
-            if (remove) {
+            if (remove && eldest != null) {
                 try {
                     ((PreparedStatement)eldest.getValue()).close();
                 } catch (SQLException ex) {