Don't ever route netmail when the destination zone/net/node matches a local
AKA (ignoring the point value).

Resolution for the issue of there sometimes being a few empty lines between the end of the BBS list and the prompt text - For example, on the last page if the BBS entries don't fill the whole screen.

Addressing a few issues noted in the previous commit: Functions declared after main() are now before main().  Fixes a display glitch when moving the selected BBS using the up/down arrow keys.  The redraw_whole_list variable now defaults to true and is set to false when needed, minimizing the number of times it needs to be changed.

More efficient screen updates (i.e., when using the up & down arrows, only move the lightbar on the 2 affected rows, don't redraw the whole list unless going to the next/previous page)

- After a lost game, allow the use of (D)isplay to toggle the game board
  display from the normal (all cells uncovered) to the last state of the board
  and the number of remaining mines, difficulty level, and remaining time
- (W)inners command now supports [All] option which displays the top-N winners
  of each level instead of only top-N winners for a specific difficulty level
- New (B)est command displays your personal best win times
- Display game times in fractional seconds (milliseconds), always
- Upon winning a game, inform the player if the win is a high/ranked time for
  that difficulty level or a personal best time

If you try to stay at the inn with a charisma of 100 or more, the innkeeper
says he won't charge you... he doesn't but you don't stay at the inn either.

Renamed the option noone_logged_on_fmt to nobody_logged_on_fmt (grammar).

If a packed messages contains no PATH or SEEN-BY lines, we can still detect
and prevent a message loop by comparing the origin address in the packet header
against the downlink's address and if it's a match, skip that downlink.
It is still possible that a packed message header contains a different origin
address than the packet header, and we're actually over-writing the packed
messge header variable with the parsed Origin: line address (if there is one),
so perhaps we'll want to compare the (actual) packed message header origin
address too at some point in the future, if loops continue to be a problem.

If show_logon_list is set to 'false' in the [logon] section of modopts.ini,
don't show the logon (last few callers) list during logon.

write_to_pkts():
If for some weird reason a downlink's address is in the PATH, but not in the
SEEN-BYs, detect as a loop and don't send to (add to an outbound pkt for) the
downlink.

This is an experimental change to see if it addresses the issue reported by
Richard Williamson and Mark Lewis with regards to dupes in the COOKING
echo from point nodes off 1:396/45.

Add a few (badly drawn) "boom" splash screens, displayed when a mine is
detonated.  I really would like an ANSI/block artist to replace or add to
these with some decent art (Grymmjack, Kirkman?).
Fixed a couple of cosmetic issues I noticed when watching the Minesweeper
YouTube video back, e.g. extra digit on the count-down timer.
Fixed a bug in place_mines() that could place one too-few mines on the board
if the player were extra lucky (the initially selected cell was chosen for a
random mine placement).
When detonating a mine while cloning, display the detonated mine in the file
board screen as detonated (and not a regular/undetonated) mine.

running a JavaScript module would crash (e.g. segfault) sbbs:
Create and use a separate JS runtime, context, and global object/scope for
global hotkey events. This means that the hotkey won't benefit from any
previously loaded/required scripts, possibly effecting the performance of the
first invocation of the hotkey handler. Subsequent JS hotkey events will reuse
the same runtime/context/global, so they'll execute fast(er).

One questionalbe change to js_execfile():
With the JS_GC (garbage collection) call *before* the JS_ENDREQUEST() call, the
process would crash in libmozjs. Moving the JS_GC() call to *after* the
JS_ENDREQUEST() resolved this issue and I'm not clear why. This 'js_cx'
parameter here is not always sbbs_t::js_cx. When called to handle a JS hotkey
event, it's sbbs_t::js_hotkey_cx, so it shouldn't interfere with the
sbs_t::js_cx being used by the currently executing JS module (shell or door).
<scratches chin>

brought down [cvs.]synchro.net this weekend. It's hard to believe we still
have unchecked calls sprintf() around, but we do.

drafx() methods. For consistency with how Minesweeper vertically centers its
game board.

in case something (e.g. Argus mailer) deleted it while we were waiting our
turn to use it. As reported by Alterego (ALTERANT)

Set generous default document size limits.
Cache text files and directory listings, nuke everything else after it's
sent to the client.

Everything has at least the default colour, no need to only fetch
colour for "link" types.
Use console.putmsg P_NOATCODES and P_NOXATTRS via ScrollBox to avoid
danger and messed-up displays.

Use P_NOPAUSE with console.putmsg  instead of turning off pause globally.

These are the vagenta menu items.  A query prompt will appear in the
address bar at the top.  Enter your search terms.  A thrilling new
dimension of gopher interactivity.  What a thrill.

directories anyway.)  Returns to where you were on the page.  Maybe
could do this for text files as well.
Allow download of text file currently being viewed with 'd'.

This will be used for internal documents (help, bookmarks) where
selector is a path relative to js.exec_dir.

with strip_ctrl.

Added o u r R j k key commands.
More feedback from status bar.

Muchos faster.
To-do list ... mostly unchanged.