Reset win/loss counters before match

The win/loss counters were not reset, because the code counted on them
being initialized to zero. But with matches startable from the menu,
this is no longer true after the first match.

Show move that causes false illegal-move claim

Without claim testing XBoard takes back moves that an engine objects to.
But when the engine is forfeited for a false claim, the move should of
course stay. So the claim test is moved to before the take-back.

Fix Spartan promotion to King

Promotions toKing were refused when in check, because XBoard thought
they would not resolve the check. Now the promotion is actually
performed before doing the check test.

Fix WB font-based piece rendering on variant switch

The piece bitmaps were only created from the font characters when the
number of ranks or files changed, because it was done in
InitDrawingSizes(). This caused missing pieces when changing to a
variant on the same board that used other pieces. Now InitDrawingSizes()
is called for any variant switch.

Fix right-alignment pproblem in WB ICS window

Use of Ctrl-R to summon the context menu still trickled through to the
default text-edit window routine, and caused right-alignment. This was
because the text-edit reacts on keyDown rather than the character event.
KeyDown of R with Ctrl pressed is now also intercepted and ignored.

Fix chat window title

The title was using the engine name, rather than the ICS handle, which
did not work outside zippy mode. Now it only uses the engine name when
no ICS handle is known. The titleis now also subjected to translation.

Fix spurious reading of old game title by JAWS

Because focus was switched to the board window before loading the
selected game from the game list, the titlebar still contained the
name of the previously loaded game, which would spontaneously be read by
JAWS. Now the focus switch is done after loading the new game (which
updates the window title.)

Add variant Spartan Chess

In the WB menu it takes the place of the defunct Twilight Chess item.
The setup routine needed patching for setting up Lances in stead of
Pawns for black, the check test had to be adapted to handle duple check,
Alfil and Dragon moves were fine tuned in this variant to make the exact
moves, and the promotion popup was adapted to show King and interpret
Rook and Bishop as Dragon and Alfil for black. The Alfil now always
moves as Modern Elephant except in Shatranj and Courier.

Fix mate test

Due to a bug in logical operators drops were tried for mate evasions
even in non-drop games, which broke it in Superchess and Great Shatranj,
and wasted CPU time in other variants.

Fix promotion suffixon disambiguated piece moves

The promotion suffixon piecemoves that needed disambiguation was not
passedon to the back-end in most variants. Now it is passed on in any
variant.

Fix promotion of Pawn-like Lance

In variants where the Lance is used to represent an unorthodox Pawn, it
must promote when it reaches last rank. With legality testing on,
however, the promoChar was ignored in ApplyMove in this case. The
promotion suffix also was not appended in SAN moves.

Fix WB promotion popup

The promotion popup did not work properly in variants where black did
not use the pieces that white wanted to promote to. Now the character
for the selected piece is taken from the piece encodings of the side to
move. In addition, selection of undefined pieces is ignored.

Fix bug in parsing variations

The moves parsed from a variation were stored in the moveList without
appended linefeed. This caused errors if the variation extended to
beyond the end of the main line,into a hitherto unused part of the move
list, as the linefeed was then not there from the overwritten move. As a
result moves sent to the engine were concatenated, triggering
illegal-move messages and bringing the engine out of phase.

Define Ctrl-R as right-click in WB Comment dialog

To allow JAWS users to load a variation through the keyboard, Ctrl-R is
equated to a right click at the position of the caret or selection.

Allow starting a variation from keyboard entry in WB-JAWS

The status of the shift key is remembered when space is pressed to
select a board square.

Disable some accelerators in JAWS version

Single-key accelerators that could result in irreversible data loss are
undesirable, because the risk of accidentally pushing them is too large.
So the non-Alt duplicates of next/previous game/position are disabled,
as are Home and End for Revert and Truncate Game. JAWS users will have
to rely on the menus for those functions.

Group engine-output memos with engine names

The engine names are now defined just before the text edits, so that
JAWS 11 will read the engine name when the memo for the output of that
engine is selected. (JAWS 10 will still read the NPS, as it reads in
geometrical order.)

Put nr CPU earlier in tab cycle of WB Common Engine dialog

By defining the max CPU control after hash size, the tabbing steps
through the controls in geometrical (and logical) order, rather than
having the max CPU in between the book controls.

Update WB language template file

Subject JAWS menu and spoken strings to translation

Let JAWS SayString buffer full sentence

The meaning of the flag parameter of SayString was changed to indicate
buffering of flushing. On flushing a message of a single space is used
as a kludge to indicate interruptability. Calls to force flushing where
added everywhere.

Adapt JAWS menu to new menu organization

