Some ZMODEM implementations set the high bit (even parity?) when sending this '\n' terminator.
As reported via IRC:
<Keyop> sexyz: !zmodem_recv_hex_header HEX header not terminated with LF: 138 (8Ah)

After calculating i (offset in list to top of window), if it's less
than zero, apply it to bar and set i = 0.

Fixes #218

Feature requested (?) by u/jumbotronjim on https://www.reddit.com/r/synchronet/:

If the client connection is from a blocked IP address (in ip[-silent].can), but still manages to get through the web server and websocketservice and have their correct IP address reported via Telnet Location, terminate the connection. Seems dubious.

Fix issue reported WitNik (BGGRSCYN) on DOVE-Net:
```
sbbs@raspberrypi:~ $ /sbbs/exec/umonitor -iC
...
sh: 1: /sbbs/exec/syncterm-iC: not found
```

These configuration properties were not previously exposed via JS.

No immediate use/need, but I *almost* did. :-)

"Old style" (e.g. FTN netmail) attachments put the filename(s) in the message subject. Supported quoted-filenames in the message subject (i.e. to support filenames with spaces in them) in addition to the traditional space-delimited filenames. Mixing quoted and space-delimited filenames (for multiple attached files) in a single message subject is supported.

It should be either EACCESS or EAGAIN, but still, could be useful to know.

Since on 2.17, pthread_afork() is not in libc, -lpthread must be
after -lcl (library link order issue)

Fixes #211

BIG thanks to Scott Comstock for giving me access to his box to fix
this.

Also change the wording of the -D option to be more descriptive.

The "EchoLists" help menu only worked when you first enter the sub-menu and would get wiped out by subsequent child-menu help text.

Filled out the "Configuring an EchoList" help text.

Fixed issue seen where deleting an EchoList's Required Key didn't set the "dirty config" flag, so saving of changes was not prompted when exiting (if that was the only change made).

Also, it's not technically an option, so remove from the option list.

Fixes the Win32 build.

Done.

At Tracker1's request/suggestion.

jsexec -Itrue
...
Result (boolean): true

More elaborate expressions can be evaluated and white-space included by quoting the "-I expression".

Also, send all cmd-line usage errors to stderr and syntax help to stdout, always (Deuce's request).

Stop lying. Programming is hard enough already.

The extra spaces caused the centering of the cryptlib version to be offset (not exactly centered).

Now the default bar is INT_MAX, so it always gets corrected when
left as default.

This is good.

Instead of setting bar to zero when possible, set it to the number
of displayed options, putting the lightbar at the bottom of the
displayed list instead of at the top.

So here's what we're doing now... we set bar so that the first item
shown in the menu is the lowest index possible that still shows cur.
bar is therefore calculated as the displayed item count minus the
distance from cur to the end of the list.

Yeah, that's the easiest way to explain it.

scrn, showbuf, timedisplay, bottomline, getstrxy

iniGetNamedStringList() was returning NULL when attempting to get the list of keys in the "root" (unnamed) section and the file is actually empty/blank. If there were *any* characters in the file, it would return an empty list (not NULL). Now, always return the empty list, even when the file is blank.

Doing so prevents extending the prototype.

The default character set for outbound mail is now auto-determined (when not explicitly specified for a message) between UTF-8, ASCII, and CP437. The [mail] DefaultCharset setting (which fell-back to iso-8859-1 if blank) is no longer "a thing".

Also: specify 8-bit content-transfer-encoding for the (potentially UTF-8 or CP-437) plain text portion of a MIME-encoded message with file attachment (7-bit was wrong) and pass down the text sub-type (e.g. could be "html") for inclusion in the mime-part header (don't assume text/plain, but still use that as default).

Also: log an error when failing to delete an attached file (e.g. from data/file/*.out).

Some (important) File methods did not support .ini files that used the !include directive because they were using the xpdev iniRead* API (which performs no "pre-processing") instead of xpdev iniGet*.

Impacted methods:
- iniGetValue()
- iniGetKeys()
- iniGetObject()

The other existing ini* methods already worked fine with nested (!include'd) .ini files. It's possible there's a slight performance penalty with the new implementation since the entire .ini file is always read for each operation and previously it was possible that only a few "lines" were read to find the key(s) of interest. However, since .ini files are not typically huge and the iniRead/file-stream method likely read large (e.g. 8-32K) blocks anyway (which is usually the entire .ini file) - I don't actually suspect any observable impact to performance.

This change was needed for the new ctrl/modopts.d support.

Added new method useful for debugging nested .ini files:
- iniReadAll()

This allows auto-inclusion of sub-directories of config files (e.g. !include modopts.d/*.ini). This, along with the next commit to js_file.c, allows module authors (or sysops) to keep their module-specific settings in a module-specific file (rather than always merging-with/managing the monolithic/shared modopts.ini). The [modname] section heading is still required in the included .ini file.

Replaced use of non-thread-safe STRERROR() with safe_strerror().

Example:
Command-line: ?showargs   " a b c "d "e f"
argc = 3
argv[0] = ' a b c '
argv[1] = 'd'
argv[2] = 'e f'   

This resolves a long-standing TODO comment.

Also, fixed a problem where multiple spaces between the module name and the first argument would result in argv[0] being set to an empty string.

The first argument can now optionally be an array of strings (e.g. as read from File.readAll()), so that multiple searches of the same file (e.g. twitlist.cfg, while importing messages) does not require multiple *reads* of the same file.