xboard.git
3 years agoAdd configure-options section to texi file
H.G.Muller [Mon, 29 Feb 2016 13:24:11 +0000 (14:24 +0100)]
Add configure-options section to texi file

The options data, saveDate, addMasterOption, installEngine, installTheme
and autoClose are now described in the manual.

3 years agoUpdate texi file
H.G.Muller [Mon, 29 Feb 2016 12:15:47 +0000 (13:15 +0100)]
Update texi file

3 years agoAdd -installTheme option
H.G.Muller [Mon, 29 Feb 2016 11:53:57 +0000 (12:53 +0100)]
Add -installTheme option

A new option of type ArgInstall is added, which will add its argument
to the -themeNames list (under the usual restriction it is not already
in there, and that it was not seen before). By appending such a (time-
stamped) option to the master settings file with -addMasterOption,
newly installed themes can be made to trickle their way into the user's
private settings file. All the theme package has to is call XBoard with
the applicable -addMasterOption "-installTheme ...." arguments in its
install script, and copy any associated files to XBoard's datadir.

3 years agoFix disappearance of premoved piece
H.G.Muller [Sun, 28 Feb 2016 11:01:40 +0000 (12:01 +0100)]
Fix disappearance of premoved piece

When a premove was done by dragging, (which for expected recaptures would
be the only way, as click-click moving selects the intended target),
the moved piece would disappear from view. This because the from-square
would not be redrawn (in normal move execution it would remain empty).
This could have been an XBoard-only problem. In any case We now force a
complete redraw of the board, which is not very elegant, but should always
fix it.

3 years agoFix bug #43792 (no highlights after rejection of premove)
H.G.Muller [Sun, 28 Feb 2016 10:34:52 +0000 (11:34 +0100)]
Fix bug #43792 (no highlights after rejection of premove)

If the sending of a premove fails to add a move to the game, because
XBoard itself judges the move illegal (which could only be determined
after arrival of the opponent move, not when it was entered), XBoard
now re-applies the move highlights for that opponent move.
 This does not solve the problem yet with legality testing off,
when it later turns out the ICS rejects the move.

3 years agoFix bug #45599 (inclusion of keysym.h in Xaw)
H.G.Muller [Sat, 27 Feb 2016 20:59:52 +0000 (21:59 +0100)]
Fix bug #45599 (inclusion of keysym.h in Xaw)

3 years agoFix bug #45773 (needless #inclusion of cairo-xlib.h)
H.G.Muller [Sat, 27 Feb 2016 19:50:27 +0000 (20:50 +0100)]
Fix bug #45773 (needless #inclusion of cairo-xlib.h)

As this header no longer exists this was causing a compilation error.
In the GTK version they are of course never needed, and the draw.c
that is also part of an Xaw build also does not call any function to
draw on an Xaw surface directly.

3 years agoFix bug #45775 (Infinite loop on nonexistent texture file)
H.G.Muller [Sat, 27 Feb 2016 19:45:01 +0000 (20:45 +0100)]
Fix bug #45775 (Infinite loop on nonexistent texture file)

3 years agoFix bug #45774 (GTK compile bug with ENABLE_NLS)
H.G.Muller [Sat, 27 Feb 2016 19:37:09 +0000 (20:37 +0100)]
Fix bug #45774 (GTK compile bug with ENABLE_NLS)

The declaration of an XFontSet function was not protected by

3 years agoMove Common Engine dialog to Engine menu (WB)
H.G.Muller [Sat, 27 Feb 2016 16:06:33 +0000 (17:06 +0100)]
Move Common Engine dialog to Engine menu (WB)

The item is renamed to "Common Settings", whith shortut 'C'.
The "Edit Engine List" item is also assigned a shortcut ('E').

3 years agoUse mouse wheel for selecting piece in Edit Position mode (XB)
H.G.Muller [Fri, 26 Feb 2016 22:35:39 +0000 (23:35 +0100)]
Use mouse wheel for selecting piece in Edit Position mode (XB)

Rather than calling ForwardEvent() and BackwardEvent() directly from the
mouse-wheel handler, we now call a backend routine Wheel(), which gets
the pointer coordinates passed to it (which we must remember during
mouse move events). This then calls the Forward/Backward event, except
in EditPosition mode, where it changes the piece in the square below the
pointer.

3 years agoAllow promotion choice in engine-defined variants
H.G.Muller [Wed, 17 Feb 2016 19:55:36 +0000 (20:55 +0100)]
Allow promotion choice in engine-defined variants

When variant shatranj, courier or makruk, which normally do not have
promotion choice, are used as parent for an engine-defined variant,
we now invoke the piece-selection routine. This to make these variants
also suitable as parent for variants that do have choice (e.g. to give
them the baring rule, which is unique for Shatranj).

3 years agoFix erasing of arrow highlight (XB)
H.G.Muller [Wed, 17 Feb 2016 11:47:45 +0000 (12:47 +0100)]
Fix erasing of arrow highlight (XB)

