quote multipart/multilingual

'multipart/multilingual' is a reserved keyword -- not to be translated

quote multipart/alternative

'multipart/alternative' is a reserved keyword -- not to be translated

quote multilingual

'multilingual' is a reserved keyword -- not to be translated

quote alternatives

'alternatives' is a reserved keyword -- not to be translated

quote Content-Language

'Content-Language' is a reserved keyword -- not to be translated

merge similar translation strings 2

- brackets
- parentheses

merge similar translation strings 2

- Compressing %s...
- Compressing %s

merge similar translation strings 1

- Can't create %s %s
- Can't create %s: %s

use a question mark in the bounce question

use a question mark in the crypto questions

remove colons from questions

Remove the :'s to align the style with other questions.

remove []s from a question

Remove the []s to align the style with other questions.

add space after question prompt

Change the code to print a space after a question.
This simplifies the translation strings.

quote 'nametemplate'

'nametemplate' is a reserved keyword -- not to be translated

add missing constants

- mutt_debug()
- mutt_yesorno()

Get rid of one-shot macro

Disable mailbox notify on close

This fixes a crash when two mailboxes have been opened within a session,
and an 'unmailboxes *' command is issued. In this case, both the current
mailbox and the previously opened mailbox have a non-NULL `notify`
member. The notify method assumes the mailbox is currently active within
the context. The context gets thus doubly freed.

merge: trivial fixes

 * clang-format
 * add parentheses around logic-with-spaces
 * boolify: mutt_simple_format()
 * rename locals to avoid clash
 * add: enum FormatJustify
 * doxy: tidying

doxy: tidying

add: enum FormatJustify

rename locals to avoid clash

We're using the `C_` prefix for config variables.

boolify: mutt_simple_format()

add parentheses around logic-with-spaces

clang-format

unify variable naming

Make sure email size is consistently computed in status bar and index

Display B for bytes when pretty-printing a size

Fix dependencies of libemail

Fixes #1618

Fix bogus mailbox size accounting

tags: Use STAILQ instead of strsep

This change introduces a new string function mutt_str_split(src, sep) to
return a STAILQ of the split string.

This new funtion is used first in tags code.

fix further crash in ci_send_message()

Signed-off-by: Christian Storm <storm@mailbox.org>

merge: update translations

 * Update Spanish translation
 * po/zh_CN: complete translation
 * swedish: fix option typo
 * Update Finnish translation
 * spanish: pgp-menu multioptions fixed
 * update German translation
 * trans: fix smime questions
 * sync translations

sync translations

trans: fix smime questions

update German translation

spanish: pgp-menu multioptions fixed

Update Finnish translation

swedish: fix option typo

po/zh_CN: complete translation

Update Spanish translation

tidy maildir_move_to_mailbox

tidy memory functions

- reduce scope of variables
- test scalars against 0

memory: return NULL, not 0

mutt_mem_free() should return NULL, instead of 0, to match code style[0]

[0] https://neomutt.org/dev/coding-style#distinguish-between-0-and-null

merge: Style Menu Options

 * add mutt_window_mvaddnstr()
 * Style Menu Options
 * smime: unify multi-option questions

smime: unify multi-option questions

Change the questions to match the rest of NeoMutt.

From:
  "1: apple, 2: banana"
to
  "(1) apple, (2) banana"

Style Menu Options

Closes: #1181
Co-authored-by: Mehdi Abaakouk <sileht@sileht.net>
Co-authored-by: Pietro Cerutti <gahr@gahr.ch>

add mutt_window_mvaddnstr()

clang-format

sidebar(update_entries_visibility): Use bool instead of short for booleans

sidebar: New option to display only non-empty mailboxes

Adds a new option $sidebar_only_non_empty_mailboxes which forces neomutt
to display only the mailboxes that have atleast one mail in them. This
option will also work in tandem with the existing $side_new_mail_only

sidebar: Refactor visibility detection

sidebar: Use an explicit fast path for checking visibility

increase space for key name

We need enough space for '<backspace>'.

implement unbind and unmacro commands

lua: fix command-line processing

The 'lua' command uses the entire line of text.
Clear the Buffer so that NeoMutt doesn't try to interpret it.

Fixes #1610

doxy: fix function comments

doxy: css fixes

fix out-of-tree test/pattern build

test: add pattern compilation tests

Closes #1522

pattern: change Pattern to use SLIST_* macros

Resolves #1224

slang/color: parse direct-color values in 24bit hex format

