tidy L10N comments

fix comments

docs: fix typos

Rename variables: string -> str

Consistent variable naming makes the code easier to read.

fix pre-increment in for loops

In for loops, there's no difference in behaviour between:

  `for (int i = 0; i < 10; ++i)`
  `for (int i = 0; i < 10; i++)`

Use the C form, `i++`, for consistency.

move switch/case break

Move the `break` inside the {}s of a case statement (for consistency).

move group to email

group.[ch] have dependencies on Address, so they should live in libemail.

Disable all history operations if history is not enabled

Fixes #1592

Fix composing a new message (again)

The old code had crept in again in 028c34.

add parentheses to conditionals

(flatcap discovers uncrustify)

ifdef: add support for environment variables

This allows the user to make config decisions based on the existence of
environment variables.  e.g.

  ifdef PS1 source interactive.rc

drop devel script

Committed by accident

[ci skip]

merge: lots of tidying

 * fp prefix
 * return r
 * return ret
 * return result
 * rename SHA1 contexts
 * rename MD5 contexts
 * create enum SortType
 * add typedef for PatternFlags
 * use enum QuadOption for query_quadoption(), mutt_yesorno()
 * separate enum for mutt_view_attachment()
 * separate enum for mailcap lookups
 * separate enum for mutt_save_attachment()
 * separate enums for pattern types
 * create TreeChar enum for tree characters

create TreeChar enum for tree characters

separate enums for pattern types

separate enum for mutt_save_attachment()

separate enum for mailcap lookups

separate enum for mutt_view_attachment()

use enum QuadOption for query_quadoption(), mutt_yesorno()

add typedef for PatternFlags

create enum SortType

rename MD5 contexts

Distinguish MD5 contexts from Context contexts.

rename SHA1 contexts

Distinguish SHA1 contexts from Context contexts.

return result

return ret

return r

fp prefix

Let ctx_update_tables update the mx size for IMAP emails

Issue #1577

Decreasing mbox size is done at ctx_update_tables, !active is not used

Issue #1577

Fix crash when an email disappears from IMAP

Issue #1577

Update to latest autosetup

Fix composing a new message

Fixes #1582

merge: fix coverity defects

 * defect 180847: Pointer to local outside scope
 * defect 189753: Inferred misuse of enum
 * defect 190188: Unchecked return value from library
 * defect 190189: Unchecked return value
 * defect 190190: Dereference before null check
 * defect 190853: Dereference after null check
 * defect 190854: Logical vs. bitwise operator
 * defect 190855: Dereference after null check
 * defect 190856: Overlapping buffer in memory
 * defect 190857: Dereference after null check

defect 190857: Dereference after null check

defect 190856: Overlapping buffer in memory

defect 190855: Dereference after null check

defect 190854: Logical vs. bitwise operator

defect 190853: Dereference after null check

defect 190190: Dereference before null check

defect 190189: Unchecked return value

defect 190188: Unchecked return value from library

defect 189753: Inferred misuse of enum

defect 180847: Pointer to local outside scope

merge: trivial tidying

 * clang-format
 * rename variables: descrip -> desc
 * rename variables: buffer -> buf
 * rename variables: command -> cmd
 * rename variables: progressbar -> progress
 * rename variables: thepid -> pid
 * rename variables: synch -> sync
 * name macro variables
 * lowercase local variables
 * expand meaningless R_BOTH macros
 * tidy mutt_edit_content_type()
 * tidy generate_body()
 * tidy mutt_alias_create()
 * small code tidying
 * boolify variables/params
 * reduce variable scope

reduce variable scope

boolify variables/params

small code tidying

- fix typo
- drop once-used define
- rename old_msg_count
- drop unnec parentheses

tidy mutt_alias_create()

- return early, reducing indent
- rename variables for clarity

tidy generate_body()

- fix variable type

tidy mutt_edit_content_type()

- boolify
- parenthesise

expand meaningless R_BOTH macros

Both what?

- R_BOTH -> R_INDEX | R_PAGER
- R_RESORT_BOTH -> R_RESORT | R_RESORT_SUB

