Bugfix stale first-click

When the opponent resigned after the user had cicked a from-square,
clicking the from-square for the first move in the next game would
produce an "illegal move" message, because it was considered a to-click!
We now set fromX and fromY to invalid in GameEnds().
Should we also reset them on mode changes?

Match handles with multiple titles for channel Chat Boxes

Only handles with at most one suffixed 'title', such as (U), (C), (IM)
or (TD), followed by the parenthesized channel number, were recognized,
so tells by people with more titles always went to the ICS console window.
Patterns are added to recognize handles with upto three such qualifiers.

Change opening array -variant fairy

We now use the position of "Chess with Different Armies", FIDE army vs.
a fantasy army (inspired on the Color-bound Clobberers) as opening
position.

Add variant Makruk

Thai Chess, played by 2 million world-wide. Required pawn promotion to
happen in a zone, and the Alfil to move as Shogi Silver, a new opening
array, and unusual Pawn placement. Because of the latter, I consider it
a setup position, so a FEN will always be sent to the engine.

DOS line endings and 644 mode for woptions.c

Remove race condition in clock switching

The move number is now incremented inside SwitchClock, when no timer
interrupt is scheduled, so the interrupt cannot occur at the wrong side
of the increment, and decrement the wrong clock.

Extensive bugfix of -autoKibitz

Suppression of printing of diverted kibitz messages failed when they
extended over more than one read. This could even lead to such
messages being erroneously saved as comments, which then lead to
crashes. The suppression mechanism now waits for a prompt, in stead of
the end of buffer. This required the printing of any leftover in the
buffer to be deferred until the next read completes it.
The number of people kibitzed to is now printed (with the "players)"
stripped off on FICS, but leaving a separating space).

Fix highlighting bug in XBoard

The from-square was drawn before the to-square was erased, which led to
a missing highlight line if they shared an edge or corner.

Send continuation lines to chat box they belong to

The variable savingComment is used as a kludge to remember where the
comment should go (i.e. which chatbox, or elsewhere), so that when a
continuation line is matched, chatPartner (which was reset at EOL) can
be restored to its original value. Also put space after handle of talker
in channel Chat Windows

update year in copyright info

Fix u64 format for cygwin

With a -mno-cygwin compile %llu and %lld are definitely not understood
by scanf and printf, although they are by the compiler. In stead, %I64u
seems to work. Put it in under control of the switch __MINGW__ .

Remove stray dot from WinBoard makefiles

There was a period suffixed to config.h in the wgamelist.o dependency.
Also wedittags.h was not listed in the makefile dependencies.

Fix bug in display of logos

The logo continued to be drawn after a boardSize change to a tinyLayout
size, although there was no space reserved for it. Fixed by explicitly
resetting logoHeight when no logo space is reserved in InitDrawingSizes().

Fix of ancient WinBoard bug: check value of int options

Only strings of digits are accepted now.

Use -keepAlive option to determine connection health

When there has been no ICS response since the previous 'ping', assume we
are disconnected and do a fatal-error exit. Also auto-start the ping
process in InitBackend3().

Accept <Enter> for changing chat partner

A kludge stolen from Alessandro's GameList filter allowed to let the
typing of <Enter> while updating the chat-partner field of the WinBoard
chat windows to act as pressing the Change button. Gives focus to the
input field of the window afterwards.

new developer release

Fix OO-castling in FRC after pasting FEN

Because the FEN parser did not set initialRights, the rights from the
previous game kept hanging there, and prevented recognition of
OO-castling (although KxR castling was still understood).

Fix reading castling rights FEN in knightmate and twokings

Scanning for a King on the back-rank did not work in these variants,
because either the Unicorn castles, or there might be two Kings there.

Fix TwoKings ICS castling-rights bug

Scanning the rank (as is done because of FRC) assigned castling rights
to the rightmost King. Even scanning the opposite direction would not
work, as it is not always the leftmost King that can castle. So we now
explicitly test for a King on e1/e8 in TwoKings and assign it rights,
relying on the Rook rights received from the ICS to control castling.

Fix piece-to-char table -variant fairy

The letters for Lance and Snake where swapped.

Make WinBoard makefiles use parser.c in XBoard directory

Fix bug for incommensurate time odds

After normalization (-timeOddsMode 1) the time odds factors can be
non-integer, and thus must be stored as float.

