[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: Wed, 28 Dec 2011 11:57:47 +0100

On 2011-12-27 10:50, Branko ÄŒibej wrote:
> On 27.12.2011 09:57, schamel23_at_spinor.com wrote:
>> On 2011-12-25 10:57, Branko ÄŒibej wrote:
>>> On 25.12.2011 10:20, schamel23_at_spinor.com wrote:
>>>> 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.
>>>
>>> You should tell that to the designers of the C runtime library. :)
>>
>> You mean the GNU runtime library,
>
> No, I mean the C runtime library. Read a C textbook if you don't believe
> me. assert() is required to call abort() if the condition is false.

I believe you, you obviously know it better than me.

Just my point is still;
abort() may be fine for command line tools,
but is silly for libraries or GUI applications.
Today's software industry has evolved beyond the seventies. ;-)

Folker
Received on 2011-12-28 11:58:23 CET

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.