improved mouse handler

I improved the mouse handler in a way which will preclude disappearence of pieces
on off-board or ambiguous (in-between squares) drag & drop moves. (As was reported
in the "premove bug?" thread. Also the command-line premove problem reported there
is now fixed, by removing the prefixing.

fix for bug #8847: moving backward while examining on FICS not reported to engine

I moved Daniel's code that backs up the engine when a bord with a lower number is received to a place where it is always done in zippy mode, rather than only when the engine is analyzing.

Apart from taking back moves, there was also the case where the
user moves forward by more than one move (usually immediately to
end-of-game). In this case a new movelist is fetched from the ICS
to recover the moves that were skipped. But this assumes that the
engine is at the start of the game and feeds all these moves to the
engine. So the engine had to be reset in this case to stay in sync.

better init for random number generator

fix for bug #27772: holdings not updated

holdings lines from the ICS nor update the holdings of boards[forwardMostMove] in stead of boards[currentMove].

allowe parsing / disambiguation of SAN moves like Xe4 in certain situations

This patch allows parsing / disambiguation of SAN moves like Xe4, if there
is only one wild-card piece X on the board. Moves with wild-card pieces
are now considered illegal if they leave the King in check by a piece of
known gait.

fixed bug when switching to variantsuper

fix for bug #27715: 2 (minor) graphic issues

I forced a total redraw when Reset is called from the ICS code in case a
new game is encountered, in the hope it will make the reported
(irreproducible) problem go away. As game switches are not frequent, I
guess there is hardly any downside to this.

fix for bug #27667: PV line missing in analysis window, part 3

"3. Wish for Engine state string (right to the icon): Can you add the actual search depth in analyzing mode? A change like "

I also added the requested display of depth of the analysis move in the
EngineOutput window. I guess a similar change could be made to wengineo.c (as this is in the
code they share in common), but no one ever requested that in the 5 years
since Alessandro wrote it. So for now I leave it alone.

fix for bug #27760: debug printf in backend.c and additional check for variant

fix for bug #10990: cmail does not seem to support .cmailgames or .cmailaliases

this is correct, deleted the text out of the documentation that claims we support it

new alpha version; first one with new naming scheme

smarter analysis of the boards that XBoard receives from an ICS

I added smarter analysis of the boards that XBoard receves from an ICS,
based on board size and piece types in the position, to recognize cases
where the board does not match the current variant, and switch to an
appropriate variant.

fix for bug #27667: window should be reference to toplevel

The remaining objections of #27667 should be fixed by bringing the
EngineOutput window under control of the TOPLEVEL compile-time switch.

partly fix for bug #27715: scaling of menu bar

the menu bar to the left edge of the window.

fixed bug reported in WB forum: second game of a match would not start when using the GUI book

fix for bug #27751: negative holding counts displayed

I put code in the back-end to prevent holdings counts from becoming negative
when a drop move is made from empty holdings. (This can happen when
re-loading ICS bughouse games, as the smoves command fails to fill the holdings.)

Updated compiling instructions.

Changes to allow fonts with any charset (such as terminal)

Affects winboard only.

Moved SIGWINCH signal so it can be used...

I goofed putting this in StartChildProcess() originally.

Added internal wrapping ability.

New options:
  -wrapContinuationSequence "foo"  (default: "\\   "
  -useInternalWrap (Winboard default: false, XBoard default: true)

Note: -keepLineBreaksICS if set, prevents wrapping, usurping the
   useInternalWrap setting.  The idea is if you want to keep ICS line
   breaks, then you don't want internal wrapping.  Because of this,
   the new default for keepLineBreaksICS is now false for XBoard.

fixed segfaul in convert.c used to convert pixmaps

new mousehandler to correct for premove and promotion popup

this is the new mouse handler for XBoard, which should fully implement
premove, including promotion popup, and yet suppress the promotion popup
on illegal moves that are not premoves. It is almost completely moved to
the back-end now, so that WinBoard will be able to use it as well. I just
must hook it up to winboard.c, in particular harmonize the call-backs of
the mouse driver into the front-end for grabbing and releasing pieces for
dragging, to make sure they have the same names and arguments.

updated black fairy pieces

here is a set of black fairy pieces with opaque white details, so they have
the same style as the orthodox XBoard pieces.

Adjustment to joining to work around server not always including space.

This patch implements commit 98aa02bda540f17f1f50be00e494efafe439b004
into the updated joining routine.  This issue seems not to be server
variance, but instead the server electing to omit the space between
words when it occurs at exactly your width setting (-1 actually).

This patch makes the joiner add back the space just like the referenced
commit.  Note: this is just a workaround to a server issue-- the joiner
actually joins correctly before this patch.

Fixed joiner detection, allowing it to work with timeseal

When timeseal's buffer fills in the middle of the continuation
sequence, detection fails, and the lines aren't joined.

As a result of this fix, measures intended to bypass joining are
no longer needed, so I removed them.  These utilize server vars
when available to prevent splitting.

fixed engingeoutput routine

removed outdated code regarding analysis window

moved bitmap to correct location

was misplaced in a previous commit

fixed wrong number of arguments for EngineOutputPopUp

new bitmap converter (including fill option) and new pixmaps

forgot to add these two lines to the last commit

removed AnalysisPopUp. Use EngineOutputPopUp instead

this was already in Winboard, copied it to xboard and removed unused code.

Proper board and holdings size when switching to variants gothic, capablanca, great and super within an ICS game.

cleaned up an old #ifdef in zippy

neglected this for the auto-width updating in xboard

Added server width adjustment based on client width changes

silence some compiler warnings

Maintainence to support all compilers.

Adjusted alternative joining method to obey keepLineBreaksICS

fix for bug #27668: e.p. field still not passed to engine

PositionToFEN now takes the e.p. rights from epStatus[moveNr], rather than
calculating it on the spot from moveList[moveNr-1] (which failed on the first move).

fix for bug #27666: naming of variants

Variant names "misc/xxx" are recognized as "normal", rather than "xxx"
to not get confused by the board "misc shogi" run by FICS.

changed enable menus

Put the "Open Chat Window", "Machine Both", and "Engine #N Settings" menu items
in the appropriate enable lists for graying them out wen not applicable.

Added a command-line option -keepLineBreaksICS true/false to control line joining.

Added code to prevent unnecessary width updates.

Turned off wrap when possible on ICS servers.

Turning off line wrap prevents splitting lines up using the
continuation sequence "\\   ".  This is desirable when using
timeseal because timeseal's buffer is sometimes too small,
causing it to break lines itself.  These lines can't be joined.

vsnprintf() must be _vsnprintf() for MSVC

added code to handle initial width update

Added width updates to ICS client on font and window size changes

What still needs to be done is placing an update call after login
to initialize with the correct width.  I didn't see where to put
that.

added ics_printf() and ics_update_width() and utility functions

Cleaned up ConsoleWndProc  (not complete, see below)

Using static variables this way is not appropriate.  I only addressed
hInput and hText in this commit though.  But imagine the WndProc being
used by multiple windows instead of just one, and you can see why
using static variables in this way is wrong.

Restructured URL code so it fits better with how winboard is set up.

cleanup: removed "#if 1" statements

cleanup: removed "#if 0" from source

a bit of cleanup of the source code

fix joining of lines split by ICS

This patch fixes the joining of lines that were split by an ICS, so that adding
a space at the break point is done only when there was not a space already.
(Some ICS leave a space at the end of the broken line, others do not.)
An extra space would interefere with board12 in Gothic Chess, which is so
long that the ICS breaks it, and the WB parser could not handle a double
space within the board.

fix for bug #27642: Clock jumps strangely in engine mode

This moves SwitchClocks() back to after incrementing forwardMostMove,
so that it knows again who has the move (which was broken by the "bare" patch),
and will take the appropriate tick length.

removed test for premove

this was added while looking for the premove bug, but is not needed

added URL detection and provided hotlinks in the ICS client

reverted winboard.c beofre URL commit to correct whitespace conversion

Added URL detection into the console text window for ICS.

added some comments and formated code

fix printing out help message (list of command line options) (tiny change)

The postfix increment always resulted in an attempt to print out an
option on the right side regardless of one actually existing. Use
prefix increment to prevent a crash when printing an odd number of
options.

Merge branch 'master' of git://git.sv.gnu.org/xboard

Updated navigation accelerators, fixing ICS problems.

Recent changes to the navigation accelerators were in the wrong
accelerator table.  Since the goal was to correct non-JAWS versions
back to 4.2.7 behavior, I updated it to how 4.2.7 did it, but still
retained the #ifdef JAWS conditional compilation.

On an ICS, the navigation keys took control away from the ics
client to navigate the game.  The update requires the ALT key to
be pressed, but it works identical to 4.2.7, not causing an
unexpected focus back to the board window when you're just editing
text.

fixed premove recapture problem;promotion popup appearing on obviously illegal moves; promotions in Superchess and Great Shatranj

1) the premove recapture problem
2) the promotion popup appearing on obviously illegal moves (the 1. e2a8
problem)
3) promotions in Superchess and Great Shatranj, where the piece can now be
selected from the holdings

