Avoid regenerating hcache/hcversion.h after building objects in hcache

Specifying the hcache directory asqa dependency for hcache/hcversion.h
means that the latter will be regenerated whenever the hcache directory
is changed, such as when object files are written into it. The net
effect is that "make; make" regenerates hcache/hcversion.h twice.

This commit fixes this problem by explicitely creating the hcache
directory as part of the hcache/hcversion.h target.

Add the mutt_ch_check API (#1139)

* Add the mutt_ch_check API

This adds the mutt_ch_check API that can be used to check whether a
string can be converted between character sets. It is stricter than
mutt_ch_convert_string in that it doesn't apply any replacement chars to
unconvertible characters and doesn't return the converted string.

* Fix ICONV_CONST

* Fix arguments to iconv()

* Free the correct pointer

Don't let info messages reset error messages' timeouts (#1234)

Issue #1100

Fix typo in comment

merge: update translations

 * add missing format specifier
 * curs_main.c: Fix plural messages with "messages"
 * drop dupe bad IDN messages
 * handler.c: Partly fix plural message about attachment size
 * make room for translation
 * mx.c: Fix plural message "messages(s) not deleted"
 * mx.c: Fix plural message "Move %d read messages to %s?"
 * ncrypt/crypt_gpgme.c: Fix plural message "%s, %lu bit %s\n"
 * ncrypt/ncrypt.c: Fix plural message "Passphrases forgotten."
 * opcodes.h: Fix plural messages with "<word>(s)"
 * pager.c: Fix plural messages with "messages"
 * pager.c: Mark two strings for translation
 * pop.c: Fix plural message "Reading new messages (%d bytes)..."
 * recvattach.c: Fix plural message "Print tagged attachments?"
 * sync translations
 * translate status 'all' and 'end'
 * Update Czech translation
 * Update lt_LT translation
 * Update po files
 * Update pt_BR translation
 * Update zh_CN translation
 * Updated Catalan translation.
 * Updated Japanese translation.
 * Updated Russian translation.
 * Updated Slovak translation
 * Updated Ukrainian translation.

sync translations

Updated Catalan translation.

Updated Japanese translation.

Updated Russian translation.

Updated Ukrainian translation.

Update zh_CN translation

drop dupe bad IDN messages

Update zh_CN translation

Translate: update zh_cn

The translate would be better

Signed-off-by: Bo YU <yuzibode@126.com>

Translate zh_cn update

msgid "Print tagged attachment?"
msgid_plural "Print %d tagged attachments?"
msgstr[0] "打印 %d 件已标记的附件？"

And i want to make clear for that:if i place %d into msgstr[0],then
msgid "Print tagged attachment" how to translate?Under my understand,if
program invokes msgid, it will display "打印 件已标记的附件",just left a
space.

Signed-off-by: Bo YU <yuzibode@126.com>

Translate: update zh_CN translate

add missing format specifier: %d

Signed-off-by: Bo YU <yuzibode@126.com>

add missing format specifier

Translate: Update zh_CN translates

Zh_CN translate update.Now, i removed about 15 string "fuzzy" tagged.

Signed-off-by: Bo Yu <tsu.yubo@gmail.com>

translate status 'all' and 'end'

Update Czech translation

make room for translation

Update lt_LT translation

Two new strings, a bunch of fuzzy strings.  Back to 100%.

Update pt_BR translation

Update pt_BR translation

Update pt_BR translation

Update po files to match new code base

Most changes are splitting up plural constructions into more "saner"
versions, e.g. _("message(s)") becomes ngettext("message", "messages",
n). In all the cases we used the old translation for the new pluralised
translation (in all plural forms). This might not be correct but yields
the same result as before the change to the end user.

opcodes.h: Fix plural messages with "<word>(s)"

Due to the implementation details we do not know whether we work with
zero, 1, 12, ... messages/files/.... So in English we use normal plural.
We leave it up to the translator to choose a fitting translation. Some
language might have other means to express this situation.

curs_main.c: Fix plural messages with "messages"

Due to the implementation details we do not know whether we
delete/undelete/mark/... zero, 1, 12, ... messages. So in English we use
"messages". We leave it up to the translator to choose a fitting
translation. Some language might have other means to express this
situation.

pager.c: Fix plural messages with "messages"

Due to the implementation details we do not know whether we
delete/undelete zero, 1, 12, ... messages. So in English we use
"messages". We leave it up to the translator to choose a fitting
translation. Some language might have other means to express this
situation.

handler.c: Partly fix plural message about attachment size

Use ngettext() to choose a (more) correct plural form for 'byte'
depending on the attachment size. There is a caveat though: We do not
print the size as number but as a prettified number, e.g.  2048 will be
printed as 2K. Some languages might inflect 'byte' different depending
on whether it is '1K byte' or '1024'. Sadly there is nothing we can do
about that in the moment (besides not using the prettified version but
that is not an option).

ncrypt/crypt_gpgme.c: Fix plural message "%s, %lu bit %s\n"

Give translators to opportunity to inflect "bit" according to the bit
size of the key.

pop.c: Fix plural message "Reading new messages (%d bytes)..."

Although the English use case with "1 byte" is unlikely to happen, other
languages might have other rules.

mx.c: Fix plural message "Move %d read messages to %s?"

recvattach.c: Fix plural message "Print tagged attachments?"

If messages are tagged, then count them and use the number to pluralise
the phrase "tagged messages" correctly.

ncrypt/ncrypt.c: Fix plural message "Passphrases forgotten."

Due to the implementation details (e.g. some passwords are managed by
gpg-agent) we cannot know whether we forgot zero, 1, 12, ...  passwords.
So in English we use a generic plural. Different languages might have
other means, so we leave the decision what to choose to the translator.

mx.c: Fix plural message "messages(s) not deleted"

Update po files

pager.c: Mark two strings for translation

Mark to strings which are displayed to the end user for translation.

Fixes #1217

Updated Slovak translation

fix inappropriate use of FREE() in ssl init error path

OpenSSL structures need to be freed with dedicated functions.

Remove mutt_buffer_new() NULL retval checks.

It will fail, rather than return NULL.

Further clean up imap_new_data() since it also had NULL checks for
safe_calloc() that shouldn't happen.

merge: trivial improvements

 * fix comment
 * fix formatting
 * tidy main
 * mutt_file_sanitize_filename bool param
 * mutt_param_cmp_strict bool retval
 * content state bools
 * use buffer printf
 * include queue.h in mutt/mutt.h
 * bool param
 * tidy handler.c
 * split out enriched text code

split out enriched text code

tidy handler.c

bool param

include queue.h in mutt/mutt.h

use buffer printf

content state bools

mutt_param_cmp_strict bool retval

mutt_file_sanitize_filename bool param

tidy main

fix formatting

fix comment

Move LSUB call from connection establishment to mailbox SELECTion (#1232)

* Move LSUB call from connection establishment to mailbox SELECTion

This changes the place where an LSUB is issued to an IMAP server in
order to populate the buffy list, when imap_check_subscribed is set.

Previously, LSUB was issued when a connection to an IMAP server was
*first* established.  This caused the commands sequence

  ":unmailboxes *<enter><change-folder>imaps://foo.com<enter>"

to populate the buffy list when *first* opening a connection to foo.com,
but not on subsequent back-and-forth between IMAP servers.

The new code moves the LSUB call - and thus the population of the buffy
list - to just before a mailbox is selected.
This fixes the above-mentioned issue, at the cost of additional LSUB
calls when switching between mailboxes on the same server. Given that
the list of subscribed mailboxes is expected to be in the tens, I think
this is a reasonable tradeoff.

Issue #1225

Honour SASL-IR IMAP capability in SASL PLAIN

* Honour SASL-IR IMAP capability
* Tidy auth plain logic
* Keep the buffer static

Fixes #1211

Apply coding style with clang-format

Do not truncate shell commands on ; or #

Fixes #1216.

check that PreferredLanguages is set

merge: trivial improvements

 * tidy param names
 * fix cppcheck issues
 * comment blocks for aliases
 * comment blocks for command parsing
 * regroup functions

regroup functions

comment blocks for command parsing

comment blocks for aliases

fix cppcheck issues

tidy param names

merge: update translations

 * fix some accidental fuzzyness
 * Update lt_LT translation
 * Update lt_LT translations
 * Update zh_CN translation
 * sync translations

sync translations

Update lt_LT translation

We're at 100%!  The fifth language to reach this goal (after recent
changes).

Update lt_LT translation

Most of the remaining untranslated messagers

Update lt_LT translation

Small fixes.

Update lt_LT translations

Low-hanging fruit among missing translations in the rest of the file.

Update lt_LT translation

Translating missing entries in the first 25% of the file.

Update lt_LT translations

All fuzzy entries have been translated.

Update lt_LT translation

Reviewing fuzzy entries in the first 90% of the file.

Update lt_LT translation

Going over fuzzy entries in the first 80% of the file

Update lt_LT translation

The Great PGP/S/MIME Option Translation.

Wow this was hard.

Update lt_LT translations

Mostly fuzzies again.  Hit a roadblock with the PGP menus.

Update zh_CN translation

Update lt_LT translation

Mostly reviewing fuzzy entries in the first 60% of the file, but I also
added some missing translations and reviewed some older translations for
consistency.

Update lt_LT translation

Reviewed fuzzy entries (and some adjacent ones) in the first 50% of the
file.

Update zh_CN translation

fix some accidental fuzzyness

Update lt_LT translation

Mostly fuzzy strings in the first 40% of the file

Update lt_LT translation

Mostly fuzzy entries in the first 30% of the file.

Update zh_CN translation

prepare to use travis' fold/timer

Convert Aliases to use TAILQ

build: stop on install failure

reply_with_xorigto: use mutt_addr_copy instead of mutt_addr_copy_list

add missing flags

Make sure all the 'set' routes through parse_set() use
MUTT_TOKEN_BACKTICK_VARS.

Revert "Partial solution to #1165 (#1176)"

Too complicated

Partial solution to #1165 (#1176)

* mutt/file.c: allow mutt_file_mkstemp to return a filename

modify mutt_file_mkstemp so that a caller may pass in a character
buffer that will be filled with the temporary filename. When a
non-null buffer is provided, mutt_file_mkstemp will *not* unlink the
temporary file and will leave cleanup to the caller.

Closes: #1165

Introduce MUTT_TOKEN_BACKTICK_VARS to expand variables within backticks

This is a new flag to mutt_extract_token which triggers variable
expansion within backticks.

With this flag:

```
:set my_foo="`echo foo $pager bar`"
:set ?my_foo
foo builtin bar
```

Without this flag:

```
:set my_foo="`echo foo $pager bar`"
:set ?my_foo
foo  bar
```

This flag is currently only passed when parsing a `set` command.

Expand variables inside backticks

This allows for example to simplify the solution given here [1] as:

```
:set pager_index_lines=`dc -e "50 $pager_index_lines -p"`
```

[1]
http://mailman.neomutt.org/pipermail/neomutt-users-neomutt.org/2018-April/000325.html

fix history menu

Fix toggle-read

Issue #1198

ci_send_message: copy the x_original_to address list instead of using it

If an end user cancels the edit_envelope action, then cleanup is called.
When using directly the x_original_to address list directly, this
cleanup stage did actually clean the x_original_to pointer. Should the
user reply again to the same message, then the pointer would point to 0
and then any "unsafe" operation on it would trigger a SIGSEGV. Copying
it solves this issue safely.

fix some docs

sync with doc tidying in 77155bb

github: remove double checking question

`Are there points in the code the reviewer needs to double check?` is mostly providing no benefits. When a PR author wants the reviewer to double check sth in particular, they can somewhere.

Additionally, passing tests should (obviously) also be part of the acceptance criteria.

merge: NeoMutt 2018-05-12