[cvs] re: bf_abort.c

[David Relson]

On Sun, 07 Sep 2003 02:37:05 +0200
Matthias Andree <matthias.andree at gmx.de> wrote:

> David Relson <relson at osagesoftware.com> writes:
> 
> > There's a simpler way to do this.  Add "#define exit(exitcode)
> > bf_abort()" in lexer_v3.l.  With that, all calls to setjmp/longjmp
> > go away.
> 
> I'd rather not hack around in lexer stuff.

Matthias,

Having bf_abort() call close_wordlists() is appropriate for bogofilter
and bogoutil, but not for bogolexer, fgetsl_test, etc.  I'm thinking we
should let the compiled flex code use exit(YY_EXIT_FAILURE), which is
what it's coded to do, and that we should use:

	void bf_exit(void)
	{
	   close_wordlists(false);
	}

	onexit(bf_exit);

David