lib-smtp: smtp-params - Remove code duplication between smtp_params_*_drop_extra().

lib-smtp: smtp-params - Remove code duplication between smtp_params_*_add_extra().

lib-smtp: smtp-params - Remove code duplication between smtp_params_*_get_extra().

lib-smtp: smtp-syntax - Add smtp_xtext_decode().

lib-smtp: smtp-params - Rename smtp_params_rcpt_equals() to smtp_params_rcpt_equal().

lmpt: Reformat lmtp-proxy.c.

lmtp: Reformat lmtp-commands.c.

lmtp: Reformat lmtp-client.c.

lib-smtp: Reformat smtp-client-connection.h.

lib-smtp: Reformat smtp-client-connection.c.

lib-storage: mail_search_args_init() - Expand "*" in SEARCH_SEQSET

This is now required by the IMAP MOVE code to correctly handle
"MOVE * folder".

imap: imap_search_seqset_iter_next() - Assert-crash if iteration doesn't progress

This can be done easily with seq_range_array_remove_seq_range(). This is
cleaner to use than invert+intersect. I originally didn't notice that
this function existed.

lib: Add unit test for seq_range_array_remove_range()

lib-storage: mail_search_args_init() - Fix converting UIDSET * to SEQSET on empty mailbox

The "*" caused seq=0 to be added to the seqset. This could have caused
unexpected issues.

Fixes at least UID MOVE on empty mailbox:
Panic: file seq-range-array.c: line 471 (seq_range_array_invert): assertion failed: (range[0].seq1 >= min_seq)

Before 1a5d89d2bfa031903e88af9aff7eafc1b373d521 this assert-crash didn't
happen, but it went to infinite loop.

lib-imap-client: test-imapc-client - Use the new sub-process test API.

This makes an effort to terminate server processes gracefully. Killing them
bluntly hampers test coverage measurement and valgrind testing.

lib-imap-client: test-imapc-client - Add server debug messages.

lib-imap-client: test-imapc-client - Split off test_run_server/client() from test_run_client_server().

lib-imap-client: test-imapc-client - Add log prefixes to distinguish client and server debug messages.

lib-imap-client: test-imapc-client - Make sure the child server process is killed.

lib-imap-client: test-imapc-client - Use getopt() for parsing command line arguments.

lib-imap-client: test-imapc-client - Add structural comments.

lib-imap-client: test-imapc-client - Move client functions.

lib-imap-client: test-imapc-client - Move server functions.

lib-imap-client: test-imapc-client - Move test_open_server_fd().

lib-imap-client: Reformat test-imapc-client.c.

lib-imap-client: Remove unnecessary "Authentication failed: disconnected" errors

If connection failed during login, there were always two log lines logged.
The first line is enough.

lib-imap-client: test-imapc-client - Client now sends LOGOUT and server expects it

lib-imap-client: test-imapc-client - login_hangs: Client stopped too early

lib-imap-client: test-imapc-client - login_hangs: Server checked wrong connection's input

master: test-master-login-auth - Use the new sub-process test API.

This makes an effort to terminate server processes gracefully. Killing them
bluntly hampers test coverage measurement and valgrind testing.

master: test-master-login-auth - Split off test_run_server/client() from test_run_client_server().

master: test-master-login-auth - Rename test_server_kill() to test_servers_kill_forced().

master: test-master-login-auth - Move global initialization to main_init/deinit().

master: test-master-login-auth - Avoid using sleep()/usleep().

master: test-master-login-auth - Use lib-signals API.

Using signal() is not portable for installing a signal handler.

master: test-master-login-auth - Ignore SIGPIPE.

master: test-master-login-auth - Don't ignore SIGCHLD.

master: test-master-login-auth - Add log prefixes to distinguish client and server debug messages.

master: test-auth-master - Use the new sub-process test API.

This makes an effort to terminate server processes gracefully. Killing them
bluntly hampers test coverage measurement and valgrind testing.

master: test-auth-master - Split off test_run_server/client() from test_run_client_server().

master: test-auth-master - Rename test_server_kill() to test_servers_kill_forced().

master: test-auth-master - Move global initialization to main_init/deinit().

master: test-auth-master - Avoid using sleep()/usleep().

master: test-auth-master - Use lib-signals API.

Using signal() is not portable for installing a signal handler.

master: test-auth-master - Ignore SIGPIPE.

master: test-auth-master - Don't ignore SIGCHLD.

master: test-auth-master - Add log prefixes to distinguish client and server debug messages.

master: test-auth-client - Use the new sub-process test API.

This makes an effort to terminate server processes gracefully. Killing them
bluntly hampers test coverage measurement and valgrind testing.

master: test-auth-client - Split off test_run_server/client() from test_run_client_server().

master: test-auth-client - Rename test_server_kill() to test_servers_kill_forced().