The menus did not correspondto the texts after translation of the main
menu bar.

Translate menus after creation of JAWS menu

Fix ICS context menu for JAWS

The Ctrl+R key did select text at the mouse pointer in stead of at the
curser, now that up and down-click were combined.

Reorganize order of WB New Variant dialog controls

The definitions are re-ordered so that at least JAWS 11 will speak the
board-size elements properly.

Fix order of elements in WB time-control dialog

Someleading texts were defined after the text edit they referred to,so
that JAWS 11 associated them to the wrong one. A tab stop was added to
the time-odds label, to make JAWS speak it.

Change tabbing order in WB sounds dialog

The 'Play' button now gets focus immediately after the controls for
selecting the sound, rather than at the end of the dialog (after OK).

Remove tab stops on first radio button of WB dialogs

The New Variant and Board dialogs had a tab stop on the first radio
button,so that tabbing toit would automatically select variant
normal or size tiny. Now it tabs straight to the current size, without
changing anything. (Important for JAWS users.)

Put OK/Cancel last in tab cycle of some WB dialogs

Fix duplicate menu character for Edit Comment

Edit Comment did have the same shortcut character as Copy Game. Changed
to m now.

fixed automake process: xboard.conf couldn't be generated if $srcdir was not the current directory

new developer release

Subject WB context menus to translation

The innerloopof the menu translation has been made into a separate
routine. This now uses GetMenuItemInfo() in stead of ModifyMenu() to
better preserve menu breaks. Menu #10 is used for context menus (from
the routine that pops them up), and for this number the orignal texts
are not saved in menuText. (This is not needed, because the menu is
created everytime anew from the resource data.)

Update translation template

Add the new menu items; add some notes.

Resize buttons in WB engine-settings dialog

The OK and cancel buttons were to narrow for most translations, and were
a non-standard order as well.

Add one-click move control to WB general-options dialog

This is a duplicate of the one in the ICS options dialog, but the latter
is inaccessible in non-ICS modes, and one-click move is now
alsoavailable in engine mode.

Create space in dialogs for translation

The translated texts often did not fit, because controls were defined
needlessly small (when there was empty space between them). In these
cases they have been enlarged to consume all available space. Some
dialogs are made wider.

Add handle for translator acknowledgement

An empty string is printed in the about box, which can be 'translated'
to a message acknowledging the translator.

Use default logo for user

When no logo with the name of the current user is found, use dummy.bmp.

added rlwrap tip to FAQ

replace hard coded paths with path from configure script

use $datadir and subdirectories to store some data files. Also make files in xboard.conf use correct path.

fixed a typo in the configure script output

added a desktop file

taken from the openSUSE project

Fix Edit Game/Position checkmarking in WinBoard

Because the entries in Edit and Mode menu were pure duplicats, the
entries in the wrong menu (Edit) were checkmarked while active. By
defining different IDs for those in the Edit menu, those in the Mode
menu are checkmarked again.

Add XBoard menu items for arrow highlight and one-click move

These items are put in the options menu; the 'Old Save Style' item is
removed.

Provide menus for editing WinBoard engine and server lists

The Edit Tags dialog in the WB front-end is adapted to act as a general
edit for string variables, by giving it an extra parameter to indicate
where to store the edited result. Menu items are added to invoke this
dialog for editing the /first, /secondChessProgramNames and /icsNames
options. This solves the problem that people would have to edit an ini
file that they can no longer locate (because the %APPDATA% folder has
different names and locations in the various Windows versions, and is
often a hidden folder on top of that).
There is a slight memory leak here, as the memory for the previous
value of the altered string option is not freed.

Configure arrow highlighting as default in XBoard

Set -highlightMoveWithArrow true in the xboard.conf file, and specify a
lineGap of 0 to make that efficient (and the wood theme even nicer).

Port highlighting with arrow to XBoard

Unfortuntely this duplicates a fair amount of code in the front end.
Removing the arrow is done by a total repaint if there is a non-zero
lineGap, because the clever algorithm for selective redrawing (based
on damage) that XBoard uses does not repair damage to the grid lines.
For lineGap=0, however, (for which the arrow is mainly intended) the
damage mechanism is used, and the damaged squares are determined by
tracing out the arrow in 64 steps, and marking all squares in the
neighborhood as damaged.

Strip CR from xboard.c

Let user decide if he wants highlights in blindfold mode

Remove the automaticsuppression ofhighlighting.

Fix fag-fell marker with logos on

The flag-fell marker "(!)" that is displayed in the clock in WinBoard in
two-machines mode was printed twice when the clock was split over two
lines (as with logos on).

