Fix some incorrect manual conversions from `c to `C

some clean-up items left...

1) Deleted player slots aren't re-used.
2) The in-memory log is never trimmed.
3) Useful auth things and a signup thing.

scfg.prepped         = TRUE
scfg.tls_certificate = -1
scfg.sys_inetaddr    = "example.com"
scfg.sys_name        = "JSDoor"
scfg.sys_pass        = "ThisIsNotHowToDoSecurity"

May help with the invalid STATUS roundcube is sending from Krueger.

The scfg_t instances are often shared between threads and we can't really share
file descriptors between threads, so we ended up with file descriptor leaks and
race conditions.

Instead, define/allocate/free a new js_system_private_t where the scfg_t* and
the nodefile (descriptor) are kept - per "system" instance.

subject header fields are non-ASCII yet valid UTF-8, set the auxattr flag
indicating that the msg header is in UTF-8.
This function is used by JS MsgBase.save_msg(), so any scripts/services that
use it (e.g. nntpservice, newslink, imapservice, etc.) to save messages with
header fields that may be UTF-8 encoded will automatically benefit from this
detection. Mixing CP437 (the default) with UTF-8 encoded header fields is not
supported, so if one header field is valid UTF-8 (not plain US-ASCII), then
they had better all be either plain US-ASCII or UTF-8 or there will be
UTF-8 decoding issues when attempting to display or convert later.

object/struct doesn't have any meaningful data. Prevent this from happening:
    SenderTime       19700101000000Z
    SenderIpAddr
    SenderHostName
    SenderPort       0

closeit parameter is TRUE

- If a telnet command was received in multiple packets, the memcpy optimization
  in telnet_interpret() would skip/drop all bytes in the subsequent pkt payload
  before an IAC char. Don't optimize when in the middle of a telnet command.
- If a received telnet command exceeds the telnet_cmd buffer, reset the
  received telnet_cmdlen and log a warning-level message
- If a telnet sub-negotiation END command is received as the beginning of a
  new telnet command, log a warning-level message and reset the telnet_cmdlen.

This fixes the occasional problem observed when using fTelnet and its sending
the "SEND-LOCATION" sub-neg command split between 2 TCP packets. Only
part of the first packet would be used as the location data and the rest
processed as input from the users (e.g. as the Login: prompt). Thanks, Ree!

This seems to unbalance the game, and forest fights just aren't dangerous
if you kill a couple players every day.

I may set FiveSkillPoints by default too since it's the last 4.x change I'm
not defaulting to reverting.

FiveSkillPoints controls if thief and death knight uses take four or five
skill points each.  I haven't figured out exactly when this change was made,
but I think if Seth made it, I'll leave it as default, but if Mike made it,
I'll change the default back to what Seth used.

blank lines.

Only feature remaining is the online battle stuff.
Whee!

Elminate unused member variable: inputfile

(ctrl/node.dab) is constantly closed and re-opened for every non-locking read.
This is really slow across network file systems and unnecessary, so use a
similar optimization as the C++ sbbs_t class where the file can (and normally
is) left open across multiple consecutive reads.

(ctrl/node.dab) is constantly closed and re-opened for every non-locking read.
This is really slow across network file systems and unnecessary, so use a
similar optimization as the C++ sbbs_t class where the file can (and normally
is) left open across multiple consecutive reads.
Create/use opennodedat() function.
Uses the new CLOSE_OPEN_FILE() macro from xpdev/filewrap.h.

the file and sets the descriptor to -1 (if not already < 0).

deprecation warning from MSVC.

in the archive, so the original wasn't maintained.

Store/restore the user's choosen tabstop value (if not the same as the default)
in/from user's props file (data/user/####.ini).

https://patchwork.kernel.org/patch/9289177/