Debug position search cache

profile

Translate search-mode strings

Quickscan

Fix duplicate loading of second engine in tourneys

To decide if new engines should be loaded, it was tested if first.pr
equalled NoProc, assuming that ReserveGame would have left the engine
loaded if the next game would need the same engines. But a reuse=0 engine
could have caused termination of the first engine process anyway, while
the second would still be alive. So unconditionally starting of the second
engine would cause redundant engine processes to abound.
  Now we only start new engine processes if both engine processes are dead.

Fix PV walking with -fSAN

The SAN options had broken the PV walking, because the shelving of the
walked PV to create space for the conversion to SAN of an incoming PV
would not be properly undone, but would add the walked PV to the game.

Refine clock adjusting

Adjusting the clocks by shift-clicking them is now only allowed with
autoCallFlag off, and an error-popup results when you do it otherwise.
Also, swtching to Edit Game mode won't reset the clocks to the stored
value anymore when you have just adjusted it.

Fix crash on adding items near book end

When Edit Book would add so many entries near the end of an opening
book that the end of the added info would surpass the old end of the
file, the copying of the tail got stuck in an infinite loop, extending
the file without limit with repeating info.

Print message on wrong use of Clone Tourney

The user is urged to first provide the name of an existing tourney file
when he presses the button without one. The button is also renamed.

Fix invalid combobox entry WB

A combobox label that was not in the list (because the user typed
it, rather than selecting one) would cause a crash.

Fix crash on loading garbage game file

XBoard crashed when a game file contained a PGN game result before
any valid game start, because data would be written in a game-list
node that did not exist yet. This would occur frequently when garage
was read; basically any asterisk in the data would do it.

Add control to set -discourageOwnBooks WB

only require 0.17 of gettext

This seems to solve many build issues with older distrubtions and doesn't seem to be a problem with newer ones.

Add option -discourageOwnBooks

This boolean option determines the default setting of the hasOwnBook
option before a tourney game starts. By setting it, the hasOwnBook
option is cleared, which would enable use of the GUI book, like the
engine was installedwith the /firstXBook option. An explicit option
on the engine install line can still overrule this, though.
 A control to set this option has been added in the XBoard Match dialog,
and it is saved in the tourney file to force consistency.

Reset initString and computerString on engine load

These options are typically engine specific, and must thus be
reset to their default value before loading a new engine.

Add Romanian lng file

Reactivate Falcon Chess

Now the patent for this has expired, Fairy-Max plays it again.
So the XBoard support for it is now switched on, and an item for it is
added to the XBoard New Variant menu dialog. (WinBoard still had that.)
In the array the Lance was changed for the Falcon as a wild-card piece.

Update texi file for new features

Update texi file for position search

The new items in the Load Game Options dialog are described, as well
as the command-line options used to store them in the settings file.

Remove paragraph about .Xresources from texi file

Command-line options can no longer be set through .Xresources already
since version 4.5.0.

Add Dutch WinBoard translation

Prevent Edit Book window from stealing focus (WB)

In WinBoard updating the book window would give it focus. Now the
focus is given back to the main window, except when the Edit Book
window is first created.

Make book-window update part of HistorySet

Fix position loading in tourneys

In a tourney a position must be loaded when the engine is not yet
started, and the old version of LoadPosition could not handle that:
It would start up the engine, and send it commands unconditionally.

Fix resetting -fSAN on engine change

There were two places for loading another engine, Load (from menu) and
SetPlayer (in tourneys), and the previous patch for this missed the
second one.

Make too-long game non-fatal

GameEnds is now called when the maximum number of moves is exceeded,
rather than DisplayFatalError. Not sure if this properly terminates the
game in all situations, but DisplayFatalError (after calling GameEnds)
would also wait for the user to confirm the error popup before really
exiting, so that in an unattended automatically running tourney there
was not much fatal, and the entire tourney would in fact be finished.

Keep book file open

Rather than opening and closing the book file on every probe, it is
now only closed and opened when the name of it changed.

Suppress playing of book moves with weight 0

Bookmoves with weight 0 could be played when the sum of the weights
of all moves from the position was 0. So it was not possible to suppress
the only move by giving it weight 0. This is now corrected.

added NEWS for release of 4.5.3

better contrast for XBoard icon on a dark background

translationproject.org: updated Ukrainian translation

updated pot file and send to translation project

(tiny change) Clarify "-name" option is Xt-only

There's a set of standard options with will get used by Xt, before Xboard
itself will use them.  "-name" is an Xt-only option which will set the name
and resource of the window to the value requested.  It has nothing to do
with setting opponent/player names.

Signed-off-by: Thomas Adam <thomas@fvwm.org>

