quote_regexp -> quote_regex

browser: <change-dir> mutt_realpath() <goto-parent>

For "regular" filesystems (not IMAP or Notmuch folder hierarchies),
<change-dir> and the browser GUI now resolve paths via a new mutt library
function `mutt_realpath`. This means no more path buildup such as
"a/b/../../c". Additionally, Symlinks are resolved automatically. This is
a circumstance of realpath(). Note that a future PR is planned to add a
non-symlinking option.

The new <goto-parent> function has a default shortcut key of 'p' for parent.
Since this function uses the pre-existing `mutt_get_parent_path()`,
<goto-parent> may also work on IMAP and Notmuch folder hierarchies
(tested and working on IMAP)

Note: The paths passed to `mutt_get_parent_path()` formerly would not
parse correctly a path which included a final trailing slash (eg 'abc/')
So, I added it a check to that function to remove it before hitting
the parser.

Note2: The browser for IMAP was janky before this commit, but the browser
is not the primary mailbox navigation tool, afterall. At least 'p'
<goto-parent> adds to its functionality. I expect a similar evaluation for
Notmuch folder browsing, although I have never used Notmuch. Any
contributions to make a better IMAP/Notmuch folder browsing experience
are welcome. I have reported some of the fallbacks in Github PR #1037
comments (look for IMAP headlines).

Fix off-by-1 bug in strnfcpy, add test cases

Co-authored-by: Michael Bazzinotti <mbazzinotti@gmail.com>

fix rfc2047 field decoding

fix parsing of urls containing '?'

The URL schemes come in two patterns
- smtp://user:pass@host
- notmuch:///path?query

The 'host' types don't take a query string.
'notmuch' and 'mailto' don't use a 'user:pass' string.

This fix will still be confused by a notmuch URL that has a '?' in the
path component, but no query string.  (but I don't think the code ever
coped with that).  e.g.

- notmuch:///dir?name/

Fixes #1043

Let mutt_str_strnfcpy return the string length

Let mutt_str_strfcpy return the string length, add tests

merge: history refactoring

 * reduce scope of variables
 * rename functions
 * code tidying
 * doxygen: add comments

doxygen: add comments

code tidying

- Check ints against 0
- Rename parameters
- Bracket logic
- Reduce scope of variables

rename functions

reduce scope of variables

merge: trivial improvements

 * coverity
 * name unicode chars
 * Fix spelling
 * doxygen tidying
 * Add a blurb about $new_mail_command in the mailboxes section.

Add a blurb about $new_mail_command in the mailboxes section.

doxygen tidying

Fix spelling

name unicode chars

coverity

merge: md5 improvements

 * Remove unused functions, unexpose implementation functions
 * Implement and use mutt_md5_toascii
 * Initial tests
 * Use mutt_md5_toascii some more
 * Add and use an API to process a NULL-terminated string
 * More API polishing
 * Rename test cases to better reflect what they're doing

MD5 - Rename test cases to better reflect what they're doing

MD5 - More API polishing

MD5 - Add and use an API to process a NULL-terminated string

MD5 - Use mutt_md5_toascii some more

MD5 - Initial tests

MD5 - Implement and use mutt_md5_toascii

MD5 - Remove unused functions, unexpose implementation functions

Kill some magic numbers, use mutt_array_size

Make Weekdays, Months, and TimeZones statics, expose mutt_date_make_tls

Kill mutt_rfc2047_encode_32 macro

Fix reply_regexp which wasn't formatted correctly.

And add Swedish "Sv:" prefix to reply_regexp.