On large board with long-range oblique moves (like in Grande Acedrex)
the head of the arrow was not always erased (and not always drawn either).
This was partly due to the arrow pointing to a corner of the target square
even if the path was nearly orthogonal, while the damage trail is always
assumed to connect the square centers. Having the arrow point more
precisely and widening the damage trail a bit solved the problem.

3 years agoAllow promotion on two-leg move
H.G.Muller [Tue, 9 Feb 2016 09:59:31 +0000 (10:59 +0100)]
Allow promotion on two-leg move

After a second-leg of a SAN move the parser now checks for apromotion
suffix, and if any is found this is used by ParseOneMove / LoadGameOneMove
to set the promoChar. In converting to SAN for the parseList the promoChar
is now appended to the second leg, rather than the first, and the Edit Book
window will also show it. The promochar is written behind the kill square
in the moveList, and when sending the move there to the engine it is
appended to the second move of the multi-move pair.

3 years agoFix type-in of hit-and-run captures
H.G.Muller [Tue, 9 Feb 2016 09:25:40 +0000 (10:25 +0100)]
Fix type-in of hit-and-run captures

Processing of the move in ParseOneMove (which basically duplicates
the code of LoadGameOneMove) was not yet adapted to ignore ';' as a
promotion character when it is internally used to append 'kill squares'
where locust capture should take place..

3 years agoAllow setting of piece nicknames from pieceToChar string
H.G.Muller [Tue, 9 Feb 2016 08:53:44 +0000 (09:53 +0100)]
Allow setting of piece nicknames from pieceToChar string

A piece IDs in the pieceToChar string can now be suffixed by "=L",
with L an arbitrary single letter, to define a nickname for the piece.
Such a nickname would make it possible to specify a complex ID like +I'
as promotion choice, and by indicating this in the pieceToChar string
the engine can configure that through the 'setup' command. This facility
is needed to force promotion after capture of a contageous piece in
variants with Shogi promotion, such as Maka Dai Dai Shogi.

3 years agoSlight speedup of parsing promotion suffix
H.G.Muller [Tue, 9 Feb 2016 08:31:11 +0000 (09:31 +0100)]
Slight speedup of parsing promotion suffix

3 years agoFix printing of 'x' in position diagram
H.G.Muller [Mon, 8 Feb 2016 12:21:20 +0000 (13:21 +0100)]
Fix printing of 'x' in position diagram

The times where 'x' was used as PieceToChar result for an empty square
are long behind us; 'x' now is a valid piece ID.

3 years agoImprove variant recognition for enabling buttons (XB)
H.G.Muller [Sun, 7 Feb 2016 21:48:34 +0000 (22:48 +0100)]
Improve variant recognition for enabling buttons (XB)

Buttons for variants were enabled when the name of the variant they
selected was a sub-string of waht the engine could play, so that
Grand Chess (variant grand) would be enabled by Fairy-Max' grande-acedrex.
Now we test if the matched sub-string of the engine's variants feature
is immediately followed by a comma (or at the end).

3 years agoFix using VariantMen PGN tag for both colors
H.G.Muller [Sat, 6 Feb 2016 17:18:29 +0000 (18:18 +0100)]
Fix using VariantMen PGN tag for both colors

Black pieces were not recognized as the same as white pieces.

3 years agoAlways assume FEN in variant-fairy PGN game is initial position
H.G.Muller [Sat, 6 Feb 2016 15:19:52 +0000 (16:19 +0100)]
Always assume FEN in variant-fairy PGN game is initial position

As variant fairy is a catch-all without a defined initial position,
the FEN tag n a PGN game for this variant must be considered an initial
position. So we copy over the FEN position to the initial one in this
case. Otherwise castling would always be thought illegal in fairy games
loaded from PGN.

3 years agoProcess VariantMen PGN tag
H.G.Muller [Sat, 6 Feb 2016 14:12:19 +0000 (15:12 +0100)]
Process VariantMen PGN tag

On loading a PGN game the VariantMen tag is now parsed, and the
move definitions are assigned to the mentioned pieces.
This only works when the pieceToCharTable is defined in advance!

3 years agoLet VarianMen PGN tag work with dressed letters
H.G.Muller [Sat, 6 Feb 2016 12:46:43 +0000 (13:46 +0100)]
Let VarianMen PGN tag work with dressed letters

The internal 'character' codes > 127 are now printed as letter + suffix.

3 years agoRemove debug printf
H.G.Muller [Sat, 6 Feb 2016 09:25:27 +0000 (10:25 +0100)]
Remove debug printf

3 years agoFix spurious promotion partners
H.G.Muller [Sat, 6 Feb 2016 09:21:21 +0000 (10:21 +0100)]
Fix spurious promotion partners

When defining a new promotion pairing through ^L the default partners
of L and +L must be made partnerless, or they would still promote to +L
or demote to L.

3 years agoFix printing of piece ID in illegal SAN moves
H.G.Muller [Fri, 5 Feb 2016 10:26:51 +0000 (11:26 +0100)]
Fix printing of piece ID in illegal SAN moves

Illegal SAN moves did not add ' or ! suffixes to the piece ID,
and printed shogi-promoted pieces as a '+' without ID.

