We'll hold a reader lock under the session is established, which
should prevent blocking other threads unless something is beating
on get_ssl_cert() (which would be a different bug).

This still needs to be figured out, but at least this should fix
the immediate issue.

Asked for a way to expand list of recipients for a specified user

Remove some redundant redundancies, redundantly

it's only called from load_cfg() which has already set it to -1.

We also don't need to check prepped twice to make sure it's extra true.

The design is to #include eventwrap.h for cross-platform source

Ctrl-A in the Failed Logins dialog now selects-all rather than copy-all (just
use Ctrl-A, then Ctrl-C to copy all).

warning: result of comparison of constant 100000 with expression of type 'uint16_t'
(aka 'unsigned short') is always true

./scfg.h:80:9: warning: 'SUB_HDRMOD' macro redefined [-Wmacro-redefined]

This is always going to call the terminal server's lprintf function (when used
with libsbbs.so/sbbs.dll) which is probably not what was intended.

Resolve reported clang warning:

inkey.cpp:592:20: warning: implicit conversion from 'long' to 'int' changes value from 2147483648 to -2147483648 [-Wconstant-conversion]
         if(!term_supports(MOUSE))
             ~~~~~~~~~~~~~ ^~~~~
 ./sbbsdefs.h:574:19: note: expanded from macro 'MOUSE'
 #define MOUSE           (1L<<31)                /* Mouse supported terminal                             */

This is hacky cruft here. We shouldn't have the declaration in multiple
places. It probably shouldn't be in sbbs.h.

load_cfg() calls free_cfg() before do_cryptInit() is called, so the ssl_rwlock
was uninitialized here.

Holding the lock around session establishment should not be needed,
but we need to protect tls_certificate read and usage.  Since we
don't have rwlocks in xpdev (yet?), hack together a crappy rwlock
that does what we need.

Also, remove "CONSECUTIVE", which doesn't really make sense when the number
of login attempts in 1.

There's a hidden checkbox on the CodeInputForm now (explaining why the edit/
combo box is now moved up).

Use lower case y/w/d/h/m suffix.

If a duration is an exact multiple, don't display ".0" fraction. This is not
consistent with byte_estimate_to_str() string generation, but I think it's
more in line with human expectations.

Don't display "90d" as "12.9w" (unless the requested unit is one-week).

Renamed the "Perm Filter Threshold" to "Auto Filter Threshold".

Changed help text from "Permanent" to "Persistent"

More readable/friendly log messages and ip.can reason strings