On Thursday 14 September 2006 21:44, Oleg Broytmann wrote:
> On Thu, Sep 14, 2006 at 09:38:59PM +0300, Dan Pascu wrote:
> > BTW, what is the status on this? Do you intend to include it? If so,
> > what is holding it back for so long?
>
>    I didn't want to include in 0.7.1. I wanted to apply it after
> releasing 0.7.1.
>    But now the question is - does it really help? Does the original
> poster have a different problem not covered by the patch?

It helps to take off the burden from the garbage collector by removing a 
known circular reference, which is a good thing to do in my opinion. The 
garbage collector knows how to deal with circular references, but that is 
just to protect programs against overlooked references, not to encourage 
programmers to grow careless and ignore them if they know about them.

I do not know what problem the original poster has, but I'm guessing that 
he experiences something else, because the circular reference problem is 
not evident unless you also enable memory tracing, in which case the 
garbage collector no longer frees circular references. Else the garbage 
collector does its job and eliminates the referenced objects, but as I 
said with a performance penalty which can be avoided if the patch is 
applied.

So even if the specific problem of the original poster is not solved by 
this patch, it doesn't mean that the patch is not useful. Removing the 
circular references that we know about is always a good thing to do and 
is also harmless.

-- 
Dan

-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
sqlobject-discuss mailing list
sqlobject-discuss@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sqlobject-discuss

Reply via email to