pop: move refno

`refno` isn't used outside of the POP backend.
Move `Email->refno` to `PopEmailData->refno`.

The value was saved in the Header Cache, but the restored value was never used (see pop_fetch_headers()).

pop: refactor Email->edata

Use a wrapper function around Email->edata, like the other backends.

remove used member

Body->is_signed_data was introduced 15 years ago and never used.

Clear connection input buffer when closing

An RFC-violating reply from the Dovecot SMTP submission service
uncovered a bug in Mutt.  Mutt was not clearing out the input buffer
when closing the connection.

Dovecot was including an extra 250 response after data submission:

  DATA
  354 OK
  Subject: Test 3

  Test 3
  .
  250 2.0.0  223 byte chunk, total 223
  250 2.0.0 OK id=1iIqT1-0004wS-Ac
  quit
  221 2.0.0 Bye

The multiline 250 reply requires a hyphen in the first response.  Mutt
closes the connection after the quit, but ends up leaving the
unexpected second line in the input buffer.  This causes an error in
the next usage of the connection.

Clean out bufpos and available when closing the socket.

Co-authored-by: Richard Russon <rich@flatcap.org>

merge: add $toggle-quoted-show-levels feature

 * Add ability to limit levels hidden with toggle-quoted command
 * Give (*line_info)[n] a name

Give (*line_info)[n] a name

Add ability to limit levels hidden with toggle-quoted command

Closes: #1802
By setting the configuration variable `toggle_quoted_show_levels`, when you
execute the `toggle-quoted` command, the specified number of levels
will continue to be visible.

fix: hcache namer function

Fix error introduced when applying upstream Buffer refactoring.

merge: refactor colour handling / notifications

 * rename and reorder color constants
 * tidy whitespace
 * rename vars
 * convert retval to CommandResult
 * tidy code
 * rename/move function
 * do_uncolor change param to retval
 * encapsulate all the colours
 * fix max quote depth
 * encapsulate defs
 * encapsulate quoted
 * encapsulate color_list
 * move mutt_color_free
 * move parse_object
 * initialise lists
 * move color_line_new
 * move mutt_colors_free
 * deglobalise Colors
 * implement more uncolor commands
 * colour notifications
 * sort symbols
 * docs: color/uncolor

docs: color/uncolor

sort symbols

colour notifications

implement more uncolor commands

deglobalise Colors

move mutt_colors_free

move color_line_new

initialise lists

move parse_object

move mutt_color_free

encapsulate color_list

encapsulate quoted

encapsulate defs

fix max quote depth

encapsulate all the colours

do_uncolor change param to retval

rename/move function

tidy code

convert retval to CommandResult

rename vars

tidy whitespace

rename and reorder color constants

Let reply_to take precedence over C_ReplySelf

merge: upstream refactoring

 * Re-enable and cleanup format-flowed space stuffing
 * Remove manual note about space-stuffing only once
 * Reuse the static space_stuff function for unstuffing
 * Convert crypt-gpgme to use struct Buffer for tempfiles
 * Convert crypt_extract_keys and signed handler to use buffers
 * Convert edit_one_message() to use buffer pool
 * Convert run_decode_and_handler() to use buffer pool
 * Convert hcache db4 lockfile to buffer
 * Convert hcache_open to use buffer pool
 * Fix built-in pager checks for help and attachments
 * Convert mutt_edit_headers() to use buffer pool
 * Convert mutt_help() to use buffer for tempfile
 * Convert save-hook and fcc-hook to use buffer pool internally
 * Fix mutt_write_mime_body() application/pgp-encrypted handling
 * Convert imap_open_new_message() to use buffer pool
 * Convert imap fetch_new and fetch_message tempfiles to buffer pool
 * Convert imap_cachepath() and callers to use buffers
 * Convert parse_path_(un)list to use buffer pool for path
 * Convert start_debug() to use buffer pool

Convert start_debug() to use buffer pool

Co-authored-by: Richard Russon <rich@flatcap.org>

Convert parse_path_(un)list to use buffer pool for path

