On Tue, 2007-01-16 at 15:58 -0800, Daniel Rall wrote:
> On Tue, 16 Jan 2007, Bhuvaneswaran Arumugam wrote:
>
> > On Tue, 2007-01-09 at 15:11 +0530, Bhuvaneswaran Arumugam wrote:
> > > Hello,
> > >
> > > This is a follow-up patch for r22933. Please find the attached patch.
> >
> > Any review comment for this?
>
> Looks good to me, +1.
So, shall i commit this one?
> > > [[[
> > > Destroy the APR subpool before every successful return statement.
> > >
> > > [in subversion/libsvn_client/]
> > > * delete.c
> > > (delete_urls):
> > > * repos_diff.c
> > > (svn_client__get_diff_editor):
> > > * externals.c
> > > (switch_external):
> > > * list.c
> > > (get_dir_contents):
> > > Destroy the APR subpool before returning successfully to prevent
> > > iterative pool creation.
> > >
> > > Patch by: bhuvan
> > > ]]]
>
> I wish the log message could be more compact, though I'm not sure
> exactly how I'd change it.
If someone can make this better, thats fine.
--
Regards,
Bhuvaneswaran
Received on Mon Jan 22 10:42:35 2007