Let second click on piece make only capture, with -oneClickMove

Normally this would clear the highlighting of that piece.
Required new field in DisambiguateClosure to count nr of captures.

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?

Add promotions and e.p. to oneClickMove

For e.p. the to-Square of the Pawn has to be clicked. Promotions are
always to Queen when you click the to-square, but use the promotion
pop-up when you click the from-square.

Fix error that compiler does not notice

Boolean GLT_GetFromList did not return a value. But no error on Ubuntu!

new developer release

updated year in copyright info

One-click moves

Playing on ICS or against engine, clicking a square from or to which
only a single move can be made causes the move to be done, when the new
option -oneClickMove is set to true.

Use squares for computer seek ads

Shape indication passed in sign bit of color (char) variable.

Use right mouse button to view seek ads

Right down-click does the same as left down-click, right up-click
simulates a left up-click off board (so it misses all dots).

Use right-click miss to refresh Seek Graph

The 'moving' parameter had to be made into an int for this, to indicate
right clicks with code 2. Right press now already pushes the clicked ad
to the back, so that the up-click will display an underlying one.

Bugfix pushing back seek ads on right-click

On a right button press the previous in stead of currently hit ad was
receiving the penalty.

Indicate seek-ad multiplicity

If more than one seek ad is inside the capture range of the mouse
pointer, an exclamation point is printed before the seek-ad text.

Some fixes to make seek-ad suppression work in WinBoard

In WinBoard the z-depth was immediately erased by redrawing the graph;
the clearing of zlist had to be moved from PlotSeekAd() to AddAd()
(where it really belongs). Also aging of z-depth is now suppressed on
mere pointer motion.

Dynamic Seek Graph

The new option -autoRefresh sets FICS and ICC to report removal of seek
ads, and then removes those from the Seek Graph. New seek ads are added
to the graph, as soon as the ICS reports them.

Implement SeekGraph in XBoard

For now only controlled by left mouse clicks on board. New options
-seekGraph true|false and -sg to switch this feature on.

Facilitate selection of coinciding seek ads

Seek-ads that are viewed but not answered get a distance penalty on the
up-click that makes their selection more difficult next time, so that
the ad they covered gets a (better) change. The penalty ages away in the
course of 5 clicks.

Slightly offset Seek-Graph dots of different color

Dots of different color are now displaced horizontally be 3 pixels, to
prevent them from completely covering each other. (Required the axis
labels to be moved a little too, to not be covered by the 1-min ads.)

Auto-show seek-ad texts in WinBoard

Hovering the mouse above a Seek-Graph dot will make the text belonging
to it appear in the message field without the need to press a button.
Required SeekGraphClick() to be called from the WB mouse event handler,
with an extra argument 'moving' to make the distinction with a real
click.

Show seek-ad text on mouse hovering (XBoard)

The Eventproc is also set to trigger on PointerMotion, and this event
then calls SeekGraphClick with motion=true to handle it.

Put poetic name of wilds in seek-ad text

Use XBoard variant names, except for 'loadable' and 'fischerandom'
(becomes 'setup' and 'chess960'), but leave number (to distinguish wilds
that transate to same XBoard variant).

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.

Shorten autoKibitz confirmation on FICS

Strip off the "players)", but leave the separatng space.

Repair damage to -autoKibitz done by FICS atomic zippy claim

The sending of -autoKibitz info was also moved back in that patch, but
unfortunately to after clearing of the programStats info. Now the
clearing has been moved also, to remain after it.

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

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 a switch USE_I64, which is
defined only in the gcc makefile when MINGW is used.

Remove stray dot from WinBoard makefiles

There was a period suffixed to config.h in the wgamelist.o dependency.

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().

Right-click refactoring: step III

Let WinBoard call RightClick in back-end, and delete the duplicate code
it had for this, after rescuing the front-end bits, and let the letter
act depending on the result returned by RightClick.

Right-click refactoring: step II

Migrate the Right-click routine to the back-end. Let it communicate back
square coordinates through pointer arguments, to accomodate their
different naming in XBoard and WinBoard. Template added in frontend.h.

Right-click refactoring, step I

Divorce front-end part from back-end part of PieceMenuPopup in XBoard
front-end, and forge the back-end part into a new routine RightClick.
The right up-click in XBoard, which called unLoadPV() directly, now
calls RightClick through PieceMenuPopUp().

Merge gamelistopt sources into gamelist source files

The files xgamelistopt.c and gamelistopt.c are merged with xgamelist.c
and gamelist.c, respectively. The WB front-end for the dialog already
resided in winboard.c, so there was little need to move it to wgameist.c
file. Makefiles adapted accordingly. Unsuitable key bindings were
removed. The low-level front-end prototypes are now defined in in
frontend.h.

Game-List options dialog for XBoard

