On Wed, 2012-01-18 at 13:34 -0800, Karthik K wrote: > On Wed, Jan 18, 2012 at 12:30 PM, Oleg Kalnichevski <[email protected]>wrote: > > > On Wed, 2012-01-18 at 11:56 -0800, Karthik K wrote: > > > As part of httpclient development, I find myself writing these helper > > > utilities to release resources ( HttpClient / HttpEntity ) etc , very > > > similar to IOUtils , from commons.io to release resources , that closes > > > 'quietly' . > > > > > > > > > o.a.h.utils.HttpClientUtils > > > closeQuietly(HttpClient) > > > and closeQuietly(HttpResponse response) > > > > > > https://issues.apache.org/jira/browse/HTTPCLIENT-1159 . > > > > > > + some test cases as well. > > > > > > Let me know your thoughts on the same. > > > > > > -- > > > Karthik. > > > > I'll happily commit the patch to SVN trunk if no objections are raised > > within a day or two. One minor issue is that the proposed patch makes > > use of static Log instances that generally are to be avoided. Otherwise > > the patch looks good to me. > > > > Cheers > > > > Oleg > > > > > Thanks, Oleg. > > I have attached a version that has no logging at all altogether. > > My $0.02 - given that this is independent with backward compatibility > still, would it be possible to backport this to 4.1.x releases , to help > incorporate this in our codebases , sooner than later.
The evil side of me wants you to upgrade to 4.2 sooner ;-) Besides, we should try to keep patch releases what they are meant to be - fully backward and forward compatible drop-in replacements. Oleg --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