added forceIllegalMove to xboard

bugfix: segfault when invalid option argument was given (bug #27427)

when given a wrong argument to an option (e.g. -tc 0) xboard aborts before setting up the window, but tried to write a message to the messageWidget which doesn't exist at that point.

small improvement for JAWS version

This is a patch to make JAWS read out any comments when /autoDiplaycomment=true.
As focus does not stay on the Comment window, the comment was not spoken otherwise.

updated to winboard internationalization scripts

Also a slight improvement of the internationalization scripting, to prevent it from crashing on messages containin a slash.

new forceIllegalMoves option

The -forceIllegalMoves option uses setboard or edit to force a position when an engine refuses the move leading to it by an "Illegal Move" message.
Especially for engines using the edit command this was a bit tricky, because of the a2a3 kludge to avoid using the white and black commands.

Merge branch 'master' of git://git.sv.gnu.org/xboard

removed _winmajor if not defined so that VC 2008 can compile the project

The oldDialog variable should just be set to 0 in all cases now, and
as such, legacy code supporting the old dialog ought to be removed.
The test for _winmajor is against < 4, which is always false at present,
causing oldDialog to be 0 in every conceviable circumstance.

getting ready for 4.4.1 release

Simplified future version changes.

Added a numeric version to config.h that is now used in winboard.rc
instead of being hard-coded there.  Just be sure to preserve the
proper format (4 comma-separated 16 bit integers) or else it won't
compile.

updated version number to 4.4.1.pre

should have done this straight after the last release, but forgot...

worked on premove bug

I made an attempt to make premove unsensitive to a race condition, just in case the premove problem was caused by that (which seems unlikely, but in any case  I added some deug printout to test this).

fixed loading of saved games via command line

the variables forwardMostMove and backwardMostMove where set to zero at the wrong place.

small fixes for the JAWS version

* I moved saying 'check' by the JAWS version to where castling also benefits from it.
* I made sure the check symbols are also recognized on promotion moves
* In the non-JAWS version, define the plain arrows as shortcut keys for the button bar

Fixed bug dereferencing garbage, causing crash.

Commit fa8be4a3fc5d81e9ec1f6c218fcf55c95d34fd10 included code to free the
resultDetails of the gameInfo structure.  That exact code exists a few lines
above where it was added.  None of these free() calls finish by setting the
member to NULL afterword, therefore repeated attempts to test/free() the
members result in an attempt to free() memory winboard no longer owns, and
an eventual crash.

link to problematic commit: http://git.savannah.gnu.org/cgit/xboard.git/commit/?id=fa8be4a3fc5d81e9ec1f6c218fcf55c95d34fd10

Added wchat.c to the project files we maintain.

Previous fix for VC++/strcasecmp() was wrong-- use StrCaseCmp().

Add resource ID for new Mute menu item.

I know this was probably done by HG, and simply left out of
his commit.  Anyway, the project can be built now after this change.

Use of strcasecmp() broke Visual C++.

wrong default value for engineDebugOutput

the new default is now 1

updated cmail.in to adapt the CVS->git change

don't reference $Revision:$ anymore. Instead use reference xboard-version via autoconf.

bugfix for protocol extensions: egtbpath -> egtpath

A new and very bad bug was discovered in the implementation of the protocol extensions:
XBoard sends a command "egtbpath" to the engine in stead of "egtpath" from the protocol specs.
After discussion on WB forum it was decided that egtpath was better than the egtbpath I originally
had proposed, and I changed it accordingly, but I had not realized that this string occurred TWICE
in backend.c, depending on if matched the -defaultPathEGTB of -egtFormats option.
I only changed the first occurence.

fix for edit-position "moves" being sent by XBoard to the engine

Engine did not start playing when out of GUI book on second move with white

Opening Engine #1 Settings crashed XBoard when there were no text-edit options in the dialog (to which to set keyboard focus)

added a "Mute All Sounds" item in the WinBoard Options menu, on Eric's request

Updated the RTF docs for the chat windows, engine-settings dialog, /keepAlive option and new game-list tag

Made the beep sounds for JAWS board navigation configurable through command-line options

Prepared a system to internationalize the WinBoard menus

added the result comment to the game-list tags when the game list is exported to the clipboard

rewrote wevalgraph for better frontend/backend separation

I hve finished re-writing the eval-graph code to have some front-end / backe-end separation,
something I had already started on my holiday in Norway (when I had no internet :-)  ).

