RE: Barracuda: CVS Update - Memory leak fix
"Christian Cryder" <[email protected]>
| Newsgroups | gmane.comp.java.enhydra.barracuda.general |
|---|---|
| Message-ID | <[email protected]> |
Yep! Last week (12/6)...I probably forgot to announce it on the list, although if you are subscribed to the Barracuda-commits list you would have seen it there. Here are the details: -------------------------- <b>csc_120602.2</b> - Bunch of minor formatting changes (ie. remove tabs that crept into the code, etc <b>csc_120602.1</b> - Change HttpServletRequestWrapper so that getParameterMap returns map values as String[], like its supposed to according to the spec. Also make it handle multiple values (previously it didn't). Thanks to Udovenko Sergey [[email protected]] for bringing this to my attention. <b>csc_120502.1</b> - Added support to BaseEvent class to support the notion of params. This means adding the following methods: - void setParam(String key, String val) - void setParam(String key, String[] vals) - Map getParams() You can use these events to "add" parameters to a URL request. Basically, if you shove any values in here, then when this event is dispatched (by adding it to the event queue) then the param values will get pulled out and appended into the underlying Http req object. Thus to the code that handles this event, it will look like the parameters where there all along. Pretty slick! Thanks to Jack Hodges [[email protected]] for submitting this patch ---------------------------------------------- Christian Cryder [[email protected]] Internet Architect, ATMReports.com Barracuda - http://barracuda.enhydra.org ---------------------------------------------- "Coffee? I could quit anytime, just not today" > -----Original Message----- > From: [email protected] [mailto:[email protected]]On > Behalf Of Jack Hodges > Sent: Thursday, December 12, 2002 1:38 PM > To: '[email protected]' > Subject: RE: Barracuda: CVS Update - Memory leak fix > > > 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 _______________________________________________ Barracuda mailing list [email protected] http://www.enhydra.org/mailman/listinfo.cgi/barracuda FAQ - http://www.jguru.com/faq/Barracuda