Because the appropriate header wasn't included, there was no
prototype.  Not having a prototype is an error in C99, which caused
the test to indicate that there was no va_copy() on darwin.

This in turn caused jsprf.cpp to try to copy a va_list using
assignment.

This is invalid, so the compiler would normally return an error.

Somwhere in here, that error became a clang crash (not trivially
reproducable).

At the end of the day, ancient broken tests in config caused the
darwin Spidermonkey builds to fail.  The new patch fixes up the
configure tests, and darwin Synchronet builds are re-enabled.

Thanks Ree!

Give simple test code a return type (default int isn't valid in C99)
and make some fallback tools executable.

Whoops.

This should normalize everything, with the possible exception of
"zoom" (the thing other OSs call maximized).  Ideally, we wouldn't
allow window size changes when "zoomed" if we could detect the state,
but it doesn't look like we can.

Fixes sourceforge issue 126 (finally!)

Apparently, macOS "maximized" is the same as "fullscreen" in SDL.

Hopefully this is the only lib like this (maybe the resources?)

strListReadFile() can return NULL upon error

Fix CID 509721

Fix for CID 509720

Make ssh_active only true when ssh_session was set.
Only do ssh cleanup if ssh_active is true.
Initialize channel values to -1 (already done in ssh_connect(), but
doesn't hurt anything).

With ssh_sock initialized to zero, if the connect fails, stdin was
closed, preventing ANSI and curses modes from functioning.

First, we need to parse it before load_settings() so we can deal
with it there.

Next, we need to keep the one from the config file available so it
is edited from the program settings, and not the command line version

Keeps the display cleaner when things work and the shift in byte
is displayed.  Should be harmless in cases where shift in is
interpreted per the standards.

Hopefully I'll be able to keep them all wrangled now.

It's likely time to find a nice format that can be used to generate
HTML and PDFs and such now.

Prevents crashes on modern FreeBSD systems that default to implicit
PROT_MAX in the JS code.
(It's possible that +wxneeded will be needed too.. still experimenting)

The first entry I tried adding was "Example..." which was not a
valid hostname, but was copied into the address and looked silly.

Now only copies into address if it's a valid, usable hostname.

Also, add a convenience macro to check if a connection type uses
the network or not.

New functions in netwrap:
bool isValidHostname(const char *str);
bool isValidAddressString(const char *str);
bool isResolvableHostname(const char *str);

We ust the last one (which calls the other two) for this feature now.

Use byte_estimate_to_str() to display upload/download byte totals (better
support for totals in gibibytes, tebibytes, etc.).

I discovered that sbbs_t::editfile() (exposed as JS console.editfile) just
silently failed if it couldn't write to the destination file
(e.g. permission denied by OS).

iniReadFile() can return NULL upon file read error or malloc error. Let's not
crash if that happens.

Even if the empty, iniReadFile() should return an empty list (not NULL),
NULL indicates a file read (or unlikely malloc failure) - so don't write back
what we read and return false (result) instead.

Part of likely work-around of issue #791: it's better to not increment stats
upon a file read error than to reset them all to zero.