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

Re: svn commit: r1222522 - /subversion/branches/1.7.x/STATUS

From: <schamel23_at_spinor.com>
Date: Sun, 25 Dec 2011 10:20:20 +0100

On 2011-12-25 07:00, Branko Čibej wrote:
> On 25.12.2011 01:01, Mark Mielke wrote:
>> Hey all. Just reading and I'm surprised that Subversion uses assert()
>> in production code. I don't think this is typical practice.
>
> You should read my rant about that in this thread. :)

Assertions should never call abort.
Assertions should be just like any error,
possibly providing more information like a call stack.

Like in Python where assertions are just normal exceptions,
which can be caught normally including all call stack information
(which for example in our case are logged into a log file
which a customer can send us. So we get the best of both worlds:
An assert does not crash the app, the app can continue,
but we can still get the callstack information).

Folker
Received on 2011-12-25 10:20:55 CET

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