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);

The parent event doesn't really matter here, so NULL is used.

Now using struct smtp_address rather than sieve_address.
In some cases, struct message_address may prove to be more suitable, but this is the best fit for now.
Adjusted to the renamed fields in struct mail_deliver_context.

It's replaced with fd-util.h which is automatically included now

Rather than continuing with binary == NULL, causing a segfault.