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

Swapped 'tell' and 'message' parsing order for colorization.  Sometimes
messages are relayed as tells and should be colorized as tells.

added some more files to be distributed via make dist

as suggested by Stanislav Brabec

fixed some implicit declarations reported by Stanislav Brabec

fixed a few more small bugs reported by Stanislav Brabec

- A function uses a 'return;' statement, but has actually a value to
return, like an integer ('return 42;') or similar: xboard voidreturn
xengineoutput.c:766, 782

- Program returns random data in a function at xengineoutput.c:602

- With a new toolchain, AM_LDFLAGS cannot contain libraries. Linker
ignores them. Fix is attached.

- ToDo: Convenient name in most projects is TODO (autotools package it
automatically).

reverted .texi file and fixed Makefile.am

the problem was actually not in the texi-file, but the Makefile.am. Reverted changes in the .texi and fixed it (hopefully) correctly now.

get "make distcheck" to work

needed to add all *.h files to Makefile.am and also the bitmaps that are needed and some other files. Also had modified copyright.texi.

changed to version 4.4.0

updated AUTHORS, README and ChangeLog files for release

fixed some typos

set castling rights after setting up a position

I was going through old WB forum posts, and I encountered a
bug report for a bug that I never fixed: The castling rights should
be properly set (actually: cleared) after setting up a position
in Edit Position mode (as opposed to pasting a FEN).

removed install files from master

they now live in their own branch

updated polyglot protocol

I added a new option type "-reset" to the protocol for the new option feature.
4.4.0 does not implement it yet, but Michel wants it for future Polyglots,
and I think it is best to update the protocol in one big swoop. So I want to already
include it in the specs now. (engine-intf.html)

I made xboard's response to an unknown option type compliant with the new protocol specs.

fixed some typos

Updated project files for DevCpp and Visual Studio to add wsettings.c

Added newline at end of file to silence warning.

workaround for error message for missing pieces used in variants and initstring bugfix

error messages showed for non-existing pieces showed up for old user supplied pixmaps. Loading a pixmap of the king for the missing pieces now.

Another bad bug surfaced: the -initString argument does not work in any
XBoard version! (Including 4.2.7b) The escape sequences for linefeed in
the string are not understood: -initString "new\nrandom\n" actually sends
the \ and n to the engine (which then does not recognize the line as a
valid command)!

So I also added a patch to expand escape sequences in the InitString and
ComputerString command-line options.

updated installe script

I re-organized the doc files in the install, so they now all go in the
WinBoard\doc folder
(and none in the root). I so included the COPYING and COPYRIGHT files there.
This required some updates of the links in some html files, and of the menu
link
to README.html in the install script. I also fixed an error with the
current directory
in the menu short-cuts of some engines (which started in the fonts folder, and
could not find the opening book there...).

updated documentation

updated installer script

new help files for windows

updated installer

undid some previous changes

some files got mixed up

fixed small bug in wsetting

trigger a focus switch to ICS input field when typing

old behavior was that "." or "," triggered this, now every printable character  except 1-9 will trigger the focus switch

bug in backend.c when receiving lines from ICS

A space was appended to every character received from an ICS in stead of just at line breaks.

fixed warning messages from compiler

fixed build on OS X

this seems to fix it. Bug reported by Louis Zulli who also did the testing.

OK, this is the beefed up version of the Polyglot info-string kludge, for WB and XB.

added missing X_LIBS references to Makefile.am

made the install script and executable

changed beta1 to beta2

fixed enable zippy in configure script and added some checks for OS X

make tried to compile zippy even when --disable-zippy was given. Also fixed a bug in xboard.c where #ifdef instead of #if was used to test for ZIPPY (it's either 0 or 1, so we need #if).

Also added some special checks for OS X, not sure if these will do it, but it's a start;)

added configure summary to configure output, also cleaned up configure.ac a bit

used AS_HELP_STRING where possible

added check for malloc.h

this should fix a build error on OS X

A small cosmetic change in the size of buttons inside a groupbox of the Engine-Setup dialog.

make xboard usable with UCI engines

I made a small patch in xoptions.c (where the corresponding Engine Settings
dialog was for XBoard), to make it at least usable for UCI engines: I now
divide the options over upto 4 columns, if there are too many to fit into
a single column. This seems to work. Still looks very ugly, but it is
functional.

fix premove for winboard

This might fix the illusive premove problem. In 4.2.7 the code was not properly
indented, and this led to an unbalanced braces error that I then fixed the
wrong way.
Click-click premoves did only work after clicking the toSquare twice. It
might be
that they actually did work after clicking it once, but that the highlights
were not
displayed (although set) for lack of a display refresh.

fixed bug for double saved games

This patch should solve the problem with doubly saved games when XBoard is
closed.
I calculate a cecksum of each saved game now (including move comments),
and suppress saving if the checksum is not changed.

fixed xengineoutput.c, to keep the duplicated code identical to that of wengineo.c.

updated xboard.texi: added warning that pixmaps are not build for all variants

added a dialog for engine-specific option settings

updated some documentation

fixed several bugs in the backend and added some more polyglot support

* I added recognition for the new UCI3 option types that are recognized in the WB
option features, to make sure this version is compatible with future Polyglots.
Also fixed a bug that failed to strip an asterisk prefix from the first
combo-box item.

* I fixed the ICS-time-in-PGN bug.

* I fixed a bug which caused a space to be missing when long lines broken up by
an ICS were joined.

better polyglot support for engine ouput

I fixed the algorithm for clearing the Engine Output window to work better with
the latest Polyglot, whight outputs UCI "info strings" amongst the thinking
output.

Beginnings of installer for 4.4.0.  HGM asked me to forward the nsi file,
but it seemed best to just add the installer tree to git even though it's
not finished.  You must install NSIS then FontName-0.7.exe (included in
installer folder) then run build.bat in the same directory.

patch backend.c to make the -noGUI option also work in ICS mode.

temporary fix for pre-select

As a temporary kludge I replaced looking into the holdings for the piece type by figuring out which piece would belong on the clicked holdings square. This is a non-trivial process, as it depends on which pieces participate in a certain variant.

changed click-click moves

I got complaints about the way XBoard 4.4.0 handles click-click moves, in particular changing the selected piece after selecting King (for which I suppressed interference with FRC castling in a rather brutal way). People tell me it is very annoying that clicks sometimes don't work to select a piece. WinBoard uses refined logic in the back-end to handle this, but the XBoard front-end is not interfaced with that yet. (This would require a major rewrite of the mouse handler.)

This temporary patch suppresses this behavior in all variants that do not support Fischer castling, and even there only exhibit it for Rook-after-King selection (which will be interpreted as an attempt to enter a castling move as K x own R).

option -adjudicateDrawMoves is not included in the XBoard list.

added it to the option list

converted files from dos to unix format

run dos2unix over a bunch of files

Stripped out unused VersionInfo