Unit test framework (#1031)

Minimal unit-test framework to test C library APIs

merge: RFC2047 improvements

 * Handle RFC2047 words split in the middle of a multibyte character
 * move rfc2047 functions to library
 * Use our base64 API instead of duplicating the decoding algorithm
 * Move mutt_rfc2047_choose_charset to mutt_ch_choose
 * light tidy

light tidy

* fix typos
* rename parameters
* add ()s to logic

Move mutt_rfc2047_choose_charset to mutt_ch_choose

Issue #1015

Use our base64 API instead of duplicating the decoding algorithm

Also, fix a memory leak while at it.

Issue #1015

move rfc2047 functions to library

Handle RFC2047 words split in the middle of a multibyte character

fix for pgpewrap in default gpg.rc (#1023)

Fixes an bug introduced by changes intorduced in commit
465fad6d21ac3b9d2ae5d12bae7705f5f04323e7 to contrib/gpg.rc that breaks signed
and encrypted mail. The issues is just a typo related to way that pgpewrap
works. This should fix issue #998.

merge: light refactoring

 * move unsigned atoi functions
 * comparison ordering
 * braces
 * move functions to library

move functions to library

braces

comparison ordering

move unsigned atoi functions

Fix a couple memory leaks in pattern.c

BUFFER.data was not being freed in a couple cases.

The pattern and simple search were not being freed if imap_search()
failed.

merge: fix coverity defects

 * drop unreachable condition
 * set new regex
 * negative check
 * check size_t value
 * unused result
 * ptr checks
 * negative index
 * tidy logic
 * fix logic
 * unchecked returns
 * forward null checks

forward null checks

unchecked returns

fix logic

tidy logic

negative index

ptr checks

unused result

add new var for calc

check size_t value

negative check

set new regex

drop unreachable condition

Simplify ALL_FILES definition, make it easier to add/remove files

merge: trivial improvements

 * rename parameters bufsz -> buflen
 * tidy mapping functions
 * move mutt_pretty_size to libmutt
 * iwyu: remove unnecessary headers
 * rename charset functions
 * mutt_str_getenv()
 * use mutt_str_getenv()

use mutt_str_getenv()

mutt_str_getenv()

rename charset functions

To avoid a clash with the configset functions to come.

iwyu: remove unnecessary headers

move mutt_pretty_size to libmutt

tidy mapping functions

rename parameters bufsz -> buflen

lua: enable myvars

Fixes #1016
Fixes #1017

merge: upstream fixes (mutt/default)

 * Fix improper signed int conversion of IMAP uid and msn values.
 * Change imap literal counts to parse and store unsigned ints.
 * Fix imap status count range check.
 * cmd_handle_fatal: make error message a bit more descriptive
 * Updated French translation.
 * Create pgp and s/mime default and sign_as key vars. (see #3983)
 * Add missing setup calls when resuming encrypted drafts.
 * mutt_pretty_size: show real number for small files
 * examine_directory: set directory/symlink size to zero
 * Update pl.po
 * Fixed GPGME translations that weren’t shown but affected the keyboard
 * docs: update encrypt-to-self
 * Update smime.rc: Typo fix, consistent headings
 * Add pgp_default_key and smime_sign_as info to contrib rc files.
 * Split Copyright and Thanks in help output.
 * strip out copyright translations

strip out copyright translations

Split Copyright and Thanks in help output.

The Copyright string is changing often, and its content is obvious.
It does not need translation. The remaining string can be translated.
This change avoids a stale translation once one of the years change.

Signed-off-by: Olaf Hering <olaf@aepfle.de>

Add pgp_default_key and smime_sign_as info to contrib rc files.

Explain $pgp_default_key vs $pgp_sign_as in gpg.rc.

Explain $smime_default_key vs $smime_sign_as in smime.rc.

Update smime.rc: Typo fix, consistent headings

docs: update encrypt-to-self

Fixed GPGME translations that weren’t shown but affected the keyboard

Update pl.po

examine_directory: set directory/symlink size to zero

The size of a directory or symlink in the folder browser is not meaningful.
For directories it means just how many blocks were allocated to hold all
entries. It does not mean that the entries are still present in the directory.
For symlinks its the size of the target.
Set both to zero to simplify the folder browser output.

Signed-off-by: Olaf Hering <olaf@aepfle.de>

mutt_pretty_size: show real number for small files

If a file is smaller than a certain size it is unfriendly to print 0K or
0,1K as number of mails or as file size. Instead use the real number.

Signed-off-by: Olaf Hering <olaf@aepfle.de>

Add missing setup calls when resuming encrypted drafts.

Calls to get the passphrase were missing for app/pgp and app/smime.
App/smime was also missing a call to crypt_smime_getkeys().

If a failure occurs, report it back, rather than just continuing.
Otherwise, postponed messages could be completely lost.

Create pgp and s/mime default and sign_as key vars. (see #3983)

The $postpone_encrypt and $(pgp/smime)_self_encrypt configuration
variables have created a somewhat messier situation for users.  Many
of them now have to specify their keys across multiple configuration
variables.

(Trac) Ticket #3983 had a reasonable request: "if my encrypt and
signing keys are the same, why can't I just specify my key once in my
.muttrc?"

The problem currently is that $smime_default_key and $pgp_sign_as are
both used to specify signing keys, and are set by the "sign (a)s"
security menu choice.  So we can't store encryption keys there because
some users have separate sign-only capability keys.

Create $pgp_default_key to store the default encryption key.  Change
signing to use $pgp_default_key, unless overridden by $pgp_sign_as.
The pgp "sign (a)s" will continue setting $pgp_sign_as.

Create $smime_sign_as.  Change signing to use $smime_default_key
unless overridden by $smime_sign_as.  Change s/mime "sign (a)s" menu
to set $smime_sign_as instead.

Change $postpone_encrypt and $(pgp/smime)_self_encrypt to use
$(pgp/smime)_default_key by default.

Mark $(pgp/smime)_self_encrypt_as deprecated.  They are now aliases
for the $(pgp/smime)_default_key config vars.

Change $(pgp/smime)_self_encrypt default to set.

The intent is that most users now need only set
$(pgp/smime)_default_key.  If they have a sign-only key, or have
separate signing and encryption keys, they can put that in
$(pgp/smime)_sign_as.  This also enables to default self_encrypt on
and solve a very common request.

Thanks to Michele Marcionelli and Vincent Lefèvre for gently pushing
me towards a solution.

Updated French translation.

cmd_handle_fatal: make error message a bit more descriptive

When there are multiple IMAP connections available, "Mailbox closed"
doesn't give a hint as to which one.  Use account info to identify which
mailbox was closed.

Fix imap status count range check.

The strtoul() call for parsing the STATUS count wasn't checking the
range properly, because it was assigning to an unsigned int.

Change to assign to a unsigned long, and also add the conversion check
from mutt_atoui().

Thanks to Charles (@chdiza) for quickly noticing the problem!

Change imap literal counts to parse and store unsigned ints.

IMAP literals are of type number.  Change imap_get_literal_count() to
use mutt_atoui() instead of atoi().  Change the return type variables
used to store the count to type unsigned int.

It's doubtful this was a real issue, but as long as we're cleaning up
incorrect atoi() usage, we should fix this too.

Fix improper signed int conversion of IMAP uid and msn values.

Several places in the imap code, when parsing "number" and "nz-number"
values from the IMAP data, use atoi() and strtol().  This is
incorrect, and can result in failures when a uid value happens to be
larger than 2^31.

Create a helper function, mutt_atoui() and use that instead.  One
place was using strtol() and relying on the endptr parameter, and so
was changed to use strtoul() instead.

Thanks to Paul Saunders for the bug report and original patch, which
this commit is based on.

Updated French translation.

Disable message security if the backend is not available.

Gitlab issue #3 exposed an awkward corner case: if mutt is configured
without PGP or S/MIME, and with GPGME, but $crypt_use_gpgme is unset.
In this case, no backend will be available, but WithCrypto will be set
with both APPLICATION_PGP and APPLICATION_SMIME bits.

That will allow various config vars to enable encryption or signing,
even though there will be no backend available to perform them.  The
message security flag might then be set, but when the user hits send,
will end up back at the compose menu due to the error.

The pgp or smime menu might not even be available to clear the
security setting!

Add a check in send.c before the compose menu is invoked, and give a
warning message for the menu ops inside the compose menu.

I believe this should prevent the issue.  However this is a corner
case combined with user misconfiguration, so I don't believe is worth
a large effort to completely eradicate.

Mention self_encrypt options in the documentation. (closes #3983)

Add a paragraph to the "Sending Cryptographically Signed/Encrypted
Messages" section of the manual.

Update copyright notices.

Review fuzzy lt translations

update translations

Fix line split logic in help (#1019)

* Fix line split logic in help

Fixes #1018

merge: trivial improvements

 * add missing {}s
 * doxygen
 * rename functions: adr -> addr
 * rename params/args/vars: adr -> addr
 * docs: drop reference to waffle backlog

docs: drop reference to waffle backlog

rename params/args/vars: adr -> addr

rename functions: adr -> addr

doxygen

add missing {}s

fix regex parsing

fix display of special chars

merge: move charset and idna functions to libmutt

 * clang-format
 * reorg: idna -> address, envelope
 * private hooks for charset
 * move charset functions to muttlib
 * move idna functions to muttlib
 * move address functions
 * iwyu / comments

iwyu / comments