modified slightly by Arun Persaud (renamed itemx to item and fixed option name for CMAIL)

Fix LoadGameOrPosition starting up engine

When LoadGameOrPosition() is called from NextMatchGame(), the Reset() that
is called in LoadPosition() when you are running from a position file
would prematurely start up the engine (without waiting for features).
The kludge to suppress this behavior for the explicit call to Reset()
is now extended to encompass LoadGameOrPosition.

Clear fSAN option before new engine load

This to prevent the new engine will inherit it from the previous one.
As the command-line options an only set it, this has to be done by
explicit assignment.

Fix some warnings (prototypes)

Fix write failures in concurrency

In WinBoard a flood of writes by differentprocesses to the same PGN can
cause a "permission denied" error in fopen. This patch causes retries after
a random (5-15 msec) wait, in such cases.

Move HistorySet to back-end

This existed in exact duplicates in both front-ends.

Make DoSleep front-end wrapper for msec sleep

DoSleep was #defined as a macro for Sleep/ sleep, with an argument
indicating msec in WinBoard, but seconds in XBoard. Nuw it universally
uses msec, by defining it as a wrapperin the respective front-ends.
(For XBoard itwraps FrameDelay.)

Fix playing sounds when -soundProgram is empty

It made no sense to just send the names of the sound files to the shell.
So the soundProgram isnow tested for being an empty string, and if so,
PlaySound() is aborted.

Add -afterGame option

This option specifies a command line to be executed after each game of
a tournament. To execute the line, the function RunCommand() was added
in the front-end. For XBoard this was a simple call to system(); for
WinBoard part of the StartChildProcess code was cloned (without the pipe
stuff).

Upgrade -serverMoves option

The (undocumented) -serverMoves option was upgraded to conform to the
ChessLive! protocol extension for sending SAN tags. A bug that allowed
upper-case promoChar to leak through was also fixed, and code was added
to write engine PVs to file. Flushing of the game result to the -serverMoves
file was also fixed.

Fix AppendComment

The joining of two comments was not performed correctly if the second
one had the braces yet to be added: the closing brace was not added in
that case.

Auto-popup ICS Input Box

Typing to the board made the move type-in pop up, but in ICS mode
this is of course undesirable. Like in WinBoard, the ICS Input Box
should pop up in stead, and accept the typed character. This is now
realized. If the box already exists and has text in it, the typed
character is appended to that text, and the box is given focus.
Spaces now also are considered as printable.

Slightly decrease sensitivity of sweep actions

The sweep-selection of promotion and drop pieces, or step through the PV
was so sensitive that it was difficult to reliably make the intended choice.
It has now been reduced by a factor 1.5-2, and the sweep selection and
PV walk now also use a different value.

Extra buttons in WB Tournament dialog

A button to copy contents of the tournament fileto the dialog is added.
Buttons to invoke the Common Engine and Time Control dialogs are also added.

Fix crash on typing non-existent enginein Load Engine dialog

Fix display state after failed analysis attempt

When an engine responded to the "analyze" command with an "unknown command"
message, a Reset was done without altering the display, causing a
discrepancy between display and internal state, which caused confusion
when it finally corrected itself. I replaced the Reset call by an
EditGameEvent call, to preserve the loaded game. The Reset was presumably
used to allow for the case where the engine would be so upset by the
analyze command that it needed to be reset by a "new" command, but this
seems overly paranoic for an engine that just recognized the command
as an unknown one.

Fix treatment of PGN score/depth info with linefeeds in them

This is needed to make the PGN more Arena-proof.

Fix quoting in Load Engine dialog

A (space-contaning)  engine name only needs quoting when there are parameters,
because the over-all engine command will be quoted anyway. For UCI engines
duplicate quoting is fatal, because when passed as the -ec argument of
Polyglot, it will be quoted yet another time, also with double quotes.

Replace Analyze File by Analyz Game function

new developer release

update po files

updated Changelog, NEWS, etc.

malloc.h is not needed

Fix ICS move-list header mistaken for null moves

Allow two-games-per-opening to work with book

A new option -seedBase is added, and written to the tourney file.
It allows all XBoard instances working on a tourney to generate the same
'random' openings in a deterministic way, based on this one-time seed
and the game number. When -loadGameIndex equals -2, it uses the same seed
for an odd and the following even game.

Fix use of random

In some places rand() was called explicitly, rather than through the
(configured) place-holder random().

Translate search texts

Add -useBoardTexture and -usePieceFont options

These options suppress the use of the given texture files or piece fonts
without relying on the kludge to disable them by prefixing an asterisk
to the file or font name. Checkboxes have been added to the Board dialog
to control them. The user can now set piece font in WB font dialog.
The sample text can be used to specify the -fontPieceToCharTable,
the font name itself the -renderPiecesWithFont value.

