It defaults to the global recipient_delimiter, which should always be used
anyway. Don't add confusion by documenting the legacy setting that only
exists for backwards compatibility.

Use local variables to pre-declare conditions that are used more than once.

Caused by recent event API changes.

Caused by recent event API changes.

It happened only when returning error, so the caller wouldn't have cared
about it anyway. This makes static analyzers happier though.

Simplifies the next commit.

Avoids growing memory usage excessively when filtering a lot of mails.

This didn't seem to result in any visible bugs though.

This is needed to avoid duplicating log lines there when the system log handler
is removed from the Sieve context.

This way, it matches the new sieve_logv().

This reduces the number of log functions for the error handler from four to just
one.