Yeah, the confusion with the memory bug was partly due to me. After your
original fix that still had a problem with the reader I introduced the
bug again when I tried to fix that.
So the lastconsumer null test really works, you only broke it with your reader fix. I guess you have to stand a beer to Carsten :)
And we all have not reviewed the patch correctly as the problem is obvious:
if (lc == null) {
lc = anObject;
}if (lc != null) {
//always true
//executing code introducing memory leak in special cases
}Shame on all of us ;) Maybe we all have to stand a beer to Carsten at next CocoonGetTogether as the release is rescued.
Joerg