lowercase local variables

Local variables should be lowercase and snake_case.

Consistent variable naming makes the code easier to read.

name macro variables

Change macros from:

`#define function(A,B,C) other(A,B,C)`

to

`#define function(ctx,buf,buflen) other(ctx,buf,buflen)`

rename variables: synch -> sync

Consistent variable naming makes the code easier to read.

rename variables: thepid -> pid

Consistent variable naming makes the code easier to read.

rename variables: progressbar -> progress

Consistent variable naming makes the code easier to read.

rename variables: command -> cmd

Consistent variable naming makes the code easier to read.

rename variables: buffer -> buf

Consistent variable naming makes the code easier to read.

rename variables: descrip -> desc

Consistent variable naming makes the code easier to read.

clang-format

merge: Define types for more flags/bitfields

 * imap: rename constants to match capability strings
 * smtp: convert capability enum to flags
 * add typedef for ImapCapFlags
 * add typedef for SmtpCapFlags
 * add typedef for AnsiFlags
 * add typedef for CheckFlags
 * add typedef for MhSeqFlags
 * add typedef for AclFlags
 * add typedef for HashFlags
 * add typedef for MixCapFlags
 * add typedef for CliFlags
 * add typedef for SelectFileFlags

add typedef for SelectFileFlags

add typedef for CliFlags

add typedef for MixCapFlags

add typedef for HashFlags

add typedef for AclFlags

add typedef for MhSeqFlags

add typedef for CheckFlags

add typedef for AnsiFlags

add typedef for SmtpCapFlags

add typedef for ImapCapFlags

smtp: convert capability enum to flags

imap: rename constants to match capability strings

merge: Define types for all of the flags/bitfields.

Being 'C' this doesn't add any type safety, but it does make the code far more legible.<Paste>

 * add typedef for ConfigDumpFlags
 * add typedef for ConfigRedrawFlags
 * add typedef for MuttFormatFlags
 * add typedef for HookFlags
 * add typedef for ImapOpenFlags
 * add typedef for ImapCmdFlags
 * add typedef for MuttAccountFlags
 * add typedef for CompletionFlags
 * add typedef for TokenFlags
 * add typedef for MuttRedrawFlags
 * add typedef for MuttThreadFlags
 * add typedef for OpenMailboxFlags
 * add typedef for MsgOpenFlags
 * add typedef for PagerFlags
 * add typedef for SendFlags
 * add typedef for StateFlags
 * add typedef for KeyFlags
 * add typedef for SecurityFlags
 * add typedef for CopyHeaderFlags
 * add typedef for CopyMessageFlags

add typedef for CopyMessageFlags

add typedef for CopyHeaderFlags

add typedef for SecurityFlags

add typedef for KeyFlags

add typedef for StateFlags

add typedef for SendFlags

add typedef for PagerFlags

add typedef for MsgOpenFlags

add typedef for OpenMailboxFlags

add typedef for MuttThreadFlags

add typedef for MuttRedrawFlags

add typedef for TokenFlags

add typedef for CompletionFlags

add typedef for MuttAccountFlags

add typedef for ImapCmdFlags

add typedef for ImapOpenFlags

add typedef for HookFlags

add typedef for MuttFormatFlags

add typedef for ConfigRedrawFlags

add typedef for ConfigDumpFlags

merge: Replace STRING constants

Constants like LONG_STRING aren't meaningful to anyone, so replace them
with their numeric equivalents.

 * replace SHORT_STRING with 128
 * replace STRING with 256
 * replace LONG_STRING with 1024
 * replace HUGE_STRING with 8192 or STR_COMMAND

replace HUGE_STRING with 8192 or STR_COMMAND

replace LONG_STRING with 1024

replace STRING with 256

replace SHORT_STRING with 128

config: add 'C_' prefix

Each config variable has a global variable backing it.
The name of the global is the MixedCase transform of the config name.
While this is useful for remembering their names, it makes some
variables hard to search for, giving many false positives.

- Copy
- From
- Help
- Move
- Sort

Prefixing the globals with `C_` prevents this problem.