3 years agoFix clipping of GTK menu-bar labels for broad boards
H.G.Muller [Fri, 5 Feb 2016 09:06:09 +0000 (10:06 +0100)]
Fix clipping of GTK menu-bar labels for broad boards

The number of characters kept in the menu-bar labels was dependent on
square size, but did not take into account the number of files of the
board. So when there were more than 8 files, the label texts were often
clipped while there was in fact plenty of room to display them fully.
Now the adjustment of square size based on number of board files is done
after the other window parameters have been picked based on the window
width calculated as 8*squareSize. Only the default line gap is then
adapted to the actual square size (consistent with what is done during
user sizing of the board later).

3 years agoChange the piece order again
H.G.Muller [Thu, 4 Feb 2016 18:43:14 +0000 (19:43 +0100)]
Change the piece order again

The internal piece encoding is now altered such that the uninteresting
ones (promoted Shogi pieces and symmetry partners) are all at the end,
just before King. These now start with the PromoBishop. A slash in the
pieceToChar string skips to this new 'promoted series', so that strings
can be made 'future proof' (for when new pieces are added before it).
The promoted Chu pieces all get new book keys by this, but as it is
unlikely that they occurin opening lines, this was considered acceptable.
The +L, +T in Chu are now represented by Pegasus and Gnu rather than
Knight and Chancellor, but the latter two are defined as fall-backs to
remain compatible with old Chu themes.
The +N, +L, +S replacements in Shogi are now also taken from amongst the
regular pieces, rather than extra images stored beyond King.

3 years agoAdapt Chu-Shogi pieceToCharTable to new piece order
H.G.Muller [Wed, 3 Feb 2016 09:24:17 +0000 (10:24 +0100)]
Adapt Chu-Shogi pieceToCharTable to new piece order

3 years agoFix demoting in Edit Position mode
H.G.Muller [Wed, 3 Feb 2016 08:34:42 +0000 (09:34 +0100)]
Fix demoting in Edit Position mode

A click on an already selected demotable piece will now cause demotion,
rather than disappearance of that piece, similar to how clicks on a
selected promotable piece will promote it.

3 years agoClear highlights after moving piece in Edit Position
H.G.Muller [Wed, 3 Feb 2016 08:23:15 +0000 (09:23 +0100)]
Clear highlights after moving piece in Edit Position

When moving a piece during Edit position the from-square remained
highlighted, but the to square was not highlighted, which would
create the erroneous impression that the (now-empty) from-square
was selected (and ready to obliterate a piece on the next click).
Now this highlight is cleared. This was preferred over also highlighting
the to-square, because this is not really a move in a game.

3 years agoFix aborted detour under-promotion XB
H.G.Muller [Tue, 2 Feb 2016 18:18:13 +0000 (19:18 +0100)]
Fix aborted detour under-promotion XB

When a dragged Pawn was released before the promotion piece was fixed,
this would fake a release on the from-square. But the repair of animation
damage would take place on the to-square derived from the actual mouse
coordinates. When this was different from the from-square, the animation
damage (consisting of removalof the piece) would thus not be repaired
by redrawing that square, as normally (when the to-square is different)
it should remain empty. Now we mark the from-square as damaged when the
detour under-promotion changes the dragged piece.

3 years agoFix default piece in Shogi promotions
H.G.Muller [Tue, 2 Feb 2016 16:34:53 +0000 (17:34 +0100)]
Fix default piece in Shogi promotions

In Shogi the default should be promotion, but it was using Queen like
the other variants. As the +/= issue was decided based on comparing this
default promotion piece with the moved piece, the Lance (which internally
is represented by Queen) would by default defer. Now the up-click of
a drag-drop move (which would bypass sweep selection) sets the default
choice to the promoted partner if the pieceToCharTable specifies one.

3 years agoFix sweep promotions to Tokin
H.G.Muller [Tue, 2 Feb 2016 16:31:41 +0000 (17:31 +0100)]
Fix sweep promotions to Tokin

Sweep promotions of multiple pieces to Tokin were not working, because
once the promoted version was shown it would always demote to Pawn.
Now the toggle always demotes to the selected piece.

3 years agoFix setting default piece from 'choice' command
H.G.Muller [Tue, 2 Feb 2016 12:10:24 +0000 (13:10 +0100)]
Fix setting default piece from 'choice' command

PieceToCharwas used instead of CharToPiece. In addition it is important
to deduce the color from currentMoe rather than forwardMostMove, as
one could be entering a move in the middle of a game (e.g. when editing
or starting a variation).

3 years agoFix two compiler warnings
H.G.Muller [Tue, 2 Feb 2016 11:09:17 +0000 (12:09 +0100)]
Fix two compiler warnings

3 years agoAllow pieces with dressed-letter ID as promotion choice
H.G.Muller [Tue, 2 Feb 2016 08:27:34 +0000 (09:27 +0100)]
Allow pieces with dressed-letter ID as promotion choice

Testing for the occurrence of a piece in the promotion set specified
through the 'choice' command now takes dressing of the piece IDs into
account.