Also exempt variant seirawan from eval-scale doubling

It is not really a drop game, so a scale from -7 to 7 in the eval graph
is good enough (like variants great and super).

Configure EGTB cache size in adapterCommand

The hash size, nr of CPUs and EGTB path are transferred through the WB
protocol interface commands memory, cores and egtpath. But there is no
WB command for detailing the EGTB cache size. So the value of the
command-line option -defaultCacheSizeEGTB has to be sent to Polyglot
through its startup command to make it known to UCI engines.

Move Hint and Book items to Engine menu in XBoard

Correct the key bindings fo view menu in texi file

The mentioned bindings for Game List and Eval Graph were wrong.

Fix sub-variation display with negative score

The test if the displayed score was the pvInfo score failed for negative
scores, because floats are truncated towards zero. So the previous fix
of spurious score/depth info was not working when the score was
negative.

Peel PV out of comment

Arena stores engine PVs not in variation format, but within both braces
and parentheses, before the score/depth info. This caused XBoard to
overlook the score/depth info, (frustrating, e.g., the eval graph),
while the PV would not be accessible for walking it as a variation.
The routine that extracts the score/depth from a comment now has
been made more clever, to recognize a leadin PV in the comment, and then
place it in front of the comment as variation (by moving the opening
brace to behind it). For some reason it does not work if the PV and
score/depth were separated by a newline, but this happens quite rarely.

Allow line-straddling result comments

Some GUIs (in particular Arena) do not have the decency to keep the
result comment on a single line. In this case the parser rules would not
recognize it, and treat it as a normal comment, plus an undetailed PGN
result. The rules are now fixed to allow linefeeds in the comment, and
between comment and PGN result. To correctly process the comment further
(e.g. in game-list lines) the linefeeds are replaced by spaces in the
back-end routines that do use the result comment.

Write key bindings in XBoard menus

Unfortunately tabs are not understood in menu texts, so fine-tuning
through spaces is necessary to get some sort of ragged alignment.

Configure XBoard to use wood board texture by default

The texture pixmaps are sought in /usr/share/games/xboard, with names
wood_l.pm and wood_d.xpm. I guess the " make install" should be adapted
to put them there. XBoard is resitent to non-existence of the files,
btw. It simply ignores the arguments in that case.

Configure -inc as volatile option in XBoard

XBoard users are not used to options being persistent, and thus have
many scripts assuming the timeIncrement is -1 when they don't specify it
on the command line, and want to specify -mps. But a persistent -inc of
0 fom a preious session would then force their -mps to zero, causing
big-time confusion.

Make WinBoard clocks translation-proof

With logos on, the splitting of the clock over two lines assumed the
lengths of the words "white" and "black" would always be 5. Used strlen.

Improve drawing of highlight arrow

A diagonal arrow sometimes had a notch in the shaft because of
independent rounding of the starting and ending point coordinates.
Furthermore the width of the shaft was unnecessary variable(as a
function of direction) because half the width was rounded and then used
twice. Now rounding of end-points is always done towards a fixed grid
of square centers, which keeps diagonals purely diagonal, and one side
of the shaft is drawn relative to the other side, after the
position of the latter has been rounded, so any rounding error affects
the shaft width only once.

Alter descriptionof Analyze File menu item

This menu item has been given a new function, so a new description was
needed.

Add description of Game List Options in texi file

Update texi file for new menu organization

The descriptions are shuffled around and organized into sections
corresponding to the new main menus. Where necessary the name of the
items was changed to their new names. The items removed from the File
menu (the next/previous/same stuff) are nowmoved to the section "other
shortcut keys", as the key bindings for some of them are still active.

Update info on key bindings in texi file

The texi file was still listing the old XBoard key bindings. Where
necessary this is now changed to the new (WinBoard) bindings, and all
the new bindings were added.

Add XBoard key bindings for Revert and Truncate

Use Home key as accelerator for Revert, End for Truncate, as in WinBoard.

Divorce the Edit and Show Tags/Comment menu items

In WinBoard the items in the Edit and View menu for Tags and Comment now
have different IDs, so they can be independently controlled. Only the
items in the View menu are check-marked. The items still do the same.

Alter WinBoard menu text

The texts are slightly adapted, mostly by choosing more convenient
tabbing characters for the new menu organization. The translation
template is updated accordingly.

Let move-history window scroll to bottom after adding move

By activating the scrollbar of the viewport through XtCallActionProc
the move-history window is scrolled to the bottom.

Fix Shogi promotion popup

