[PATCH] Optionally abort on internal errors.

Milan Broz <[email protected]> Sun, 29 Nov 2009 21:34:11 +0100
Newsgroups dev.linux.lists.lvm-devel
Message-ID <[email protected]>
On 11/29/2009 08:26 PM, Petr Rockai wrote:
> a fairly straightforward patch adding a configure option that will
> abort() the process in case it encounters an internal
> error. Undocumented so far and not sure it's appropriate to expose --
> but maybe we have a policy of no undocumented options?

I like that idea, very useful for test suite.

ACK from me.

just some nitpicking changes like

+	if (level <= _LOG_ERR && !strncmp(format, "Internal error:",
+					  strlen("Internal error:")))
(why strlen? just len or better #define the string)

- should it be in log section (global?)

- maybe only activate that when in debug (verbose mode)?

Milan