As Andre pointed out while documenting this setting on the wiki, the option seemed confusing: if a sysop could not login with "system operator access", how could they login at all? Answer: they could not.

This setting used to be called "Allow Remote Sysop Logins", back when there was the concept of a "local login", so setting this option to "No" would mean that user accounts with sysop access could only be used for *local* login. But in Synchronet v3, there's really no such concept as a "local login", so it was changed to just "Allow Sysop Logins" (period) and not a lot of thought given to how/why a sysop would actually set to this "No" or what the implications would be (presumably, nobody ever sets this to "No").

So rather than just get rid if the option altogether, I changed it to mean: an account with sysop access (i.e. level 90+) can still login, but any action that normally requires the system password will not be allowed. This includes the sysop-actions available in the FTP server when authenticating with <user-pass>:<system-pass> as the password. The sysop-user can still authenticate (and login), but none of those sysop-actions will be available to them.

Partially revert Deuce's commit ea5b359b

This explains the EAccessViolation (stack corruption during sbbs startup) that sysops have reported when using the latest nightly builds.

This macro as written won't compile on mingw32 due to the attributes
on _mktemp().  This could be worked around, but it's easier to just
disable it since I don't need it, and nobody else is building with
mingw32.

On Win32 it can't be defined until after winsock*.h is #included.
On everything else, it needs to be a pointer.

For Win32, this is actually the pointer type an ioctl() third
argument needs to be.  For other OSs, this is basically what an
FIONBIO third argument needs to be... there's really no restruction
at all on what arguments after the second to ioctl() need to be.

It seems we can't add a const to a typedef in a static_cast<>()

uintptr_t should be the same as a DWORD on Win32,
and CRITICAL_SECTION can be stored in an inptr_t as well.

This should work fine, and allow more build environments to be happy.
This is generally caused by new C++ standards having thread support
in the standard library, so wrappers getting pushed deeper in.

At some pointer, this should be redone with standard threads, but
now is not the time.

@-code expansion in text lines. Text searching for sub-board, group, or all.

See merge request !143

This is so we can deal with send(), recv() and friends taking
a void* on BSD stacks, and a char* on Win32.

Not a big deal for C where a void* is universal, but C++ hates that
type of thing.

Version 1.44:
Removed the scanScopePromptText text line and used the SubGroupOrAll line (621) from text.dat instead.  Also, the reader now supports @-code expansion in configured text strings.  This addresses issue #316 from Vincent Macaluso.
Text search now can search sub-board, group, or all like the other text searching (requested by Nelgin).
When reading the theme file, color settings are now checked to ensure they only have Synchronet attribute codes.

Thanks to Andre for pointing that out in his wiki updates.

If absent, default string 'Bulletins' will be used.

Removed attribution rather than fix its coloring. Nobody cares,
and if you do care, have you tried not caring? It's easy, it's
just a matter of doing nothing.

Bullshit is a proud sponsor of funclib.js.

On Windows, CRs in extended descriptions could result in output of
\r\r\n which could be displayed as 2 newlines (e.g. in notepad.exe).
Since the \n -> \r\n expansion already happens on Windows stdout,
we don't want them in the processed input (file metadata).

As reported by DesotoFireflite (VALHALLA), thank you!

As pointed out by DesotoFireflite (VALHALLA), both the file's
short/normal description and the extended description were being
included in all file listings. This is probably want most sysops
will want since the extended description is often used as the content
of the short/normal description, resulting in repeated content. So,
by default, if there is an extended file description, don't output
the short/normal file description.

If you want to include *both* the short and extended descriptions
(for files that have both), include both the -ext and -desc command
line options.

Initial support for offline message counting in msglist.js.

See merge request !142

Added options:
 -count
 -all_subs
 -hide_zero

This allows this script to be used to create counts of messages in the groups.
Typical use: Stats on new messages in the last x days.

Inspired by filelist.js (which is really *only* for use offline),
added support to run via JSexec or timed event with command-line
options to control the output format, sorting, and msg-age range.

Try this Mewcenary (EXTRICAT)

I think this means one of your nodeon.* files are corrupt and just deleting
that file would likely cure the situation as well.

Convenient way to get the path/name of the archive file as it was passed to the constructor.

Digital Distortion File Lister v2.02: Added the ability to do a file search (via filespec, description, or new files since last scan).

See merge request !141

a file search (via filespec, description, or new files since last scan).
A command-line parameter, -MODE, specifies which search to perform
(search_filename, search_description, or new_file_search for searching;
list_curdir lists files in the user's current directory, which is the
default).

Digital Distortion Area Choosers v1.22: Fixed a lightbar directory choosing issue in the file area chooser

See merge request !140

fixed a lightbar file directory choosing issue when using name collapsing
(was using the wrong data structure)

Since we install archive.js as the viewer for '*' by default, we expect
it to be asked to list non-archive file-types frequently, so don't be
too alarmist about it.

And change XMODEM-1K/CRC to be the '1' protocol.

warning C4244: '=': conversion from 'double' to 'uint64_t', possible loss of data

This is a millisecond timer that tries to avoid floating-point
operations.