Fix crash on start without settings file

A primordial -1 in currCurrent caused a crash when WinBoard was started
without ini filein game-viewer mode.

Fix generic-popup failure after empty engine-settings dialog

The error exit ofthe Engine #N Settings dialog in case of an engine without
options forgot to clear currentCps, with as a result that the generic popup
thought every subsequent popup was an empty one, and kept repeating the
'engine has no options' error message for every dialog.

Allow substitution of engines during tournament

Two buttons are added in the tournament options dialog, for upgrading
and for replacing a tourney participant. Lots of tests on the validity
of the request are done, and if all are passed, the tourney file is
written with the new participants (and in case of replace) with the
results of the replaced engines erased from the -results string.

Fix up/down arrows in game list

Add key bindings for loading next/previous game of list

These bindings are set both for the game list and the board window.
To make them useful, focus has to be transferred to the board widget
after loading (it tends to transfer to the tags window), which now
seems to work, but not always. From the game list the shellWidget
seems to get focus rather than the board, which is bad as it is not
sensitive to the key bindings.

Allow deletion of last book move

The book-edit function refused to delete the entiremove list,
for no good reason. This limitation now has been lifted.
Also the error message that the key for the current book entries
is different from that of the currentposition has been suppressed
if there were no entries found for the current position (so that the
book is positioned at the next key), so that it is possible to add
moves from scratch.

Speed up parser

Line numbers > 1, which are common, can exit already during move parsing,
to save lots of matching on resultsand castlings.

Fix missing files in file browser

The file browser was throwing away the first two files, assuming
they were "." and "..". Alas, this is no longer true on every Linux.
Now we actuallyc compare for these names, before skipping them.

Delay file loading to allow expose event first

Fix paging

Better fix of crash on empty game list

An empty list is now replaced by a list containig a simple message.

Add plain arrows as key binding for stepping forward/backward

Debug position selection

Print progress during load / selection of game

The title bar of themain window is used to display a running count
of the number of games loaded / searched.

Parse PGN tags without allocating memory

In WinBoard malloc/free did not seem to work when preparing the game list,
leading to a huge memory list when ParsePGNTag was used. The added code
parses the required tags in an alternative way as a work-around.

Implement searching games in Game List for a position

For this it was needed to reconstruct all positions of the games
without loading them, so they could be compared to the currently
displayed position. A new routine GameContainsPosition is made for that.
 In Disambiguate, when the global quickFlag is set, we now do the
disambiguation ignoring check,and only if this leads to an ambiguous
move, we repeat it taking check into account to see if that resolves
the ambiguity. This causes us to miss if unambiguous moves are illegal
because they put their King in check, but who cares?
 We now do not make game-list line when not needed. If we do not filter
by text, but by position, there is no need for preparing the game-list
line for games that are not selected. All for efficiency.

Allow entry of variations in PlayFromGameFile mode

Adding variations is more like editing a comment,and not really editing
the game. Therefore we allow it on loaded games. As soon as we are in a
variation, we temporarily switch to a (hidden) EditGame mode, restoring
the original mode when we revert to the main line.
  We now also stay in PlayFromGameFile mode after loading a game,
And opening variation (fromtheComment window) in PlayFromGameFile mode
is also allowed.

Force Move History refresh after loading/reverting variation

The Move History window was not refreshed after switching to or from
a variation if that variation happened to have the same length as
the main line. Now a change in storedGames since the previous refresh will
force a full refresh to avoid this. For this to work, storedGames
did have to be decremented after ToNrEvent in PopInner, because ToNrEvent
calls HistorySet before the history is restored, so we don't want
it to snatch away the full refresh. A call to HistorySet had to be
added after PopInner is done restoring the move history.

Popdown Comment window on new game

What is in this window cannot possibly remain valid, and can even be harmful.

Allow null move in analysis and edit-game mode

In some games it is allowed to pass your turn. The internal
representation picked for such a null move is the drop of an empty
square, (EmptySquare,DROP_RANK,0,0). Computer-algebraic form is "@@@@",
SAN form is "--", (for compatibility with ChessBase / SCID), and the
non-compliant forms "null", "pass"and "Z0" are recognized in
the parser as well. ApplyMove() has to test for this special case
(to not actually clear a1), and a way to enter the pass for the user had
to be found (clicking the opponent clock, like in EditPosition mode).
 The null move is made move irreversible to prevent repetition draws
spanning null moves being adjudicated.
  Such entry of null move is now allowed in AnalyzeMode and EditGame mode.