master: test-auth-client - Move global initialization to main_init/deinit().

master: test-auth-client - Avoid using sleep()/usleep().

master: test-auth-client - Use lib-signals API.

Using signal() is not portable for installing a signal handler.

master: test-auth-client - Ignore SIGPIPE.

master: test-auth-client - Don't ignore SIGCHLD.

master: test-auth-client - Add log prefixes to distinguish client and server debug messages.

master: Reformat test-master-login-auth.c.

master: Reformat test-auth-master.c.

master: Reformat test-auth-client.c.

lib-smtp: test-smtp-submit - Use the new sub-process test API.

This makes an effort to terminate server processes gracefully. Killing them
bluntly hampers test coverage measurement and valgrind testing.

lib-smtp: test-smtp-submit - Split off test_run_server/client() from test_run_client_server().

lib-smtp: test-smtp-submit - Rename test_servers_kill_all() to test_servers_kill_forced().

lib-smtp: test-smtp-submit - Move global initialization to main_init/deinit().

lib-smtp: test-smtp-submit - Avoid using sleep().

lib-smtp: test-smtp-submit - Use lib-signals API.

Using signal() is not portable for installing a signal handler.

lib-smtp: test-smtp-submit - Ignore SIGPIPE.

lib-smtp: test-smtp-submit - Don't ignore SIGCHLD.

lib-smtp: test-smtp-submit - Add log prefixes to distinguish client and server debug messages.

lib-smtp: test-smtp-server-errors - Use the new sub-process test API.

This makes an effort to terminate client processes gracefully. Killing them
bluntly hampers test coverage measurement and valgrind testing.

lib-smtp: test-smtp-server-errors - Split off test_run_client/server() from test_run_client_server().

lib-smtp: test-smtp-server-errors - Rename test_clients_kill_all() to test_clients_kill_forced().

lib-smtp: test-smtp-server-errors - Move global initialization to main_init/deinit().

lib-smtp: test-smtp-server-errors - Fix name of client_connection_deinit().

lib-smtp: test-smtp-server-errors - Use lib-signals API.

Using signal() is not portable for installing a signal handler.

lib-smtp: test-smtp-server-errors - Ignore SIGPIPE.

lib-smtp: test-smtp-server-errors - Remove useless child signal handler.

Restructure code to better match equivalent unit tests.

lib-smtp: test-smtp-server-errors - Adjust log prefix code to match other similar unit tests.

lib-smtp: test-smtp-client-errors - Use the new sub-process test API.

This makes an effort to terminate server processes gracefully. Killing them
bluntly hampers test coverage measurement and valgrind testing.

lib-smtp: test-smtp-client-errors - Split off test_run_server/dns/client() from test_run_client_server().

lib-smtp: test-smtp-client-errors - Rename test_servers_kill_all() to test_servers_kill_forced().

lib-smtp: test-smtp-client-errors - Move global initialization to main_init/deinit().

lib-smtp: test-smtp-client-errors - Avoid using sleep().

lib-smtp: test-smtp-client-errors - Use lib-signals API.

Using signal() is not portable for installing a signal handler.

lib-smtp: test-smtp-client-errors - Generally ignore SIGPIPE.

Remove per-process code that achieves the same.

lib-smtp: test-smtp-client-errors - Don't ignore SIGCHLD.

lib-smtp: test-smtp-client-errors - Turn server-side payload read error into a debug message.

This can readily occur during tests and must not issue a real error message.

lib-smtp: test-smtp-client-errors - Add log prefixes to distinguish client and server debug messages.

lib-smtp: test-smtp-payload - Use the new sub-process test API.

This makes an effort to terminate server processes gracefully. Killing them
bluntly hampers test coverage measurement and valgrind testing.

lib-smtp: test-smtp-payload - Split off test_run_client/server() from test_run_client_server().

lib-smtp: test-smtp-payload - Adjust log prefix code to match other similar unit tests.

lib-smtp: test-smtp-payload - Rename test_server_kill() to test_server_kill_forced().

lib-smtp: test-smtp-payload - Move global initialization to main_init/deinit().

lib-smtp: test-smtp-payload - Call test_files_init/deinit() in test_run_client_server().

lib-smtp: test-smtp-payload - Use lib-signals API.

Using signal() is not portable for installing a signal handler.

lib-smtp: test-smtp-payload - Properly exit child process.

lib-smtp: test-smtp-payload - Don't ignore SIGCHLD.

lib-master: master-service - Add master_service_deinit_forked().

lib-smtp: test-smtp-server-errors - Fix "many bad commands" test.

It didn't properly handle the two parallel connections. Removed a useless
context struct definition. Modified debug message for easier debugging.

lib-smtp: Reformat test-smtp-submit.c.

lib-smtp: Reformat test-smtp-server-errors.c.