Don't use the buf parameter to avoid stretching it out, as it's used
for the entire rc parsing, and typically just holds command names.

Co-authored-by: Richard Russon <rich@flatcap.org>

Convert imap_cachepath() and callers to use buffers

Co-authored-by: Richard Russon <rich@flatcap.org>

Convert imap fetch_new and fetch_message tempfiles to buffer pool

Co-authored-by: Richard Russon <rich@flatcap.org>

Convert imap_open_new_message() to use buffer pool

Co-authored-by: Richard Russon <rich@flatcap.org>

Fix mutt_write_mime_body() application/pgp-encrypted handling

It would blindly overwrite any attachment with that type, as if it
were the version label part of a multipart/encrypted message.

However attachments with extenstion .gpg are labeled that type if
$mime_type_query_command is set to "xdg-mime query filetype".  This
would cause the attachment to be overwritten too.

Add a check for a missing body->filename, which should only be the
case for the manually constructed part.

Co-authored-by: Richard Russon <rich@flatcap.org>

Convert save-hook and fcc-hook to use buffer pool internally

The hook parameters still need to be converted, but I'm working
towards it slowly.

Co-authored-by: Richard Russon <rich@flatcap.org>

Convert mutt_help() to use buffer for tempfile

Co-authored-by: Richard Russon <rich@flatcap.org>

Convert mutt_edit_headers() to use buffer pool

Co-authored-by: Richard Russon <rich@flatcap.org>

Fix built-in pager checks for help and attachments

Mutt uses the built-in pager when $pager is unset too.  Fix a few
checks to also handle that case.

Co-authored-by: Richard Russon <rich@flatcap.org>

Convert hcache_open to use buffer pool

Change the namer function and mutt_hcache_per_folder to operate on a
buffer parameter.

Co-authored-by: Richard Russon <rich@flatcap.org>

Convert hcache db4 lockfile to buffer

Co-authored-by: Richard Russon <rich@flatcap.org>

Convert run_decode_and_handler() to use buffer pool

Co-authored-by: Richard Russon <rich@flatcap.org>

Convert edit_one_message() to use buffer pool

Co-authored-by: Richard Russon <rich@flatcap.org>

Convert crypt_extract_keys and signed handler to use buffers

Co-authored-by: Richard Russon <rich@flatcap.org>

Convert crypt-gpgme to use struct Buffer for tempfiles

Use the pool where possible, but when the function calls other
long-running functions (such as mutt_body_handler() or
mutt_do_pager()), allocate the buffer.

smime_gpgme_decrypt_mime() was not freeing the "cur" struct Body generated on
error.  Add a call to mutt_free_body(cur) in the error handling.

Co-authored-by: Richard Russon <rich@flatcap.org>

Reuse the static space_stuff function for unstuffing

Co-authored-by: Richard Russon <rich@flatcap.org>

Remove manual note about space-stuffing only once

Stuffing is now performed each time.

Co-authored-by: Richard Russon <rich@flatcap.org>

Re-enable and cleanup format-flowed space stuffing

Commit 04cb5bde tried to fix re-stuffing a postponed message more than
once, but unfortunately broke the normal case of composing a new
message.  So actually, space-stuffing has been turned off the past 7
years.

Move format=flowed parameter setting below the standard message
pre-processing block.  It shouldn't be performed for draft-files even
with $resume_draft_files set.  Moving out of the block makes that
clearer.

Create mutt_rfc3676_space_(un)stuff() functions, which check the
content type and stuff/unstuff apprpropriately.  Note that the
stuff/unstuff does not depend on $text_flowed, which is only in charge
of setting the format=flowed parameter.  This parameter can also come
from resumed/resent/draft messages and should still be respected.

Add unstuffing to mutt_prepare_template().  This is called by
postponed, resent, and draft files.  This will prevent double-stuffing
in those cases.

Unstuff/restuff around editing the message in the compose menu, to
keep everything transparent to the user.  I originally put the
stuffing *after* the compose menu, but previewing the messages in the
compose menu did not work properly in that case.  It's cleaner this
way too.