The texts were adapted to shogi by printing new text items over the
Queen and Knight buttons, but these did not alwys cover it. Now the text
on the buttons is changed as it should be. Also the window title is
adapted.

Apply gettext macros to menu texts

Now texts and references are separated, the texts in the tables can be
decorated with the N_() macro, so that the _() macro no longer has to be
applied when they are used. (This makes the texts appear in the .po
files.)

Adapt some XBoard menu texts

The "Show" prefix is removed from the "View" menu items, as this seems
implied. An ellipsis is added to the Engine #N Settings items, which
could not be there before, because periods in a menu name would break
any reference to it. For some other names the ellipsis was deleted.
(These were not refered to, but who knows what we will do in the future.)

Separate menu text from menu name in XBoard

The text string in the menu-definition tables was used both for the menu
text, as well as for referencing the items from the code (for enabing or
checkmarking). Now these can be defined separately, so that altering the
menu texts is possible without breaing any references to the item.
(Moving them to another menu would still break XBoard, though!)
Currently the texts and names are still the same.

Bugfix XBoard menu translation

The gettext macro N_() was applied to the menu definition tables, but
this would break any translation, as these items are both used as menu
texts as well as reference for enabling and checkmarking. The routines
that create the menus already apply _() dynamically to these items when
they are used as menu texts.

Restrict drops in variant seirawan to back rank

Drops aren't really llowed at all in Seirawan Ches, but there exists an
engine that supports a variant where drops on the back-rank are allowed.

Let Analyze File annotate the loaded game

By not suppressing the auto-play loop in AnalyzeFle mode, we step
through the game while the engine is analyzing, and store the PV info
that the engine produces, as well as the full PV (as variation comment)
with each move.

Revive Analyze File menu item in WB

We will keep this menu after all, becase it can be given a new meaning.

Fix spurious scores in comments

The comment popup is also used to display engine PV info, but when the
contents of the window is written back when we select a variation,
it was treated as a real comment. This is fixed now by checing in
ReplaceComment() if the comment starts with the PV info, and if so,
ignore the latter. This solves the problem, but I guess the real bug is
that selecting a variation tries to write the comment even if it did not
change.

Display error for wrong use of Machine Match

Reorganize XBoard menus

Reorganize WinBoard menus

New main menus Edit, View and Engine are added; the step menu
disappears. The menu items are redistributed over the new menus.
Sometimes duplicates are created. No adaptation of short-cut characters
as made yet (as this affects translation).

removed trailing whitespaces from NEWS

merged readme_HGM.txt into the NEWS file

seems like a better place for the release notes, although those releases where not officially part of GNU, but the NEWS file already covers similar releases, so it might as well cover all of them.

typo in date of developer release

but will leave the tagged version as it is, since it's still a unique version number

new developer release

added latest version of parser.c

updated Changelog, etc for developer release

Revert "adding gnu-readline support"

This reverts commit c95ef32a700132a134f335f857ca6f1b6f3c169f.

Conflicts:
backend.c

better to use the (relatively) new ICS input box then add readline support.

Revert "bugfix: fixed readline support with icslogon option"

This reverts commit b9cdd7422ad1037d7376bce4f8f2db2669396e83.

Conflicts:
backend.c

better to use the (relatively) new ICS input box then add readline support.

Make Hoplite moves irreversible in Spartan Chess

Consider Lance moves in variant Fairy (where the Lance represents the
Spartan Hoplite Pawn) as Pawn moves, so they reset the 50-move counter.

Give Lance moves of Amazon in variant super

Now that Lance is no longer a wild-card piece the moves of Amazon must
be programmed explicitly, to keep the possibbilty to play Superchess
with legality testing on.

Change representation of Bede in variant fairy initial position

This used to be Lance, but the promoted Bishop (Dragon Horse)
is more applicable. Especially now that Lance is supposed to move like a
Berolina Pawn.

Make yyskipmoves also suppress examining of drop moves

Not sure if this does any good, but it seemed illogical that this would
happen in all other moves, but not drops.

Define moves for Dragon Horse outside Shogi

Let it move like Bishop + Wazir + Dababba.

Fix safety crash when appending comments

The length limit imposed on safeStrCpy was one too short when copying
an existing comment to the space allocated for the longer (concatenated)
one. It allowed no space for the terminating '\0'.

Fix safeStrCpy

The test for when to terminate a non-fiting string was one off, and the
debug message was written stdoutin stead of debugFP.

Fix default holdings size for variant seirawan

The holdings for 7 pieces werealways considered an overrule, so that the
variant name was prefixed with "8x8+7_".

Fix home-dir crash

When confronted with a path name starting with ~ and not containing a /
XBoard would segfault.