3 years agoFix castling rights
H.G.Muller [Mon, 1 Feb 2016 20:51:04 +0000 (21:51 +0100)]
Fix castling rights

The expansion of the number of piece types to > 64 had broken the
accounting of castling rights, because the code for NoRights is also
a piece code (as it is stored in the board), and had gotten a value
so high that it could no longer be stored as signed char. This is
solved by declaring the initialRights array as unsigned char instead.

3 years agoImplement engine-requested settings-popup WB
H.G.Muller [Mon, 1 Feb 2016 19:45:14 +0000 (20:45 +0100)]
Implement engine-requested settings-popup WB

3 years agoAllow O1 as Betza castling descriptor
H.G.Muller [Mon, 1 Feb 2016 16:32:05 +0000 (17:32 +0100)]
Allow O1 as Betza castling descriptor

The notation On is now also recognized as castling for n=1, but this
uses the same system as non-royal castling ('guarding'), which is
performed as hit-and-run capture of the Rook (or whatever you want to
castle with) while moving to the destination (one square away here).
This to avoid the ambiguity with normal King moves (which for the
guarding always existed).

3 years agoAllow engine to force popup of its settings dialog
H.G.Muller [Sun, 31 Jan 2016 23:12:41 +0000 (00:12 +0100)]
Allow engine to force popup of its settings dialog

An engine can now force popping up of its settings dialog by sending
'feature done=3' to the GUI. It can clear its option list (e.g. for the
purpose of resending it because of altered settings) by sending
'feature done=0'. Such resending would cause the dialog to be updated
when it was already open, or close it and pop up a new one when the
engine specifies it has to be redesigned by ending the list with
'feature done=2'.

3 years agoChange pieceToCharTable order of pieces beyond Lion
H.G.Muller [Sun, 31 Jan 2016 16:28:40 +0000 (17:28 +0100)]
Change pieceToCharTable order of pieces beyond Lion

The positions in the pieceToCharstring no longer correspond to internal
piece type, but are remapped such that more useful symbols reside near
the beginning. The piece types themselves could not be re-ordered,
as this would alter the calculation of the opening-book hash key.

3 years agoExpand numer of new piece types to 2 x 11
H.G.Muller [Sun, 31 Jan 2016 14:53:05 +0000 (15:53 +0100)]
Expand numer of new piece types to 2 x 11

The number of pieces added between basic and promoted series, and between
the latter and King is increased to 11, and they are ordered differently.
The image used for the Chu-Shogi White-Horse piece,which was a duplicat
of the Knight, is now replaced by a winged Knight.

3 years agoUse flexible promotion assignment
H.G.Muller [Sat, 30 Jan 2016 08:09:03 +0000 (09:09 +0100)]
Use flexible promotion assignment

The promoted partner now comes from an array, rather than by adding
a constant (11, or 27 in chu) to the base type. The array is initialized
so the first 11 pieces (the old 'base pieces') point to the second 11
(the old 'promoted series'), and the latter point back (so the array
also contains the demotions!). The pieces above 21 (= Lion) point to
themselves.
  Parsing of the pieceToChar string now considers ^, - and * prefixes
rather than independent IDs. Nevertheless it is the - and ^ that get
stored in the pieceToCharTable (the latter as '+') for the corresponding
piece. But the ID that follows them specifies what other piece they are
the demoted or promoted partner of, rather than their own ID, and is
thus used to update the promoPartner array for both of them.
  A * prefix does mean the following ID is the ID assigned to the piece,
but that the piece promotes to Tokin (if the latter is assigned a '+').
  The default pieceToChar string for varaint chu is adapted to use this
new assignment mechanism of promotion partner. (Chu is so far the only
variant that needed other pairing than the default.)

3 years agoLet PROMOTED and DEMOTED macros use argument
H.G.Muller [Fri, 29 Jan 2016 16:41:58 +0000 (17:41 +0100)]
Let PROMOTED and DEMOTED macros use argument

These macros were expanding to offsets added to what followed them.
Now the thing that followed them is passed to them as an argument.
This will allow them to expand the prmomotion or demotion to a table
lookup, rather than just an addition. Where PROMOTED appeared in a case
label, though, it was just a readability enhancement for the fixed
assignment, and a new macro PROMO equal to the old PROMOTED is introduced
for that.

3 years agoFix Chu promotion with added pieces
H.G.Muller [Fri, 29 Jan 2016 09:32:02 +0000 (10:32 +0100)]
Fix Chu promotion with added pieces

There were still some hard-coded 22s in the code, which no longer described
the difference between Pawn and Tokin and the number of promotable pieces,
now that new pieces have been added.

3 years agoMake move to own piece a swap rather than capture
H.G.Muller [Thu, 28 Jan 2016 16:55:15 +0000 (17:55 +0100)]
Make move to own piece a swap rather than capture

This is useful for Tamerlane Chess, where you can one time during the
game have your King swap places with a friendly piece.

3 years agoImplement 'choice' engine->GUI command
H.G.Muller [Thu, 28 Jan 2016 16:32:19 +0000 (17:32 +0100)]
Implement 'choice' engine->GUI command

