abort or assert?

In most of the places where we currently say:

   if (blah) abort();

I feel we should be saying:

   assert(!blah);

There may be some reasons to abort() in a few cases, such as to avoid possible
data loss. If I prepare a patch to convert to assert() where it seems
appropriate, perhaps someone could review it with this in mind.

Does this sound reasonable?

- Julian

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Nov 9 16:40:48 2004