[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r17078 - in trunk/subversion: mod_authz_svn

From: <kfogel_at_collab.net>
Date: 2005-11-03 15:51:16 CET

Greg Hudson <ghudson@MIT.EDU> writes:
> > Log:
> > Follow up to r17077: Use sizeof() instead of a hardcoded buffer length.
>
> Why are we using fixed-sized buffers at all?
>
> (I realize this seems to date to a pre-existing svn_strerror(), but it
> seems like a botch which should be corrected, not propagated to a new
> interface.)

I think we're doing it to avoid allocating memory, the assumption
being that running out of memory might have been the ultimate cause of
whatever error we're handling. At any rate, this was why I chose to
propagate the API rather than "fix" it. See the note in the docstring
of svn_err_best_message().

(I'm not wedded to this decision, just explaining it...)

-Karl

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Nov 3 17:10:48 2005

This is an archived mail posted to the Subversion Dev mailing list.