Bugfix legality null move in parsing with -testLegality off

The pointers were compared, in stead of the values.
Also in stead of rejecting null moves with legality testing off, it rejected moves on the main diagonal.

Chmod 644

Implement castling for variant CRC

added missing sounds files to be able to compile on windows

updated to unstable version number

release of version 4.4.2

Signed-off-by: Arun Persaud <arun@nubati.net>

add DIFFSTAT and SHORTLOG to tar-ball

remove trailing \r in xboard output

there is an annoying carriage return (displayed as ^M) at the end of the PV in the engine-output window, not
only with Fairy-Max, but also with UCI engine running under Polyglot. This patch replaces the trailing CR that Linux programs send after the PV line y a space.

double buffer size to prevent overflow

This patch doubles the size of the ICS input buffer to 16KB, to prevent
the suspected overflow with -keepLineBreaksICS false when using
"inchannel 1" on FICS and over 500 people are tuned in.

fix problem with empty string in -firstNeedsNoncompliantFEN

This fixes a bug in the use of the -firstNeedsNoncompliantFEN option with
an empty string as argument. (Which led to truncation of the FEN.)

updates makefiles to include dependency on config.h

adds the dependency of wbres.o on config.c introduced by having the VersionInfo in the resource

correctly apply some check boxes from the option menu

Usng the check-boxes in the Option->Adjudications menu to indicate the
engine reports absolute scores is now copied directly to the ChessProgramState,
from where it is used, in stead of only to appData. This makes the change take
effect immediately, in stead of after restarting.

score sign in analysis mode

this does not alter any of the current behavior with compliant engines,
but does extend the correction for non-compliant score reporting under
control of the -first(second)ScoreIsAbs option to cases where the engine
is analyzing or playing on an ICS.

fix for bug #28077: xboard needs to link against x11

hopefully fixed automake to take care of this

Fix castling rights when copying FEN to clipboard (again)

missed a spot last time

fixed some outstanding pixmaps

fixed transparency issues

new developer release

fix the irritating wandering off of the MoveHistory window in XBoard on opening/closing.

use xtell for talking to handles, but tell for talking into a channel.

by changing the chatboxes to use xtell in stead of tell, we broke
the possibility to use the chat-box for chatting into a channel (which is
apparently not possible with xtell).

fix bug in bughouse drop menu

This patch fixes the bughouse drop menu, which was broken by the
new mouse driver, and could no longer drop white Pawns.

removing some rather verbose debug messages that seem no longer needed.

fix engine-sync on move-number type-in bug

fix declaration of engineOutputDialogUp

was defined as BOOLEAN (=char), but should be int

fix double start of zippy engine after switch to gothic

In ICS mode we now switch to the proper variant as soon as we receive
the generic game-start message, to prevent a mismatch when the first
board is received (which would trigger fetching of a move list).

the last move before the time control now gets its time listed in the PGN

fix casting rights after FEN pasting

fix crash on engine crash

fixed build on openbsd

added missing header file

cleaned up some debug messages and typos

removed files that should only be in the windboard directory

add Winboard source files into tar-ball

forgot to add them when rewriting the Makefile.am

added missing library for build on OS X

Xmu seems to be missing

updated version number to unstable

release of version 4.4.1

cleaned up ChangeLog and NEWS

use xtell instead of tell in wchat

updated files for a new release, added a shortlog and a diffstat

prevent buffer overflow

This is a buffer-size adjustments to prevent the JAWS version from
crashing when it has to read out loud very long comments.

Further copy/paste fixes

We needed to implement the XA_TARGETS target in order for more
sophisticated apps to really believe we put anything on the clipboard
they can use.

Make copy/paste position and game use clipboard, bug #27810

Copy actually sets both the clipboard and the selection for
convenience and compatibility with the old way of doing things.  Paste
pastes from the clipboard by default, but the new -pasteSelection
option lets you get back the old behavior of pasting from the
selection.

switch focus to the board after loading a game

fix for bug #27826: fixed autoDisplayComment

apparently the -autoDisplayComment option was not working,
not even in WinBoard. This patch should fix it,both for WB and XB.
(Unfortunately xboard.c and winboard.c were using a different name
for the same front-end variable I needed, so I had to change one of them.)

fix for bug #27826: ported two options to xboard

also updated the documentation

fixed jaws version

