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

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.