The new code now appears to work exactly like the old code, and I guess it could be ported
to gtk comparatively easily.

fix for keepalive and chat window

I made a fix to the keepAlive option so it does not send the date command
to the ICS after every move. This required changing ScheduleDelayedEvent()
so that it does not execute the event if a new identical one gets scheduled.
I also added the option to XBoard. Affects
xboard.c (DOS!) and winboard.c

(This also cures the regression of the winboard.c copyright patch)

I extended the chatbox patch to also listen to channel tells, and divert
them to a chatbox for that channel number. Affects
backend.c and wchat.c

That should make those patches fully operational. No docs for either of them
yet, though.

fixed parse bug for pgn files

fixed the existing bug that the time-control tag was displayed as "?" in the game list; turns out it was never parsed on reading PGN files

allow the result comment as a display item in the game list

fixed the HAVE_LIBXPM-dependent compile errors

added a chat window to keep track of multiple conversations

The chat-window patch affects:
winboard.c, backend.c, winboard.rc, resource.h, backend.h, and adds
the file winboard/wchat.c, which also affects makefile.gcc and makefile.ms.

In addition I added a new option -keepAlive, which affects
common.h (in addition to winboard.c and backend.c).

As I had to modify backend.h, I also defined the -reset option type there,
which we are going to add later. This required some changes
to suppress warnings in wsettings.c.

more files for git to ignore

most of these are generated by make distcheck

another bug fix found by Stanislav Brabec

returned -1  in some case although the variable was a char...  changed to signed char