No change in functionality, just code comprehension

If enabled, this option prevents logins to this service from changing the
protocol (aka modem) field for a user logging-in.

This is a work around for Deon who has some message areas set to require
"PROT NOT IMAP" for access requirements and when his IMAP client connects
and logs-in, that changes the protocol field in his user account (to "IMAP")
which means that user account cannot see/read those message areas (until the
protocol field is changed by another login to another server/service) and
thus won't receive echomail notifications for message received for them in
those areas.

Defeat the Windows getaddrinfo feature:
If the pNodeName parameter contains an empty string, all registered addresses on the local computer are returned.

at least for me, "fe80::5dfc:b29c:85d6:8de7%21"

at least, "vert.search.charter.net" in my case

This is just the ones mentioned in the jsdocs.  I'll add more
if/when I want them. :D

corresponding wth new user DELDATE field

alert() doesn't throw exception on null/undefined, just no arg

deny() throws exception on no args or null/undefined arg

So don't check for null/undefined argv[0] if not passed one

Hopefully this will fix the occasional crash in the testsuites
(and presumably, occasional crash in other things that use background
threads with js.exec(), if there are any).

Except this time, in the right place.

This reverts commit ed8c30da.

This reverts commit 7eb498aa.

Hopefully this will fix the occasional crash in the testsuites
(and presumably, occasional crash in other things that use background
threads with js.exec(), if there are any).

User records (aka slots) will be preserved this many days since the user was
deleted (if the DELDATE field is non-zero) otherwise, days since last user
activity.

The initial purpose is to support new user field: DELDATE (user record
deletion date) - this allow user record retention for a period after
deletion rather than after period after last user activity.

The Win32 usereditor needs updating to use this function too.

Reported in unmonitor, which always passes NULL.

Previously, different files had different sets of wrappers around
pthread_mutex_* and rwlock_* functions.  This commit removes those
and puts assert_ prefixed ones in threadwrap.h and rwlockwrap.h.

It also uses them in all the bits that SyncTERM uses.

This means debug builds are much more likely to unexpectedly
terminate with an assertion... when it does, let us know!

While we're here, add debug assertions to the pthread stuff.

More method will now throw exception if called without args:
bbs.check_filename(), bbs.logkey(), bbs.export_filelist()

bbs.check_syspass() will prompt for system password if passed null or undefined
value.

Fix (and document) the optional channel argument to bbs.multinode_chat()

Throws an exception now instead of using uninitialized args:

;eval
Parameter(s): bbs.compare_ars()
InternalError: allocation size overflow

Updated JSDOCS for methods (mostly just adding punctuation). Fixed a typo
or two.

Likely will fail for Linux if we need some OS-specific suppressions
like we do for FreeBSD.

... an exception (e.g. chat_sec.js), so return false instead.

This fixes issue #864

Thank you Accession and Nelgin for the report

Now that valgrind is working, remove the verbose flag, and have it
return 1 if any errors occur.

We need some JS_GC() suppressions though.

make the output less 'chatty'