A dialog similar to that of WinBoard was cloned from the game-list
window. The XBoard front-end is in a separate file xgamelistopt.c,
which in fact implements most of the low-level front-end as back-end.

Refactoring of game-list-options dialog

The back-end part is separated off, and moved to a file gamelistopt.c.
Front-end for now stays in winboard.c. No prototypes in frontend.h yet.

Fix of ancient WB bug: test integer option values for validity

Only a string of digits preceded by an optional sign is accepted now.
Before, any string was accepted, and leaving out a value could lead to
the next option being ignored (if it did not need a value) because it
was considered the value for the previous one!

Fix bug in edit-position of holdings

The holdings were effectivelly taken one too large, so that a promoted
shogi Pawn moved to, or created in the holdings was not demoted to Pawn,
but put in the dark (which you could see by the holding count if you did
it twice). The overflow tests in these two cases now have been corrected.

Save fonts in XBoard settings file per boardSize

Each -boardSize now can have its own font specified in the settings file
(and thus by a command-line option, although it would in general not be
very useful to specify a font for a size other than the selected). E.g.
a prefix "size33:" to the font name will apply it only to -size 33.
An unspecified-size font option (not normally in the settings file) will
always prevail over a font option specific for the current size.
A font of some size must have been used or specified in order to be
saved.

Fix width of filter field in XBoard GameList

An empirical dependence of the width of this text edit on the square
size now makes for an acceptable layout at any board size.

Use FICS atomic draw claim for sending move in zippy mode

For this the relaying of the move had to be moved until after the
adjudication, which again has to be done after MakeMove(), which is a
bit dubious, as MakeMove might take significant time when animating.

Send FICS atomic claim to ICS if move creates draw after offer

The FICS format "draw MOVE" is used to send the move to the ICS  when
the user enters a move that adjudicates as a claimable draw, and he
offered a draw before the move. For the benefit of ICS that do not
understand this format, the plain move is sent afterwards.

Also allow user to claim by offering draw before his move

Use the existing userOfferedDraw variable to test for such offers in
Adjudicate().

Also adjudicate after user move

Call Adjudicate() in FinishMove().

Refactoring of adjudication code

Put all the rule-based (as opposed to score-based) adjudications in a
separate routine, so they can also be called on user moves. This
requires sending the drawing move to the opponent engine to be made
conditional on existence of the latter. Execution of the code is no
longer dependent on the condition TwoMachinesPlay, but actual game
terminations are now all restricted by an alternative condition (no ICS
mode).

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().

Use arrow keys in XBoard GameList for entry selection

Add translations for arrows and home/end, to call LoadSelectedProc with
argument to indicate the step, and let the latter perform the action.

Allow <Enter> to apply filter in XB GameList filter edit

Apply a translation to intercept the <Enter> key. Set focus back to the
list display afterwards.

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.

Move duplicat gamelist code to backend

The filtering routines are moved from xgamelist.c and wgamelist.c to
gamelist.c, and a prototype for them is placed in backend.h.

Port game-list filtering to XBoard

Added a text edit and apply button to the XBoard game-list dialog, and
only have it print those lines that match the filter. Translate index in
list to index of game in file when a game is selected, and index in file
to index in list when a line is highlighted. Duplicates some code
between xgamelist.c and wgamelist.c, which should be moved to back-end.

Convert to DOS line endings

For easier understanding of upcoming diffs of winboard.c

Refactoring of move-history code

The WinBoard code in whistory.c was split in a front-end part (remaining
in whistory.c) and a back-end part (now in history.c in the xboard
directory). This to allow easy porting to other platforms.

Add fixed-time/move button in XBoard time-control dialog

Put the TC-type buttons in a radio-group, and highlight the selected mode.

Fix bug in XBoard PV display

The PV was unloaded on anything but a button release, in stead of the
other way around.

fixed hardcoded location of config file for xboard

the location should now follow sysconfdir during the configuration process.
Also added some more output to the configure script.

new developer release

fixed Makefile.am to handle config file correctly

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.

Fix parsing of O-O and O-O-O after FEN pasting in FRC

Because the initial rights were not set in the FEN reader, the
initial rights of the previous game remained in force, so that the
parser did not understand OO-castling (although it would allow KxR
castling).

Fix reading FEN castling rights for knightmate and twokings

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

Fix omission in castling refactoring for ICS received boards

A default of -1 was still used, in stead of NoRights, when no Rook was
found. In theory this could have had effect in a setup board 0 in FRC,
when there were no Rooks on the back rank.

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 eftmost 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 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).

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.

Fix piece-to-char table -variant fairy

The letters for Lance and Snake where swapped.

Make WinBoard makefiles use parser.c in XBoard directory

Docs update for new features

Fix new args parsing of -lowTimeWarningColor