A new CECP command is implemented: With 'choice XYZ' in reply to a
'lift' or 'put' command an engine can redefine the promotion choice
of the promotion move that emitted these to X, Y or Z. (Always capitals!)
For now only affects sweep & detour promotions, not the promotion
popup.

3 years agoLet target-square highlighting prevail over legality test
H.G.Muller [Thu, 28 Jan 2016 13:08:28 +0000 (14:08 +0100)]
Let target-square highlighting prevail over legality test

For engine-defined variants the highlight command is accepted even
with legality testing on. If an engine explicitly marks a square as a
legal destination for the picked-up piece, XBoard now always accepts
the move, even when according to its own move generation, even when based
on the engine's supplied 'piece' commands, going there would be illegal,
and legality testing is on.

3 years agoAdd support for Multi-PV Margin
H.G.Muller [Thu, 28 Jan 2016 11:32:12 +0000 (12:32 +0100)]
Add support for Multi-PV Margin

An engine option with a name containing multi, PV and margin is now
recognized, and will lead to printing of a multi-PV header (during
analysis in the Engine Output pane) even in absence of a MultiPV option.
Right-clicking the part of that header between 'fewer' and 'more'
allows tuning of the option value through horizontal mouse moving.
During this tuning the value is displayed in the message field of the
main window. When there is no MultiPV option to specify the number of
PVs, the margin value will be displayed in the header instead of the
number of PVs.

3 years agoFix highlighting in text memos (GTK)
H.G.Muller [Thu, 28 Jan 2016 08:51:37 +0000 (09:51 +0100)]
Fix highlighting in text memos (GTK)

The yellow highlighting as used in Engine Output to indicate the
walked PV, or in Move History to show the current move, was only
working the first time that text would be highlighted. Apparently
to unhighlight it the yellow tag had to be removed, rather than a
white tag applied.

3 years agoFix pasting of moves after starting from position file
H.G.Muller [Wed, 27 Jan 2016 19:14:07 +0000 (20:14 +0100)]
Fix pasting of moves after starting from position file

When starting from a position file boards[0] would be set to the position,
but initialPosition was not. In LoadGame() initialPosition would then be
copied to boards[0] before loading the game, and without FEN or variant
PGN tag would not be corrected.

3 years agoMove Common Engine menu item to Engine menu
H.G.Muller [Wed, 27 Jan 2016 15:49:20 +0000 (16:49 +0100)]
Move Common Engine menu item to Engine menu

The new name of this menu item is 'Common Settings'.

3 years agoFix double-clicks for copying in Edit Position mode
H.G.Muller [Wed, 27 Jan 2016 15:03:04 +0000 (16:03 +0100)]
Fix double-clicks for copying in Edit Position mode

The time measurement between clicks on the same square in Edit Position
mode was seeing presses as well as releases, so that a fast click would
be mistaken for a double click on that square. Only presses should be
judged this way. In addition the piece should be grabbed for dragging
again on the second press, as the first release had ended the drag.

3 years agoFix book probing
H.G.Muller [Wed, 27 Jan 2016 10:22:42 +0000 (11:22 +0100)]
Fix book probing

