Re: send_xml returns void in mod_dav_svn/update.c?

On Mon, Nov 10, 2003 at 04:08:43PM -0600, kfogel@collab.net wrote:
> Greg, any idea why we ignore the result from apr_brigade_vprintf() and
> just return void from send_xml() in this mod_dav_svn/update.c code:
>...
> Almost all callers of send_xml() return (svn_error_t *), so if it
> returned an error we would have a way to propagate that error.
>
> But, there's probably some reason it's void?...

No particular reason. The potential for an error condition there is pretty
small. Although (thinking about it), if the connection was terminated, you
could abort the report process. (the brigade function has a fast-path
return, but you'd still chunk thru the FS work)

Changing it to an svn_error_t, and propagating that should be just fine.
I'd suggest that somewhere in the chain of callers of send_xml(), somebody
should use use SVN_ERR_W() to target what was going on.

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org