You are viewing a plain text version of this content. The canonical link for it is here.
Posted to ojb-dev@db.apache.org by ar...@apache.org on 2004/02/10 16:18:53 UTC
cvs commit: db-ojb/src/java/org/apache/ojb/broker/accesslayer RsIterator.java
arminw 2004/02/10 07:18:53
Modified: src/java/org/apache/ojb/broker/accesslayer RsIterator.java
Log:
fix, make sure that the reference object in weak map was not
use when became 'null'
Revision Changes Path
1.58 +10 -7 db-ojb/src/java/org/apache/ojb/broker/accesslayer/RsIterator.java
Index: RsIterator.java
===================================================================
RCS file: /home/cvs/db-ojb/src/java/org/apache/ojb/broker/accesslayer/RsIterator.java,v
retrieving revision 1.57
retrieving revision 1.58
diff -u -r1.57 -r1.58
--- RsIterator.java 6 Feb 2004 13:01:52 -0000 1.57
+++ RsIterator.java 10 Feb 2004 15:18:53 -0000 1.58
@@ -1015,27 +1015,30 @@
public void beforeClose(PBStateEvent event)
{
- if(ref != null && ref.get() != null)
+ if(ref != null)
{
- ((RsIterator)ref.get()).releaseDbResources();
+ RsIterator rs = (RsIterator) ref.get();
+ if(rs != null) rs.releaseDbResources();
ref = null;
}
}
public void beforeRollback(PBStateEvent event)
{
- if(ref != null && ref.get() != null)
+ if(ref != null)
{
- ((RsIterator)ref.get()).releaseDbResources();
+ RsIterator rs = (RsIterator) ref.get();
+ if(rs != null) rs.releaseDbResources();
ref = null;
}
}
public void beforeCommit(PBStateEvent event)
{
- if(ref != null && ref.get() != null)
+ if(ref != null)
{
- ((RsIterator)ref.get()).releaseDbResources();
+ RsIterator rs = (RsIterator) ref.get();
+ if(rs != null) rs.releaseDbResources();
ref = null;
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: ojb-dev-unsubscribe@db.apache.org
For additional commands, e-mail: ojb-dev-help@db.apache.org