Change the stuff/unstuff functions to preserve the original
hdr->content->filename.  The "hack" previously used would interact
poorly with editable body files (mutt -i -E).  Fortunately space
stuffing was pretty much disabled except in unusual cases before.

Co-authored-by: Richard Russon <rich@flatcap.org>

fix: broken index_format

A chance alignment made enum symbol MT_COLOR_INDEX_NUMBER equal to 37
(ASCII %).  This was causing problems when inserting colour codes into
the index_format string.

build: fix autosetup warning

add: log warnings

Distinguish between errors and warnings in the log and displayed on screen.

  color warning brightyellow default

init.h fix cast

tidy do...while loops

fix crash on exit

Even if mx_mbox_open() fails, the Mailbox might still have an Account.

docs: update list of colours

Update the list of objects that can be coloured and give examples for
each category.

merge: upstream refactoring

 * Properly revert on an autocrypt postpone failure
 * Fix memory leak when attaching messages
 * Convert bcache to use buffer pools
 * Convert mutt_complete() to use the buffer pool
 * Convert compose check_attachments() to use buffer pool
 * Fix mutt_save_message() "enter_fname()" to use buffer version
 * Create mutt_buffer_get_field()
 * Convert buffer callers of mutt_get_field() to use buffer version
 * Convert compose menu fname variable to buffer
 * Convert compress setup_paths() to use buffer
 * Fix memory leak in compose write-fcc function
 * buffer: tidy checks for empty Buffers

buffer: tidy checks for empty Buffers

Fix memory leak in compose write-fcc function

The outer multipart was not being freed after writing.

Co-authored-by: Richard Russon <rich@flatcap.org>

Convert compress setup_paths() to use buffer

Co-authored-by: Richard Russon <rich@flatcap.org>

Convert compose menu fname variable to buffer

Co-authored-by: Richard Russon <rich@flatcap.org>

Convert buffer callers of mutt_get_field() to use buffer version

Co-authored-by: Richard Russon <rich@flatcap.org>

Create mutt_buffer_get_field()

This will reduce the buffer->dptr fiddling all over the place into one
spot until mutt_enter_string() can be converted.

Co-authored-by: Richard Russon <rich@flatcap.org>

Fix mutt_save_message() "enter_fname()" to use buffer version

Commit be632f11 converted this function to use the buffer pool, but
accidentally used the non-buffer prompt when there was already a
buffer-converted version.

Co-authored-by: Richard Russon <rich@flatcap.org>

Convert compose check_attachments() to use buffer pool

Modify error message so that the filename comes last, to prevent a
long path from hiding the error message or prompt.

Co-authored-by: Richard Russon <rich@flatcap.org>

Convert mutt_complete() to use the buffer pool

Add helper functions mutt_buffer_substrcpy() and
mutt_buffer_concatn_path().

Remove mutt_concatn_path() because mutt_complete() was the only
caller.

Co-authored-by: Richard Russon <rich@flatcap.org>

Convert bcache to use buffer pools

Co-authored-by: Richard Russon <rich@flatcap.org>

Fix memory leak when attaching messages

Setting actx->idx[]->content->parts to NULL causes a memory leak,
because message attachments store a BODY there that needs to be freed.

I looked through the various use cases of ci_send_message() and
actually believe content->parts will be NULL for all other cases
except message attachments.  From the comment added to
delete_attachment():

  Other ci_send_message() message constructors are careful to free
  any body->parts, removing depth:
   - mutt_prepare_template() used by postponed, resent, and draft files
   - mutt_copy_body() used by the recvattach menu and $forward_attachments.

  I believe it is safe to completely remove the "content->parts =
  NULL" statement.  But for safety, am doing so only for the case
  it must be avoided: message attachments.

Co-authored-by: Austin Ray <austin@austinray.io>

Properly revert on an autocrypt postpone failure

Co-authored-by: Austin Ray <austin@austinray.io>

merge: fix new mail behaviour