turns out the new mouse handler broke the JAWS patch, as I discarded
the function IsPromotion. I adapted it to call HasPromotionChoice instead.
Also fixed two arnings by removing a no-longer needed debug print.

reformated html to be correctly validated

quick fix for "White Mates" in parser.l

fixed a regression

changed stderr to debug output, since stderr is closed in winboard

removing empty lines from ICS play

this patch causes removal of spurious prompts during ICS play also if the prompt is preceeded by a number of empty lines (as the board for the opponent move is on FICS).

another bug in VriantSwitch: an unitialized board was printed.

Must have been there a long time, but this time it crashed WinBoard.

more work on variant switch

the previous solution to the cosmetic problem of displaying a wrong shuffle
in an ICS game that was observed from the very start unfortunately turned
out to cause a real error in diplaying a stored game through the smoves
command, for variants not using the FIDE setup as standard.

This patch displays the position from before setting up the start position
of the new variant, but does setup boards[0] to the initial position of that
variant. (If this is a wrong shuffle, it is thus ot displayed, and a new board
will immediately be sent from the ICS to correct the shuffle.)

force full redraw in winboard

Full redraw is now always forced when highlightMovesWithArrow is on,
rather than only when highlight info is available. (As in drop moves the
latter is invalid, but we must still erase the arrow from the previous move.)

fix for bug #27790 and 277772.

Suppressing the display of a wrong shuffle when observing shuffle games from the first move(#27790 bis), and the sounding of the FICS bell on crazyhouse captures (#27772 bis).

fixed bug related to unsigned char in convert.c

I also fixed the unsigned-char bug in the convert.c for bitmaps that caused the
129x129 XBoard bitmaps to be defective, and a bug that required fixup with sed
afterwards (so I could use it to create a new m33s.bm)

unguarded debug printf. added the appropiate if statement

fix for two compiler warnings

fix for bug #27799: fix for nested-nested-nested variations

The book-probe code forgot to close the book file after opening it.

fixed by adding an fclose at two points.

removed unused v54?.bm bitmaps from resource file

it seems the v54?.bmp bitmaps were actually included in the WinBoard build
(in the resource file), although they were never actually used. So I have
removed their inclusion in the resource file now.

replaced defective bitmaps with copies from 4.2.7

removed bitmaps files that are not needed any more

fixed some bitmaps

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

Drop an obsolete script that was only used to update my personal web site

Silence warnings when compiling 64-bit xboard

Some harmless warnings were caused by 64-bit mode having 32-bit
int but 64-bit long and pointer.  Add casts (etc.) to silence them.
This might require adding a definition of intptr_t (a signed int
type that is the same width as a pointer) on old systems that don't
have it.

Fix up man page some more

Sigh, the project history was still partly in reverse chronological
order and partly in forward order.  Swapped the order of some
sentences to help straighten it out.  I don't really care about this,
so other project members should feel free to do what they like here,
but the messed-up order was bugging me.

Fix some issues in the XBoard man page

Correct places where the XBoard man page was calling XBoard
"WinBoard".  Change references to "WinBoard engines" and "WinBoard
protocol" to "XBoard/WinBoard ...".  Update bug reporting guidelines.
Reorganize project history to something more like chronological order
-- it had gotten pretty randomly ordered.

new pre-release version; updated version numbers

This patch adds <Enter> to the characters that cause an automatic switch to the ICS console when typed to the board window.

fix to the minor graphics issue contained some typos, as was remarked in the bugs reports

improved mouse handler

The improved mouse handler still needed an extra call to DrawPosition to
work in XBoard.

NPS plays and pondering

Clock updating did not work correctly in node-based time controls when the
engine was pondering. (It was not really meant for use with pondering. Oh well...)
This patch checks if the engine reporting the node count is indeed thinking
(as opposed to pondering).

holdings update and regression fix

The holdings-update patch was not yet satisfactory, because FICS seems to send
holdings both before and after the board! And those received before did overwrite
the holdings received after the previous board. I know tried to fix that in a kludgey
way, by hiding a flag in the board that tells if the holdings have already been written
by the ICS or not, and ignore new holdings when they are. (Except in bughouse,
where holdings updates cannot be tied to a particular board and multiple updates
between boards are normal. (I hope?).)

I also undid a regression in my previous fix for the minor graphics problem,
and solved it in a slightly different way to prevent flicker.

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].