Because Chess engines will not accept null moves, SendMoveToProgram had
to be adapted to send not the move, but the position after it in
case of a null move. This erases the move history in the engine,
so it will not react to 'undo' for that null move, so BackwardInner
had to be adapted to test for intervening null moves, and if one is
found, approach the target from the other side after loading the
earliest position before it not crossing another null move, and
then loading the moves upto the target.
  XBoard clock clicks had to be fixed, to pay attention to the shift key.

Give WinBoard Game List its own font

The font fort he Game List used to be shared with the Engine Output and
Move History windows, but this made it impossible to use a figurine font
in the latter two (which displays piece symbols for capitals) without
messing up the Game List. An option -gameListFont has been added to specify
the font for the Game List(and save it in the settings file), while a
control for it is added in the fonts dialog.

Fix crash on OK after Save Changes in Tags and Comment popup.

The global variables tagsText and commentText were used to pass
a string obtained from a widget (and thus not malloc'ed), which
caused a crash when the GenericReadout tried to use those variables
for the new value. Fixed by using GenericReadout (which does malloc)
to read out the text,in stead of GetWidgetText.

Revert "fixed segfault in xengineoutput", fixed in the backend now (from HGM)

This reverts commit 33a0e73843115ff6d4eacaf963a38f2e8b20b295.

Suppress some XBoard warnings

fixed segfault in xengineoutput

Disable -showTargetSquares in ICS mode

Help like this is considered cheating on FICS, so it is disabled for
all 'classical' ICS variants. (For variants eyont Shogi we don't care.)

Put grab on sweep-selecting in Edit Position

A right down-click starting a sweep-selection event did not grab the
mouse events, so that an up-click outside the window would go
unnoticed, and the pointer motion stayed coupled to selecting the
piece until the next click. By returning 2 from the RightClick back-end
routine we request the grab.

Write more labels on score axis of eval graph

The labels for +/-1 and +/-3 are now also written, provided there is
enough space.

Improve Eval Graph  with -evalZoom and -evalThreshold

The score range between -1 and +1 can now be blown up by a factor
specified through -evalZoom. The threshold below which histogram bars
are no longer printed (wich used to e hard-coded 25 centi-Pawn) can now
be set through the -evalThreshold option.

Print reversible plies done in stead of plies to go

Printing the number of plies left to the draw-adjudication point was
confusing when the N-move rule was set to a non-standard value.
In stead we now print the number of plies since the last irreversible
move, so the user can compare it to both the engine claim point (100)
and the adjudication point he set (which might be beyond 100).

Let WB eval graph react ot single left-click

Unlike in XBoard, the WinBoard eval graph awatys had required double-
clicking to go the the corresponding move, while there really is no
reason for this, as it is no text edit, and single-clicks have no function
(like select).

Add -scoreWhite option

This is similar to -absoluteAnalysisScores, except that it works in
all modes, and flips the score given by the engines in the engine-output
window when black is to move.

Print 50-move counter in Engine-Output title

Display score/depth in Eval Graph title

No longer print it in the comment window.

Let -sweepPromotions also work for click-click moves

A click-click promotion no longer causes auto-queening, but the down-click
on the to-square causes a Queen to stick to the mouse pointer, and then
allows the user to change it through dragging backwards before releasing it
with an up-click.
  In WinBoard this required a slight change in DragPieceBegin, because
the dragged piece there only was made visible when the mouse pointer
actually started moving (and dragInfo.pos was set), presumably under the
assumption that before mouse move the piece was already on the clicked
square. Which in the case of a to-square was of course not true. (Because
a piece jumps to the mouse pointer when the click is not centered on a
square, this gives less jittery static clicks.) So DragPieceBegin has
been given an extra argument now, to indicate whether it should attach
the piece to the mouse pointer immediately.
  To make the click-click sweepPromotions work in Shogi (for which the
drag-drop sweepPromotions could never work, because it is not clear the
piece has a posibility to promote when you pick it up), translation of
the sweep-selected piece to a Shogi promoChar had to e added.

Improve -showTargetSquares on click-click moves

The markers on move targets are now kept when a first click merely
selects a piece, rather than dragging it elsewhere. The second click
(by definition on the to-square) then clears the markers.

Let PGN parser accept lower-case piece in drop moves

The @ sign prevents amiguity with other notations anyway.

Fix parser.c line endings

Fix writing searchTime in tourneyFile

This had to be derived from the variable searchTime, which gives it
in seconds, as appData.searchTime (a string in minutes) is not reliable,
as it contains the command-line value, which might have been altered
through the time-control dialog.

Fix resetting searchTime if other TC mode is chosen (XB)

This fixes a bug in the new time-control dialog.