Re: [PATCH] possible improvement to svn log with "forward" revision range (issue 3830)

Thank you for your feedback, Philip.
I have attached an updated patch.

> A short comment along the lines 'Sets *REVISION_DELETED ...'

I have added a comment for function check_for_deleted_rev.

> No whitespace before ')'.
>
> Split long line before '&& session_opt_revnum'

I have updated the formatting of the source code.

> You don't clear err, so if the following two calls return an error then
> err leaks.

I have modified the logic how the errors are handled (and now correctly cleared).

I needed to introduce an error when the REVISION_DELETED can not be determined:
   return svn_error_create
     (SVN_ERR_CLIENT_BAD_REVISION, NULL,
      _("Could not determine revision where target was deleted"));

> A temporary session is used to identify the deleted revision, and then
> another session is created to run log. Since the temporary session
> "works" could it be permanent and used for log?

I have modified the code to reuse the iterpool which is later used for fetching the log.
I hope it is used correctly - i am not very familiar with this stuff yet.

Is there any more feedback which i should integrate?

Dirk