Since the script storage code is widely engrained in the Pigeonhole sources,
this is a very large commit that cannot be reduced further into smaller
preparatory bits.

lib-sieve-tool: sieve-tool - Rename filename to location parameter for sieve_tool_script_compile/open()

It's no longer used by anything, and the setting doesn't exist anymore.

The settings were already read at this point, so this didn't actually do
anything. Fixing the code to actually set it would break existing tests,
so just remove it.

Also change the callers to not bother storing it separately. Instead, they
can just use mail_user_r->service_user.

Fixes argument list too long error.

Change was:

lib-mail: message_address_parse() - Change fill_missing parameter to flags

Signed-off-by: Phil Carmody <phil@dovecot.fi>

Needed for use in test suite.

Dovecot core doesn't have copyright notices in header files.

Cleanup performed with the following semantic patch:

	@@
	expression E;
	@@

	- if (E != NULL) {
	- 	i_stream_unref(&E);
	- }
	+ i_stream_unref(&E);

	@@
	expression E;
	@@

	- if (E != NULL) {
	- 	o_stream_unref(&E);
	- }
	+ o_stream_unref(&E);