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

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

From: Malcolm Rowe <malcolm-svn-dev_at_farside.org.uk>
Date: 2007-01-08 22:28:47 CET

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

  • application/pgp-signature attachment: stored
Received on Mon Jan 8 22:28:55 2007

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