Re: [PATCH] issue 2154 - svn blame on a directory over DAV does not return gracefully

Attached is a patch based on Justin's comment.

'make davcheck' gave some errors in different test cases, merge_tests.py
once, but they didn't recur when run separately, leading me to think it
may have been due to my httpd performance configuration. 'make check'
passed.

Regards
Sameer

Change report functions to check for interim errors before calling
ap_fflush(). This avoids creating the HTTP packet header with the
default (and currently invalid) HTTP 200 status.

* subversion/mod_dav_svn/file_revs.c
  (svn_repos_get_file_revs):
     Call ap_fflush only if there is no existing error

* subversion/mod_dav_svn/log.c
  (dav_svn__log_report): Ditto

* subversion/mod_dav_svn/version.c
  (dav_svn__drev_report,
   dav_svn__get_locations_report): Ditto

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org