The Zobrist key for Kings had been inadvertantly changed in the patch that
assigned new keys to the pieces added for Chu Shogi (the 'chu-promoted
series'), because the King would be considered as a chu-promoted piece.

3 years agoFix crash on pasting garbage FEN
H.G.Muller [Wed, 20 Jan 2016 20:30:51 +0000 (21:30 +0100)]
Fix crash on pasting garbage FEN

The auto-size patch had made XBoard crash-prone on pasting pure garbage
with Paste Position, because this would set the number of files or ranks
to values <= 0. Which would cause a floating pont exception during
drawing of the board in XBoard.

3 years agoPrevent changing time control during game (XB)
H.G.Muller [Wed, 20 Jan 2016 19:37:52 +0000 (20:37 +0100)]
Prevent changing time control during game (XB)

An attempt to pop up the Time Control Options dialog during a game
will now be met with an Error popup to say it cannot be done, like
in WinBoard. Before it would reset the game.

3 years agoFix Error popup in Tournament Options
H.G.Muller [Wed, 20 Jan 2016 19:31:34 +0000 (20:31 +0100)]
Fix Error popup in Tournament Options

Since Tournament Options has been assigned its own DialogClass (MasterDlg)
instead of being one of the TransientDlg, the Error popup should really
test if MasterDlg is already up, to know if its parent window is th
main window or not.

3 years agoUse ii in Betza notation for 3rd-rank Pawn push
H.G.Muller [Thu, 14 Jan 2016 18:16:50 +0000 (19:16 +0100)]
Use ii in Betza notation for 3rd-rank Pawn push

The 'i' modifier specifies the move is only for virgin pieces. We now
use 'ii' to indicate a move that can be made when the piece is standing
on the square in front of that were such a piece starts in the initial
setup. This is intended for Pawns that can still be pushed extra far
after they have made a first single step, such as in Wildebeest Chess,
where pawns on 2nd rank can move 1, 2 or 3 squares ahead, and on 3rd rank
can still move 1 or 2 (fmWifmW3iifmW2).

3 years agoFix spurious undo at game start
H.G.Muller [Thu, 14 Jan 2016 15:49:34 +0000 (16:49 +0100)]
Fix spurious undo at game start

For ping-supporting engines the logic in handling unexpected moves
is completely changed. Such moves can occur when an engine think is
interrupted by a 'force' command, which in many engines is only
processed after the think completes naturally with a move, and could
cross such a move anyway in engines that would abort the search promptly.
If the 'force' is the result of the user switching to EditGame mode,
such a move should be undone, if it occurs at game end or start,
the moves can be ignored, as the following 'new' will erase all memory
of them. So in EditGameEvent, if the engine was thinking, a flag is set
to indicate its upcoming move should be undone, and ping is sent after
'force'. The 'pong' response would arrive after the move (if any), and
clears the flag. In all other cases of ping imbalance, incoming moves
are ignored.
 While waiting for a pong after EditGameEvent the user interface is
frozen, and a message is displayed to keep the user quiet.

3 years agoSend ping in EditGameEvent
H.G.Muller [Thu, 14 Jan 2016 13:36:47 +0000 (14:36 +0100)]
Send ping in EditGameEvent

An EditGameEvent can interrupt tinking of an engine, and the engine
might or might not send back a move as a result of that thinking.
To solve this, a "force" command sent to an engine playing white or
black will now be followed by a "ping". (If supported, of course.)

3 years agoFix variant recognition in ICS mode
H.G.Muller [Wed, 13 Jan 2016 21:23:55 +0000 (22:23 +0100)]
Fix variant recognition in ICS mode

The patch that made recognizing variant names of the 'exotic variants'
(i.e. those beyond Shogi) more strict, by requiring an exact match,
had broken variant recognition on the Variant ICS. In that case it
has to act on strings like "ICS unrated xiangqi game", and this now
defaulted to variant normal. The requirement that the variant name
should be the first word of the string is now dropped in ICS mode.

3 years agoLet FENs handle Betza initial rights in castlingless variants
H.G.Muller [Wed, 13 Jan 2016 19:35:36 +0000 (20:35 +0100)]
Let FENs handle Betza initial rights in castlingless variants

When a variant has no castling, or is redefined to have no castling
through a 'piece' command for the King, the FEN castling field of a
written FEN will now contain the file ID of all pieces with initial
('i') moves in their Betza definition. On reading the virginity of
pieces not thus mentioned in the castling field will get their
virginity revoked.

3 years agoFix setting of initial virginity on PGN read
H.G.Muller [Wed, 13 Jan 2016 15:07:48 +0000 (16:07 +0100)]
Fix setting of initial virginity on PGN read

The back-rank virginity bitmaps were not cleared in ParseFEN.

3 years agoIncreas number of engine-defined variants to 15 (WB)
H.G.Muller [Mon, 11 Jan 2016 23:01:31 +0000 (00:01 +0100)]
Increas number of engine-defined variants to 15 (WB)

The number of engine-defined variants that can be selected in WinBoard's
New Variant dialog is increased from 9 to 15 by adding some radio buttons.

3 years agoFix parsing of OO castling when redefined
H.G.Muller [Mon, 11 Jan 2016 21:58:50 +0000 (22:58 +0100)]
Fix parsing of OO castling when redefined

A 'piece' command can redefine the castling stride of a King,
and OO castling for non-Fischer castling is internally encoded as the
King step, and thus has to know this.

3 years agoFix writing FEN castling rights for non-edge 'Rooks'
H.G.Muller [Mon, 11 Jan 2016 21:44:04 +0000 (22:44 +0100)]
Fix writing FEN castling rights for non-edge 'Rooks'

Non-Fischer castling rights were only recognized on FEN writing when
the castling partner was in the corner. Now it can be anywhere on the
correct side of the King.

3 years agoFix reading FEN FRC castling rights when King not on last rank
H.G.Muller [Mon, 11 Jan 2016 20:05:28 +0000 (21:05 +0100)]
Fix reading FEN FRC castling rights when King not on last rank

There was still a hard-coded rank 0 / BOARD_HEIGHT-1 there.

3 years agoFix Makefile for Dragon and minor Lion image
H.G.Muller [Mon, 11 Jan 2016 19:21:05 +0000 (20:21 +0100)]
Fix Makefile for Dragon and minor Lion image

These new pieces were not mentioned in the list of SVG data.

3 years agoAdd Wolf, Camel and Zebra bitmaps to WB
H.G.Muller [Mon, 11 Jan 2016 19:13:41 +0000 (20:13 +0100)]
Add Wolf, Camel and Zebra bitmaps to WB

Built-in images for some of the new pieces are added for WinBoard.
These are Wolf, Camel and Zebra, but also the duplicat Lion is defined.

3 years agoFix white Iron General image
H.G.Muller [Mon, 11 Jan 2016 15:03:24 +0000 (16:03 +0100)]
Fix white Iron General image

The white background of this image was not entirely filled.

3 years agoFlip Unicorn image
H.G.Muller [Mon, 11 Jan 2016 15:01:58 +0000 (16:01 +0100)]
Flip Unicorn image

There are now too many Knight-like piece images, that are too similar.
So the Unicorn from now on will look to the right.

3 years agoAdd white Zebra piece image
H.G.Muller [Mon, 11 Jan 2016 10:39:10 +0000 (11:39 +0100)]
Add white Zebra piece image

This image had not made it into git with the other new pieces.

3 years agoCorrect backup pieces for addition of minor Lion
H.G.Muller [Mon, 11 Jan 2016 10:35:22 +0000 (11:35 +0100)]
Correct backup pieces for addition of minor Lion

The added piece type was not put in the list of backup pieces, so that
the pieces beyond it got out of phase, and no backup for the Prince
would be found, leading to an error popup about missing default pieces.
The error message on this has now been improved to mention the piece
casing the trouble.

3 years agoNever castle when King has other initial moves
H.G.Muller [Sun, 10 Jan 2016 22:54:48 +0000 (23:54 +0100)]
Never castle when King has other initial moves

The ApplyMove routine assumed that any lateral King step of more than
a single square was a castling, and then swung the nearest piece in
that direction around the King. This failed in Grande Acedrex, where
the King can (amongst others) jump 2 squares sideways as an initial move.
So we now first check if the King has its moves redefined in a way that
does specify no castling but does specify other initial moves, and perform
any move as just a simple King move in that case.

3 years agoFix edit command for double-digit ranks
H.G.Muller [Thu, 7 Jan 2016 22:32:47 +0000 (23:32 +0100)]
Fix edit command for double-digit ranks

Rank numbers above 9 were printed as their internal 1-character
representation.

3 years agoAdd duplicat of Lion (and Flying Dragon)
H.G.Muller [Wed, 6 Jan 2016 19:12:10 +0000 (20:12 +0100)]
Add duplicat of Lion (and Flying Dragon)

Two more piece types are added: a duplicat of the Lion in the unpromoted
series, and a Flying Dragon as its promoted counterpart. This Lion does
not suffer from the Lion-trading restriction of Chu Shogi, and can be used
as a general piece. Its pictogram is somewhat smaller than that of the
Chu Shogi Lion.

3 years agoFix typos in winboard.c
H.G.Muller [Sun, 3 Jan 2016 13:09:24 +0000 (14:09 +0100)]
Fix typos in winboard.c

3 years agoLet parsing of O-O castlings pay attention to castling rank
H.G.Muller [Sun, 3 Jan 2016 11:38:18 +0000 (12:38 +0100)]
Let parsing of O-O castlings pay attention to castling rank

The parser assumed O-O and O-O-O castlings would always take place
on the back rank, even when the newly introduced castlingRank array
specified differently.

3 years agoFix FEN castling rank for Knightmate
H.G.Muller [Sun, 3 Jan 2016 11:22:25 +0000 (12:22 +0100)]
Fix FEN castling rank for Knightmate

The newly introduced castlingRank array was not correctly set on reading
a FEN for Knightmate, because it was looking for Kings rather than
Unicorns.

3 years agoReplace Flying Dragon piece image by Gnu
H.G.Muller [Sat, 2 Jan 2016 18:53:35 +0000 (19:53 +0100)]
Replace Flying Dragon piece image by Gnu

How could we not think of this before!

3 years agoAlso define Lance image for Amazon in WB
H.G.Muller [Tue, 29 Dec 2015 22:37:43 +0000 (23:37 +0100)]
Also define Lance image for Amazon in WB

3 years agoUse hoplit helmet for Copper General in Chu Shogi
H.G.Muller [Tue, 29 Dec 2015 22:33:52 +0000 (23:33 +0100)]
Use hoplit helmet for Copper General in Chu Shogi

3 years agoSkip in pieceToChar to Tokin always
H.G.Muller [Tue, 29 Dec 2015 21:46:17 +0000 (22:46 +0100)]
Skip in pieceToChar to Tokin always

The piece after the skip symbol '/' in the pieceToCharTable was
defined as CHUPROMOTED Pawn, which only is a Tokin in variant chu,
but a Cannon in other variants. The intension was to always skip
to Tokin, though.

3 years agoAssign new images to the new pieces
H.G.Muller [Tue, 29 Dec 2015 21:20:24 +0000 (22:20 +0100)]
Assign new images to the new pieces

The 5 new pieces behind Lion, and their chu-promoted versions,
are assigned the 9 new images. The 10th piece gets the Lance image,
to povide a duplicate for this that will not be considered a Pawn
in any variant (so it can be used for Amazon).

3 years agoAdd 2x9 new piece images
H.G.Muller [Tue, 29 Dec 2015 21:13:03 +0000 (22:13 +0100)]
Add 2x9 new piece images

Wolf, Camel, Zebra, Dragon, Wizard, Viking, Iron, Copper and Tower
images are added.

3 years agoFix parsing of pieceToChar strings
H.G.Muller [Tue, 29 Dec 2015 19:43:39 +0000 (20:43 +0100)]
Fix parsing of pieceToChar strings

The skipping to the chu-promoted series was not working. The character
that triggers the skipping has been changed from : to /, because : is
a narrow character potentially useful as ID suffix.

3 years agoFix color of white SVG pieces
H.G.Muller [Tue, 29 Dec 2015 17:02:08 +0000 (18:02 +0100)]
Fix color of white SVG pieces

The newer SVG pieces were bright white, rather than the ivory default color.

3 years agoFix piece command after ID-suffix patch
H.G.Muller [Sat, 19 Dec 2015 10:10:40 +0000 (11:10 +0100)]
Fix piece command after ID-suffix patch

Piece IDs without suffix or & behind them to make them count for both
colors were no longer recognized in 'piece' commands, because there
was a null character behind the extracted piece-name field. And this
null character was seen as occurring in the suffix string.

3 years agoFix bare-king adjudication in Atomic
H.G.Muller [Thu, 17 Dec 2015 11:44:09 +0000 (12:44 +0100)]
Fix bare-king adjudication in Atomic

In Atomic a bare King cannot win, but it can already have won, when
the last piece of that side captured or destroyed the opponent King
when exploding. The win-to-draw correction now recognizes this
situation, so that the win is uphold there.

3 years agoErase old logo before drawing new one (XB)
H.G.Muller [Wed, 2 Dec 2015 11:53:19 +0000 (12:53 +0100)]
Erase old logo before drawing new one (XB)

In case of an invalid or absent logo, the previous logo kept being
displayed. Now the area is painted grai first.
 Also fixes a bug in the expose area.

3 years agoAllow debug output to go to child process (WB)
H.G.Muller [Fri, 27 Nov 2015 20:01:22 +0000 (21:01 +0100)]
Allow debug output to go to child process (WB)

A -debugfile name starting with tree slashes will not be interpreted
as a file to write on, but (after stripping the slashes) as an executable
to run, and send the output to. This could be used by an external
broadcasting agaent.

3 years agoFix DarkSquare bug in piece counting
H.G.Muller [Fri, 27 Nov 2015 16:55:13 +0000 (17:55 +0100)]
Fix DarkSquare bug in piece counting

When counting pieces for adjudication purposes, a DarkSquare on the
board caused an out-of-bounds access to the counter array.

3 years agoFix piece commands for suffixed piece IDs
H.G.Muller [Thu, 26 Nov 2015 21:12:22 +0000 (22:12 +0100)]
Fix piece commands for suffixed piece IDs

Due to signed-/unsigned-char problems piece IDs suffixed with ' or !
would not be recognized. In addition a & behind it would not be correctly
recognized diue to the alignment problem the suffix character caused.
Both issues are fixed now.

3 years agoAllow skipping over black squares
H.G.Muller [Wed, 25 Nov 2015 18:26:54 +0000 (19:26 +0100)]
Allow skipping over black squares

'Skip pieces', defined with a 'j' prefix on a slider(asopposedto rider),
which skip the first square(s)of their path irrespective of occupation
should also be able to skip over holes in the board.

3 years agoDisplay new user logo when username is entered
H.G.Muller [Tue, 24 Nov 2015 19:10:27 +0000 (20:10 +0100)]
Display new user logo when username is entered

When the Edit->Enter Username type-in was used, a new user logo was
loaded, but not immediately displayed. While the old user logo could
already be up, during a man-machine game. Now it is also immediately
displayed.

3 years agoTry to load bitmaps for all pieces (WB)
H.G.Muller [Tue, 24 Nov 2015 19:00:16 +0000 (20:00 +0100)]
Try to load bitmaps for all pieces (WB)

WinBoard only contains build-in bitmaps for the first 22 piece types,
and also only associated filenames with those that can be used to
provide external images. Now a new naming convention for piece images
is introduced, like piece22_49s.bmp for a 49x49 bitmap for solid piece 22.
All pieces that were not loaded according to the old convention now
try to find images with this naming convention if a -pieceImageDirectory
is defined.

3 years agoFix sortng of mate scores
H.G.Muller [Tue, 20 Oct 2015 15:30:34 +0000 (17:30 +0200)]
Fix sortng of mate scores

With the new convention for mate scores (100000+DTM), lower numbers
represent faster mates and thus better scores, and the sorting of PVs
in the Engine Output window now takes that into account.

3 years agoPrint mate scores as #N in message field
H.G.Muller [Tue, 20 Oct 2015 15:22:53 +0000 (17:22 +0200)]
Print mate scores as #N in message field

3 years agoStart implementing EPD test suites
H.G.Muller [Tue, 20 Oct 2015 13:05:50 +0000 (15:05 +0200)]
Start implementing EPD test suites

An option -epd is added that can be used in combination with matchMode
to let XBoard do a test suite on a file of EPDs with a "bm" qualifier.
It would record the time to solution. Reporting still has to be implemented;
as yet unsolved problem is that currently black-to-move EPDs would
be presented to the second engine.