This did not occur in the XBoard color translation table yet.

Allow editing of holdings in EditPosition mode

Dragging pieces to and from the holdings now consistently keep track of
the home square of pieces and the piece counts, as does creation of
pieces in the holdings through the piece menu.

Indicate squares a lifted piece can legally move to

Under control of the new option -showTargetSquares when
-highlightDragging and -testLegality are on, the squares a piece can
move to are marked by fat dots in the highlightColor (non-captures) and
premoveHighlightColor (captures), as soon as you grab it for dragging.
In variants with mandatory capture, the capture target of other pieces
is marked with the highlightColor (as non-captures are then not allowed).

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.

Display PV right-clicked from EngineOutput window

The clicked PV is selected and the memo gets focus, so that it is
highlighted. This selection is restored in the proper place after
insetrtion of a new line in the memo.

Display PV on right-clicking board

Bugfix legality null move in parsing with -testLegality off

The pointers were compared, in stead of the values.
Furthermore, instead of null moves, it rejected moves on the main diagonal when legality checking was off.

Also exclude moves with empty squares and enemy pieces.

Newly lexed winboard/parser.c

Harmonize declarations of XBoard and WinBoard

The engine-exit and settings patches make some stuff common to WinBoard
and XBoard, which needed to be harmonized, and is moved to frontend.h.

Allow Ctrl-C copying from EngineOutput window text to clipboard

Allow popup-less (fatal) exit of engine after tellusererror

Improve layout of Engine #N Settings dialog

An attempt is made to align the input fields of spins and text-edits

Let XBoard print version with argument --version or -v

Fix castling rule assignment in shuffle games

The conversion to in-board storage for rights copied directly to boards[0] in steadof
initialPosition, and was then overwritten when the latter was copied with uninitialized
rights to boards[0]. This affected FRC and CRC, where rights are non-standard.

Implement castling in -variant caparandom

CRC should be treated as FRC, but until now it never was. We now test for FRC_TYPE_CASTLING.

Fix default castling rights on reading incomplete FEN

This was overlooked in the conversion to the in-board storage of rights.

Fix NoncompliantFEN defaults.

This must be NULL to get compliant FENs, not "", which would suppress castling and e.p. fields.

Change format of -adapterCommand.

first or f at the start of an option name are now automatically changed into second or s,
rather than needing %% to be recognized. The values of -ec and -ed in the default
-adapterCommand are now quoted (now XBoard understands quoting in the engine command line).

Make WinBoard defaults for -fd and -sd equal to "." rather than empty strings.

Delete old indirection settings-file code

Correct XBoard default debug file name

Suppress saving font settings.

The fonts are size dependent, and might thus not be reusable in the next
session. For now we therefore suppress the saving of them in the
settings file, until we diversify the fonts per boardSize.

added missing sounds files to be able to compile on windows

Changes needed to compile master branch.

implements the eval-graph window for XBoard

improves the XBoard handling of the engine command line

it now understands limited forms of quoting, so engine binaries with spaces in their file name can be handled.

A better and more flexible way of invoking Polyglot

rewrote uci.c. No more use of temporary files, all data passed through the engine command line. Adds a new command-line option -adapterCommand for this.

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

restoring windows (EngineOutput and MoveHistory) on startup

restoring the open / closed status of the
auxiliary windows (EngineOutput and MoveHistory) on startup
in XBoard like they do in WinBoard, which makes sense now that
we aso remember their coordinates between sessions.

fixed some outstanding pixmaps

fixed transparency issues

use linux style options for config file and add a system config file for XBoard

This patch makes XBoard write the options to its conf file in
Linux style, "-option value" rather than Windows style "/option=value".

The WinBoard option parsing code has always understood both formats
(including any hybrids), but the format it preferred for writing might
look quite strange to Linux users. I also attached a file to be installed
as /etc/xboard/xboard.conf on "make install".

fix window positioning

This patch adds stuff in the header files that allow the various front-end files
to share information on which windows exist, are up, and how they are positioned.
The EngineOutput, MoveHistory and GameList windows now all remember their
position and size, even when they are closed when XBoard exits.

same argDescriptor parsing for Xboard and Winbaord

This patch makes XBoard and WinBoard use the same argDescriptor
list of options, and (largely) use the same code for parsing and saving them.
This is acheived by moving this code to a new file args.h, and #including this
in both winboard.c and xboard.c (so that tiny differences, e.g. in default
values in the table, are decided by the context). XBoard has new menu items
"Save Settings Now" and "Save Settings on Exit" in the "Option" menu.
XBoard uses /etc/xboard/xboard.conf as default settings file.
A new option, -saveSettingsFile, can specify where settings should be saved,
without parsing the file. (So that it also works if the file does not yet exist.)

new developer release

changed release name to <branch>-<date>

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