Fix Ctrl-Z/Undo so it only redisplays from the correct column/offset.
Note: K_NOECHO mode appears to still have some bugs if/when K_EDIT is also
used (which would be a weird combination of modes).

Don't allow a message title input prompt to exceed the width of the terminal
(noticed when uploading a file to be emailed from the terminal server).
This really should be a generic solution in getstr(), which I'll get right on.

- instead of enabling/disabling raw input mode

exist in msgbase_open(), called from sbbs_t::postmsg().

Replace a lot of hand-rolled/pasted SMB message text creation code
in sbbs_t::postmsg() with a call to smb_addmsg(). This fixes the problem of
messages posted using the terminal server were not subject to message hash
create or dupe checking.
Also created a (currently private) helper function called msgbase_open() which
contains a lot of boilerplate code that is usually pasted around/before/after a
call to smb_open(), so I plan to use this more later.

Access_D's reported problem with re-routed netmail.

if/when spy socket is disconnected.

to inform the MSVS version selector which IDE to open the solution file in.
This file was not updated during the migration from MSVC2010 to MSVC2013 and
appears to be another sighting of this bug:
https://connect.microsoft.com/VisualStudio/feedback/details/793984/sln-file-not-upgraded-to-vs2013-by-version-selector

Now using a local-scope variable for temporary mailproc pointer.
Log a warning when external mail processor creates an error drop file.

base could not be opened for some reason.

(part of previous commit of main.cpp)

Only used in IPv6 patch right now.

detection/conversion when exporting message text not work if the message
contained line-feed terminated lines.

when the destination node was configured for "Direct". Apparently others have
been working around this issue with a "Route to" configuration directive to
route a node's netmail to itself. This work-around should no longer be
necessary, but anyone that was depending on the old behavior (routing mail
for nodes configured as "Direct") will need to adjust their configuration.

posted messages, not just email/netmail.

bit-flag macros).

file. Previously, any threshold set by manually editing the .cfg file would be
lost when saving changes using echocfg.

USEPACKER...
Basically, if there are more than one USEPACKER lines of the same archive
type as the new one, the user will be added to the first instance of the
USEPACKER, and every other USEPACKER line of the same type will be deleted.

We likely don't want to do that either.

all AREAFIX lines except for the one that was updated.

This is apparently "bad".  Stop doing this bad thing (We'll still delete
AREAFIX lines without an address though).

eventual failure to open any files or even log the failure in the error.log.

destroy the ssh_mutex (which was not initialized).

for the static 64-byte sbbs->telnet_cmd buffer, it will bork the session.
(Seen with a long user-name and DISPLAY when using the OS X telnet client).

FTP passive port range, and all ports would fail to bind, the logged error
would report failure for the last port+1 I(rather thant the last port).

Beautify a lot of the remaining strange closing brace style.
Use FREE_AND_NULL() where appropriate.
Log SMB error strings for most common smblib function failures.
Log debug output in the areas.bbs alter function (help isolate Joe's crash).

requests - should handle more unexpected echolist lines (e.g. extra long, blank
or whatever). Hopefully resolves crashes in strdup() Joe is seeing.

with an invalid file descriptor (e.g. -1), don't attempt to close it.

SAFECOPY() ain't safe for pointers... use strdup() since that's what we
want anyway.

variable (with various race conditions). This resolves a long LONG standing
bug that caused crashes when shutting down or recycling servers/services
while one or more servers/services were in use.
This change requires every thread which calls into libjs to have its own JS
runtime.

skipped/ignored: import_netmail() moves the file position to the end of the
message but returns non-zero (-2) when encountering an AreaFix message
and the calling code assumes a non-zero return value indicates an error and
the file position is somewhere *before* the end of the message (the NUL char).
Resolved by saving the current file position before calling import_netmail()
and restoring it after, so it matters not what improt_netmail does with the
file pointer or what the return value is (in this case).
Thanks for the report, Joe!