Fix a number of regressions to do with new mail.

 * fix: mbox_check_stats() retvals
 * fix mbox's handling of new mail
 * fix <change-folder> when $change_folder_next is set
 * fix sidebar colour for 'new' mailboxes
 * change check_stats() to return number of new emails

change check_stats() to return number of new emails

fix sidebar colour for 'new' mailboxes

This depends on the new-ness of a Mailbox, not the presence of new
emails.

fix <change-folder> when $change_folder_next is set

fix mbox's handling of new mail

`Mailbox.has_new` is tied to the timestamp of an mbox, not its count of
new mail.  Visiting a Mailbox will clear the flag, touch(1) the file.

Whether the new emails get changed to old emails depends on $mark_old.

fix: mbox_check_stats() retvals

Change mbox_check_stats() to clearly show when a Mailbox has new mail.
Return:
 * -1 Error
 *  0 Success, no new mail
 *  1 Success, some new mail

A bug was introduced when mbox_check_stats() was created that caused
'mbox' Mailboxes to always appear as if they had new mail.

drop unused variable

'suffix' hasn't been used since the iconv code was stripped out.

initialise gpgme pointers

These typedef'd types are all pointers, so initialise them to NULL.

doxy: tidy wrapper comments

window: fix clear_screen

The compose screen wasn't being cleared properly by just `clearok()`.

build: WIFEXITED is defined in sys/wait.h

merge: refactor use of curses

Tidy:
 * eliminate curses from Progress Bar
 * move colour definitions to color.h
 * tidy naming of color functions
 * move KeyEvent out of mutt_curses.h

Refactor:
 * SET_COLOR() to mutt_curses_set_color()
 * NORMAL_COLOR() to mutt_curses_set_color()
 * ATTR_SET() to mutt_curses_set_attr()
 * curs_set() to mutt_curses_set_cursor()
 * beep() to mutt_beep()
 * addch() to mutt_window_addch()
 * addnstr() to mutt_window_addnstr()
 * addstr() to mutt_window_addstr()
 * clrtobot() to mutt_window_clrtobot()
 * move() to mutt_window_move_abs()
 * printw() to mutt_window_printf()
 * mutt_window_mvaddstr()
 * clearok() to mutt_window_clear_screen()
 * mutt_window_getxy() to mutt_window_get_coords()

refactor: mutt_window_getxy() to mutt_window_get_coords()

Encapsulate a curses function to reduce dependencies.

refactor: clearok() to mutt_window_clear_screen()

Encapsulate a curses function to reduce dependencies.

refactor: mutt_window_mvaddstr()

Encapsulate a curses function to reduce dependencies.

refactor: printw() to mutt_window_printf()

Encapsulate a curses function to reduce dependencies.

refactor: move() to mutt_window_move_abs()

Encapsulate a curses function to reduce dependencies.
This function works with absolute screen coordinates rather than coords
based on a MuttWindow.  This function should be eliminated soon.

refactor: clrtobot() to mutt_window_clrtobot()

Encapsulate a curses function to reduce dependencies.

refactor: addstr() to mutt_window_addstr()

Encapsulate a curses function to reduce dependencies.

refactor: addnstr() to mutt_window_addnstr()

Encapsulate a curses function to reduce dependencies.

refactor: addch() to mutt_window_addch()

Encapsulate a curses function to reduce dependencies.

refactor: beep() to mutt_beep()

Encapsulate a curses function to reduce dependencies.

refactor: curs_set() to mutt_curses_set_cursor()

Encapsulate a curses function to reduce dependencies.

refactor: ATTR_SET() to mutt_curses_set_attr()

Encapsulate a curses function to reduce dependencies.

refactor: NORMAL_COLOR() to mutt_curses_set_color()

Encapsulate a curses function to reduce dependencies.

refactor: SET_COLOR() to mutt_curses_set_color()

Encapsulate a curses function to reduce dependencies.

refactor: move KeyEvent out of mutt_curses.h

KeyEvent doesn't belong in mutt_curses.h.

refactor: tidy naming of color functions

Tidy colour functions for consistency.