Re: [PATCH] Enable error lifetime checking by default in maintainer mode

On Mon, Jan 08, 2007 at 10:25:07AM -0800, Daniel Rall wrote:
> On Sun, 07 Jan 2007, Malcolm Rowe wrote:
> > Enable error lifetime checking by default in maintainer mode, causing
> ...
>
> I'm in favor of this change.

r22934, thanks, both.

> Alternately, SVN_DEBUG_ERROR could be
> defined when SVN_DEBUG is defined. This is only slightly more
> complex, but does leave in a convenient toggle for people who need
> more control over things.
>

I thought about doing something like that (actually, defining an
SVN_DEBUG_NO_ERROR to disable it instead, inverting the #ifdefs), but
I thought it would be easier to see if anyone violently objected to
enabling it by default instead.

I figure if anyone really wants to disable it, 'svn merge -c-22934 .' is
just as easy as commenting out an #ifdef SVN_DEBUG / #define SVN_DEBUG_ERROR.

Regards,
Malcolm