RE: Barracuda: CVS Update - Memory leak fix
Jack Hodges <[email protected]>
| Newsgroups | gmane.comp.java.enhydra.barracuda.general |
|---|---|
| Message-ID | <[email protected]> |
Christian, Did you review and check in the BaseEvent setParam/getParam code? Jack -----Original Message----- From: Christian Cryder [mailto:[email protected]] Sent: Thursday, December 12, 2002 9:44 AM To: Barracuda Subject: Barracuda: CVS Update - Memory leak fix Hey folks! Just a note, I have reviewed and checked in both of the patches supplied by Shawn (comments below). I also added the utility method getCookie() requested by Jake. Note that this patch solves a (potentially serious) memory leak in Barracuda, so it would be well worth your while to upgrade to the latest from cvs... Thanks tremendously (and kudos) to Shawn for making the effort to track this one down! -------------------------------------------------------- <b>saw_121102.2</b> - Added in Cookie support to the HttpRequester class in Plankton. This effectively lets you connect to a URL that returns a cookie, and then you can reuse the requester for other URLs that are going to require that cookie. Thanks to Shawn Wilson [[email protected]] for implementing this. <b>saw_121102.1</b> - Updated BTemplate and TemplateHelper to properly release references between the component and the models. Basically, whenever you use a BTemplate the destroyCycle() method MUST be called when you're done with it, or the template models will retain a reference to the BTemplate, and the objects will never get cleaned up. This was a problem in TemplateHelper where we create temporary BTemplate components and then discard them. We also had to modify BTemplate itself so that a) if you add a model it first tries to deregister the model (ie. in case you are overwriting an existing model, which BlockIterator does) and b) when destroy cycle is called we need to run through and deregister all the models before nulling out the model list. This effectively fixes the only known memory leak in Barracuda, so its a very important patch. Thanks to Shawn Wilson [[email protected]] for contributing</p> -------------------------------------------------------- Holler if you have questions... Christian ---------------------------------------------- Christian Cryder [[email protected]] Internet Architect, ATMReports.com Barracuda - http://barracuda.enhydra.org ---------------------------------------------- "Coffee? I could quit anytime, just not today" _______________________________________________ Barracuda mailing list [email protected] http://www.enhydra.org/mailman/listinfo.cgi/barracuda FAQ - http://www.jguru.com/faq/Barracuda _______________________________________________ Barracuda mailing list [email protected] http://www.enhydra.org/mailman/listinfo.cgi/barracuda FAQ - http://www.jguru.com/faq/Barracuda