And render them using SLtt_set_color.

The way to use direct-color values in the config file is:

color status "#ff0000" "#630000"

Note that the '#' needs to be quoted in order to avoid discarding it
as a comment.

At the moment there's no way to use direct color values with ncurses,
so this feature is only supported when using slang.

Tested on iTerm 3.1.5 with s-lang 2.3.1a.

color: switch color types to uint32_t

So they can be used to store truecolor values which are 24bits.

notmuch: close thread queries' hcache on SigInt (#1606)

Ensure read_threads_query() behaves the same as read_mesgs_query() when
receiving SigInt.

docs: update credits

maildir: remove unnecessary function

()s on ternary

comment summary bindings

compress: rename command strings

Distinguish them from other structs' members.

logging: improve range logic

Make NotMuch hcache-aware

Closes: #1600

Fix table in man page

Previously:

    /usr/bin/tbl:<standard input>:824: unrecognised format `\'
    /usr/bin/tbl:<standard input>:824: giving up on this table

It seems the period after table headers must be on its own line.

merge: trivial fixes

 * trivial ()s and NUL
 * pop: add typedef for pop_fetch_data()
 * doxy: add links
 * more NULs
 * fix typos
 * add checks to imap_account_match()
 * add check to imap_adata_find()
 * maildir: rename context functions
 * add flags to maildir_mbox_check()

add flags to maildir_mbox_check()

maildir: rename context functions

These functions now work with Mailboxes:
- maildir_add_to_context()
- maildir_move_to_context()

add check to imap_adata_find()

add checks to imap_account_match()

fix typos

more NULs

doxy: add links

pop: add typedef for pop_fetch_data()

trivial ()s and NUL

Do not prepend "attach_save_dir" to an absolute path

This happens when saving multiple attachment. Assume `attach_save_dir = ./`.
When saving multiple attachments to `/tmp/att`, the proposed path
becomes `.//tmp/att`.

hcache: fix hash generator

`struct Header` was renamed to `struct Email`

Fixes: #1595

added: localtime_r/gmtime_r wrappers

Fixes: #1571

Fix incorrect IMAP message purging bug.

Thanks to Ivan Middleton @imiddle for the awesome bug report and
suggested fix.

The bug is most easily generated using Gmail with the $trash variable
set.

Deleted messages are first copied to the $trash folder.  If this is
set to "[Gmail]/Trash", then Gmail inteprets the copy as a "delete"
and sends EXPUNGE messages back for the messages.

cmd_parse_expunge() and cmd_parse_vanished() set the hdr->index to
INT_MAX, which subsequently an imap_expunge_mailbox() will use to
remove the messages from the local mailbox.

If we close the mailbox instead of sync it, Mutt will end up executing
the 'Deleted' flag setting before processing the expunge (because
"imap_check_mailbox() -> imap_cmd_finish()' doesn't set check_status
when we are closing).  The expunged messages will then be included in
the set of 'Deleted' flags.

Unfortunately, because the messages are sorted by *index* before
msgset generation, an incorrect range of UIDs will be sent, which
could easily include messages that should not be deleted.

This fix is a minimal fix for a stable bug fix  excluding
messages with the index set to INT_MAX from all msg sets.  Other
things that should be investigated in master are:

- sorting by UID instead of index before msgset generation
- unsetting the 'active' flag in cmd_parse_expunge() and
  cmd_parse_vanished() instead of waiting until imap_expunge_mailbox()
  to do so.

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

Fix opening IMAP messages when the body cache is disabled

Fixes #1574

mx: fix mbox double free

Fixes: #1597

Fix pointer assignment to '\0' introduced in 495153

merge: merge translations

 * Corrected several minor errors in the spanish translation
 * Update Czech translation
 * More miscellaneous improvements to spanish localization
 * Added a few miscellaneous translations
 * Update lt_LT translation
 * Polish translation update
 * update po files list
 * sync translations

sync translations

update po files list

Polish translation update

Update lt_LT translation

Fixed three new fuzzy entries and a lot of typos in old translations
because I decided to do a spell check on a whim.

Added a few miscellaneous translations

More miscellaneous improvements to spanish localization

Update Czech translation

Corrected several minor errors in the spanish translation

iwyu: optimise #includes

use SecurityFlags more

Some functions missed out on the earlier upgrade.

merge: Use NUL or NULL rather than 0

 * Use NUL or NULL rather than 0 when setting
 * Test string contents against NUL (==)
 * Test string contents against NUL (!=)