Re: problem revealed by issue #2398 (server-side assertion)

On 2/13/06, kfogel@collab.net <kfogel@collab.net> wrote:

> Any thoughts about this patch's general approach?

Why change from an assert to a SVN_ERR? The assert was wrong because
of the pointer comparison, not because of what it's testing, it's
still fundamentally misusing the API to pass two different filesystems
to this function, so why not make it an assert?

-garrett

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Feb 14 00:31:45 2006