- Attempt to suppress go aheads in both directions.
- Attempt to set binary mode in both directions
- Request the remote echos

These express the assumptions that SyncTERM always makes, so it's
a good idea to tell the remote this.  For systems that don't like
this, there is the Raw "protocol".

When changing between 8 and 16 row fonts in RIP mode, the cursor
start and end was left for the old font size... 16 -> 8 would leave
the cursor one line below the current position, and 8 -> 16 would
leave the cursor in the middle of the cell.

This hacks the vstat deeper to fix up the cursor as well.

Reported by skipperdoodle1947.

This is a code clean-up, no change in functionality

It's possible to have multiple transfer protocols with the same mmemonic, but
with different transfer types supports (which would be weird, but possible).
This allows us to use protnum() to replace a lot of copy/pasted prot looping
and comparing logic throughout sbbs.

Also added a variant of sbbs_t::quit_key() that takes and returns a string
(for easy concatenation to strings of key chars).

If an ANSI sequence spanned multiple recv() calls, it would get
silently dropped if RIP was enabled.

Reported by skipperdoodle1947 (who does awesome stuff)

for use by JS bbs.xtrn_prot_menu() immediately, elsewhere soon.

Also use console.quit_key for localization support

In place of the copy/pasted/ported from C++ version of the same logic

... that the user has access to

So we don't have to reimplement this logic in user_settings.js (or
equivalent) any longer.

Also, fix manpage generation for CMake.

I've forgotten one of these for every RC except the first one for
v1.2. :(

Thanks nelgin.

The amazing Haiku community has redesigned the SyncTERM icons for me!

Unfortunately, I had to remove the default "Synch" icon from ciolib
so that SyncTERM could have its own separate icon. :(

I'm not sure if the msvc builds are using the ciolib.rc file or not,
and I'm not sure how they can add an icon... I'll net DigitalMan sort
that out (sorryish).

MRC: Session stats, mentions, themes, indenting, buffer display, and more

See merge request !464

This was sometimes causing an integer scaled mode (ie: 4×) to be
reduced to very slightly less than that (ie: 3.996255×), making the
window one pixel smaller than it was supposed to be due to integer
truncation.

This could result in being unable to increase the window size using
the Alt+Right-Arrow shortcut and could make new windows one full
size smaller than they needed to be (such as C64 windows).

While we're here, fix the SDL and X11 outputs to trust what
bitmap_drv_init_mode() does.  It was fixed when GDI mode was being
written, but the other drivers weren't updated to take advantage of
that.

Should resolve issue 156 reported by DigitalMan

The ini key value name remains the same: batxfer

<nelgin> batxfer? Why not called it batchxfer ? The extra 2 characters help :)

[shrug]

(for new installs)

Replaces hard-coded batch transfer menu, same functionality

Whee.

Use localizable [Q]uit key

Removed trailing whitespace

Now it doesn't leave the console all messed up.

Helpful for a JS batch file transfer menu

Returns count of files removed now (not bool).

This allows GetConsoleScreenBufferInfoEx() to work for palette
changes, allowing Console to not suck.  However, we now prefer
ANSI mode over Win32 Console, so make that so it can actually
fail on Windows of stdout is a TTY and we can't set the
ENABLE_VIRTUAL_TERMINAL_PROCESSING flag.

With this, ANSI mode is used in Windows Terminal, and Win32 Console
is used in Legacy Console, and all should be good with the world.

(as is/was done in useredit.cpp)

Fix the default download protocol columning display (missing final
newline in many cases).