update frame.putmsg() "normal" ctrl-a code to use attributes from frame initialization (ansi "normal" is relative to the frame's settings). to emulate standard "normal" behaviour, initialize frames with BG_BLACK|LIGHTGRAY

Apparently, calling close(-1) on WinXP will cause a crash. Don't do that.
Make closeuserdat() resilient to an un-open file descriptor and use it
instead of close().

Hopefully The BRoKEN BUBBLe BBS stays "up". :-/

Also, added Starstocks. Stalk your favorite stars here!

This fixes the stat() issue on Windows XP/2K3 by allowing sbbs to benefit
from the run-time library updates that Microsoft releases periodically
(like https://www.microsoft.com/en-us/download/details.aspx?id=53840).

For more info on the stat() issue which caused all kinds of sbbs errors
(e.g. creating directories initially, but a lot more):
https://stackoverflow.com/questions/32452777/visual-c-2015-express-stat-not-working-on-windows-xp

Since we are using cryptlib which requires the MSVC runtime DLL anyway
(it is the default build behavior of MSVC), we weren't really gaining
anything from statically linking the CRTL (LIBCMT.LIB). And for some reason,
an up-to-date MSVC2019 still has (links-in) a LIBCMT.LIB file that includes
this stat bug. All the online help resources I found just to seem to suggest
updating the CRTL DLLs (on the target system), with no mention of any fixes
available for the static CRTLs on the build system. But with the no gain
from static linking anyway, I figured it was time to switch to DLL CRTLs.

The debug builds are still statically linking the CRTL for no particular
reason.

So don't bother linking to 00index*, just the ftp root dir should work
(for a while yet, anyway)

Previously, just the NodeNIsNotInUse message would be displayed.
Instead, display the NoNeedToSendMsgToSelf message.
Sysops are free to message themselves.

active_clients is destroyed in cleanup(), so it must be re-initialized inside
the server init/recycle loop.

bool (JSVAL_FALSE), not number (JS_FALSE).

Throw an exception if no filename is specified (rather than just abort the
script by returning JS_FALSE).

The return value is supposed to be a bool, not an int (JS_TRUE != JSVAL_TRUE).

Fix reported and observed crash in cleanup() (in ftp, mail, websrvr)
when failing to create the temp directory. This was due to cleanup()
being called before the protected integer "active_clients" was
initialized.

Also, md() needs to return the errno value (not a BOOL) since the
caller may be in another DLL with a different errno (which likely
has a value of 0/no error).

... in other non sbbs.dll modules (e.g. *srvr.dll).

So now log error messages elsewhere where appropriate.

DLL-exported and optimized md() a bit.

Extended node status can (and often does) contain Ctrl-A sequences.
Strip those characters/sequences before putting into the NodeForm->ListView
lines here.

If the user's terminal was auto-detected as being ANSI, yet they had their
account settings manually-overridden to use PETSCII, automatically switch
back to auto-terminal mode (e.g. with ANSI).

Read and write the Log fonts when importing/exporting sbbsctrl.ini.
Use sbbsctrl*.ini as the default Import Settings file mask.

This was done in the default/classic command shell, but no others.
The termsetup module handles terminal setup tasks that should correspond
with the user's configuration of their terminal preferences (e.g. iCE colors,
alternative fonts). If escape sequences need to be sent to the terminal
server (the BBS) to coordinate the settings, this is the module that does
that.

Script errors-out if a user selected it and there was no such directory.

find_startup_dir() returns an index, not a bool.

Create a default server.ini pointing to localhost:10088 (same behavior as
if the file is absent) and allow the installer to modify it to point to
the Executive Chiggun's B.B.S. if the sysop opts-in.

Don't prompt a new user to forward email to their netmail address if they
provided an invalid netmail address (not supported by the system).

If a sysop has an invalid netmail address setup for forwarding, don't try to
forward email (or new user feedback) to that address.

Re-ordered the new user terminal questions a little bit (ask for the backspace
key first, to get earlier manual PETSCII detection). I'm not sure why I was
enabling AUTOTERM along with PETSCII before. Removed that as PETSCII cannot
be auto-detected.

I noticed on one particular system that Canceling or Completing
the configuration wizard on a fresh install, sbbsctrl.exe would
just shut down (no error dialog or anything, likely a crash of
some kind). Instrumenting StartupTimerTick() didn't reveal anything
useful (it ran to completion).

By changing the method of launching the Configuration Wizard,
I was able to eliminate this observed problem. Now, the StartupTimer
runs twice on a fresh install (just once for a normal startup),
and the second run of the StartupTimerTick starts the configuration
wizard.

I also reverted to the previous behavior of dynamically creating
and destroying the wizard for each use. There's just too much state
information to restore if the config wizard is run a second time.

Also, removed a bunch of old Registry settings readings (v3.10/11
upgrade support) and commented out code.