Fix -zippyVariants option

Due to the change of cps->variants from (char []) to (char *), copying
zippyVariants to it on behalf of v1 engines would no longer work. This
is fixed by using ASSIGN() in stead of safeStrCpy().

Fix initial board sizing WB

The initial board size was based on the assumption the board would be 8x8.
This could push auxiliary windows off screen if the board was smaller,
after which EnsureOnScreen would undock them and park them in the upper-
right corner before the board assumed its true size. InitPos() is now called
before the initial sizing to et the board format correctly, InitDrawingSizes()
had to be made resitant to calling before a board size was picked for this
(because InitPos() might call that too).
  ActivateTheme() now uses the new board size, not the previous one (as was
indicated by size -2), so that -boardSize specifications inside a theme
will take effect.
  Board size petite should not be forbidden for Shogi; all pieces exist there.

Fix grabbing of selected piece

A second static click on a selected piece should deselect it, but an attempt
to drag an already selected piece should not 'bounce off'. It was already
possible to move a selected piece to another square, but the dragging was
not animated. This is now fixed, by only skipping the code to start dragging
when the second click is part of the only-move double-click for only capture.
(Before it was skipped on any second click.)

Allow entry of negative numbers in spin control (WB)

Windows numeric controls do not accept negative numbers, so in case the
range of a spin option can go negative, the style ES_NUMBER should not be
set on the control.

Allow drops / promotions/ deferrals to be edited into book

For no real reason only NormalMove would be accepted when parsing the list
of book moves.

Fix GUI book after setup position

The routine to send move to the engine would automatically append a 'go'
is a preious setboard had put the engine in force mode, but this should
not happen on a book hit, where the engine should even be put in force mode
if it was not yet there.

Fix book creation

Due to duplicate use of the same file pointer variable the game file would be
closed before any games were read from it.

Make PGN parser immune to unprotected time stamps

The parser choked on time stamps of the form dd:dd or dd:dd:dd if one
of the fields happened to be 00, because it wouldmistake that for a
non-compliant king-side castling. By excluding this interpretation when
the 00 is immediately preceded or followed by ':' this is now prevented.

updated po files for new release (make distcheck)

new version number for release 4.7.1

updated Changelog, NEWS, etc.

Update zippy.README

It is explained how to generate multi-line commands for -zipyGameEnd.

Fix Xaw file browser

The 'next page' entry of the Xaw file browser contained a \177 character
to keep at sorted at the bottom. This caused trouble for gettext.
Now the sorting range is adapted so the 'next page' is never part of it.
  More seriously, the listbox fill code had a wrong cast when testing for
an empty listbox on the first element, so that it only tested the low byte
of the pointer, in stead of the entire pointer for NULLness. As a result
the contents of the listboxes could suddenly disappear when even when it
was non-empty, because the first filename was allocated at an unlucky address.

Make Chat Box window obey -topLevel option

It was always popping up as a dialog of the main board, covering it.
One could argue it should always be to-level.

Fix scrolling of Chat Box

Seems the problem is that scroll_to_iter does not work until after some time,
needed to calculate line heights. Use scroll_to_mark instead, after clumsily
adding a text_mark.

Fix GTK SetInsertPos

SetInsertPos() was not yet ported to GTK, with as a result that the cursor
was positioned at the start of a line recalled in the ICS Input Box,
rather than at the end. The code also called SetInsertPos to force scrolling
to that point in the text (which in Xaw is an automatic side effect). So for
multi-line text edits this function calls ScrollToCursor(). For some reason
this does not work when replacing texts in the Chat Box memo, when switching
chat, although it does work in the same memo when a line is added to the
existing text.

Print score of final position in Analyze Game

The score/depth of the final position of the analyed game is added
(as an explicit comment) behind the PV of the forelast position.
To make this work, a new 'addBraces' mode (3) had to be added to
AppendComment, which suppresses stripping of the score/depth from
the comment.

Fix horrible bug in reading scores from PGN

Reading scores like -0.94 flipped the sign, because the integer part
-0 would read as 0, and thus was no reason for sign flipping.

Fix disappearence of pieces that were moved illegally

Dragging damages the from-square, but it is not marked as such, because
selective board redraws to update highlights and such would draw the piece
that is really still there, making it flash on the from-square before
the position after the move is drawn. This, however, backfires when the
move was illegal, as then there is no draw of a new position, and the
from-square stays empty. For now this is solved by making DisplayMoveError()
order a total redraw. (As this is rare, only done in case of an error,
and might even throw up a pop-up, the performance cost of this is acceptable.)

Fix quoting of book name in tourney file

The argument of -polyglotBook was written to the tourney file unquoted,
which effectively made it disappear if it was an empty or blank string.

Provide compatibility with Alien Edition setup command

The setup command of the Alien Edition has a board-size specifier
(possibly as prefix on a parent-variant name) between pieceToCharTable
and FEN. The regular version would choke on that, precluding engines
designed for the Alien protocol extension to have their setup processed
when running under regular XBoard. This patch makes it skip the extra info.

Fix saving of analyzed game

Anotated games after batch-mode Analyze Game were not auto-saved,
because the GameEnds() call supposed to do that was called with a NULL
argument for resultDetails. Now it is called with the resultDetails as
pre-existing in the gameInfo, or an empty string if none exist. This bug
was masked by the fact that in standard game ends (mate, etc.) the
analyzing engine would declare game end and supply a message.
 Also makes a distinction between Analyze Game and regular Analyze mode
when resuming analysis after loading a game; only the former now would
trigger a batch analysis.

Fix spurious popup after batch-mode Analyze Game

A batch analysis of games was not properly terminated, because the game
mode was tested after it had already been changed. No oldGameMode is
tested in stead.

Let XBoard -autoBox option also affect move type-in

Fix GTK box popup

Key values > 255 are used for PgUp, PgDown etc., so to pop up only on
printables needs a two-sided test.

Fix fatal error on unsupported board size

When a board size unsupported by the engine was requested through the
New Variant dialog, this was discovered while the dialog was still up,
causing the error message to pop up as a child of the New Variant dialog,
so that it would immediately be taken down as the latter closed.
As a result it could not be confirmed, so that XBoard would not exit, and
was left in a pathologic state (ignoring window-close). This is fixed
by performing Reset (which detects the error) after PopDown of New Variant.

Accept setup command for non-standard board size

The code to test if board-size is standard for the current variant
has been moved to a subroutine, to allow its calling when receiving
a "setup" command from the engine.

Fix buffer overflow in feature parsing

String features (variants, egt, myname and option) relied on a buf[MSG_SIZ]
for processing their value. The Nebiyu engine had combobox options that
required more than that. All string features are now stored in allocated
memory rather than in static arrays, and StringFeature allocates a buffer
of sufficient size for them. Only limitation now is the low-level input
buffer in the InputSource threads, whih is a static buffer of 4096
(=INPUT_SOURCE_BUF_SIZE) characters.

Fix vertical sizing of GTK board

The clipping of the boad at the bottom was due to resizing of the clock
height when we first write it with a big font. This patch delays calculation
of the final size until after the clock height changes.

Updated German translation

Updated Ukrainian translations

Fix of argument error

Oops. Pevious patch for skipping unknown multi-line arguments
was still buggy.

Let popping up of WinBoard chatbox for channel open it

WinBoard now issues a +ch command for a channel when you pop up a chatbox
for that channel. A work-around is added for the fact that the public
FICS seems to ignore the first such command.

Make skipping of unknown option smarter

To make XBoard better resistant to the presence of disbanded or future
options in its settings file, encountering those is no longer a fatal error.
But the error recovery only consisted of skipping the rest of the line.
This fails badly on string options with a multi-line value, which is
enclosed by {braces}. We now detect that case, to skip to } instead.

Add -backupSettingsFile option

For better configurability of WinBoard, an option is added to optionally
read a settings file, namely if no other settings file (other than the
master file) was read so far. This can be used to import settings from
a previous version, without destroying its settings file.

Repair WinBoard compile error

__GITVERSION was not defined in WinBoard, as it was only defined with a
compiler flag by ./configure, which WinBoard does not use.

Work-around for Xt selection bug

In some Xt implementations calling extend-end with 0 params causes a crash,
although the mmanual specifies this should default to a call with 'PRIMARY'.
The default selection is now explicitly mentioned in all extend-end calls.

fix bug #38401: xboard.texi doesn't build with texinfo-5.0 (tiny change)

xboard-4.7.0's documentation is not buildable with texinfo-5.0, it fails with a lot of error messages from makeinfo like
./xboard.texi:1026: @itemx must follow @item
makeinfo seems right, @itemx is not used as described in the texinfo manual.
Attached patch puts each item of the "Options" menu in it's own @subsection and formats the description of each item as a @table as neccessary. This way, xboard builds again and the manual doesn't look too bad.

updated po/pot files

Merge branch 'v4.7.x' to get DIFFSTAT, Changelog, etc up to date in master

Conflicts:
configure.ac
winboard/config.h

new version number for developer release

updated po/pot files

updated Changelog, NEWS, etc.

Merge branch 'master' into v4.7.x

Try to make life more bearable in Xaw menus

The alignment still sucks, but not as much as before.

Fix -topLevel option

This option was copletely ignored, after the refactorig.

Fix testing for valid window placement Xaw

Suppress printing of status line in dual-board mode

Fix sizing of slave board GTK

For lack of a known method to specify a default size for Graph widgets,
we have to use the same system as for the main board: create with 0x0 size,
and measure the size of the outer window to determine margins.
After every popup we then resize to the required board size plus these
margins. (Awful...)

Better handling of undefined window parameters

Repositioning and resizing a newly created dialog was based on the width
parameter of the stored sizing info being valid. Now it is separately
tested if the position is valid, so that sizing and moving can be done
independently. This allows us to store position info, but not size info
for the slave board, so that it automatically will use the size of the
main board when first opened.

Limit debug print to debug mode

Fix repositioning of GTK windows

After 'hide' and 'show' GTK would not position the window in the same place,
but center it on the parent. Now we reposition the window after 'show' based
on the WindowPlacement data (if available).

Remember window params of slave board

The slave board now has been assigned a WindowPlacement, which is also
saved in the settings file, through new options -slaveX/Y/W/H.

Fix repairing of arrow damage

The '2' bit in damage[][] to indicate the arrow was erroneously cleared,
rather than kept.

fix OK-response in gtk dialogs, see c7f8df124

the fix in c7f8df124 needs to be applied to both of those if cases

removed two more translations from debug output

fixed segfault of g_markup_printf_escaped which needs utf-8 strings

defined text-domain as utf-8 solved this

fixed gtk-warning

removed some translation calls for messages in the debug log

add keyboard shortcuts back into Xaw version

in debug mode also print the git-version if available during build

Print game-list timing messages only in debug mode

Fix title of top-level windows

Cure GTK warning in top-level windows

There is no OK button in top-level windows, so the 'response' signal is
not defined, and should not be connected.

Fix graying of Revert menu item

The new code was checkmarking it in stead of enabling it!

Add -onlyOwnGames option

This suppresses auto-saving of ICS observed games.

Allow display of 50-move counter in zippy mode

No chance it could be considered cheating there.

Add new vertical pixel fudge

With the new GTK window type the board is 13 pixels too small.

Don't add PV moves on board clicking in AnalyzeMode

Playing PV moves by right-clicking the PV was only intended for clicks
in the engine Output window, selecting from multi-PV analysis output,
where by default it plays only the first move. PV walkig by right-clicking
the board, however, would start the walk at the end even in AnalyzeMode,
and thus add the entire PV. Which is probably not useful. So adding moves
in that case has now completely been suppressed, so it can be used for
harmless viewing of the latest PV.
 The behavior has also been made subject to a persistent Boolean option
-appendPV, configured to true, with a checkbox in General Options.
 A checkbox has also been added there for controlling the -variations option,
which is now also configured to true.

Remove checkbox for 'Move Sound'

This function is better controlled from the Sounds dialog.

Make switching between board windows absolute

Selecting the window in which to draw the board when the slave board for
-dualBoard was up was done by a toggle. For unknown reason this could cause
a situation where the drawing took place in the other window than the
intended one. The code is made more robust by explicitly specifying which
window to draw in, based on the partnerUp flag.

Warn about experimental nature of dual board

The message field, which for now is unused, now contains a warning message.

add close buttons to gtk windows

use windows instead of dialogs for toplevel elements

keyboard accelerators for both front ends.

moved them out of the string definition, so that gtk can add them automatically.
Also wrote a function that loads them for X11.

TODO: add them again when creating X11 menu names

added rotated shogi pieces for -flipback option and moved them to the themes directory

Merge branch 'master' into v4.7.x

add test for pkg-config

exit configure in case we can't find it.

Only perform e.p. capture if there are rights

The heuristic for e.p. capture was such that any non-forward move with
a Pawn to an empty square would delete the piece behind its new location
(Xiangqi exempted). This is now limited to cases where EP_STATUS indicates
that the deleted piece was a Pawn that performed a double-push on the
previous move. (Or if there is unknown EP_STATUS.)
 This makes XBoard more generally suitable for use with variants that have
non-conventional Pawn moves (with legality testing off). To guarantee this
will not have any unexpected effects in normal Chess, this patch will only
be active if legality testing is off.

Unreserve tourney game on exit during engine load

While waiting for the feature timeout we are not yet in TwoMachines mode,
so quitting XBoard at that stage would not unreserve the upcoming game.
A global flag 'startingEngine' similar to 'waitingForGame' now signals
this condition.

Better fix of feature timeout

The previous fix of the 2nd engine's done=1 aborting the 1st engine's
feature timeout by starting the game caused problems with v1 engines,
or other engines that would never send done=1. It is now fixed by
explicitly testing for a pending feature timeout after resurrecting
the 1st engine, rather than having TwoMachinesEventIfReady wait for
initDone to be set by reception of done=1.
 Also refrain from freezing the UI between match games, as in TwoMachines
mode the UI is mostly disabled anyway.

Connect Ctrl key in WinBoard

The Control key is now checked during move entry, for excluding moves
during analysis, or for copying pieces when editing a positions. But the
WB front-end did not read it out yet!

Fix Makefile for install of Xiangqi pieces

Add Xiangqi piece images to project

Let the images install in themes/xiangqi.

Revive -flipBlack option

The -flipBlack option in the cairo version is now implemented the same
way as it was in XBoard, swapping white and black pieces in filpView mode.
This requires a set black piece images that are the upside-down versions
of the white ones.

Merge remote-tracking branch 'origin/master' into v4.7.x

Conflicts:
NEWS

Remove empty-square SVG images from project

Add desktop stuff for .xop MIME type.

Icons and desktop file were missing, as was the automake code to package them.

Update NEWS file

Add boolean -autoBox option

This option controls the automatic popup of the ICS Input Box on typing
a printable character, which was annoing to some. The option can be
set from the ICS Options dialog.

Draw both coords in a1

The lower-left board square needed two inscriptions, but DrawOneSquare()
only allowed one, so that the file ID would not be written. Now two strings
can be passed, and when align > 1 to indicate the alignment of the top
string, it is still checked if a bottom string is present as well, which
then is drawn with alignment 1 (= bottom right).
 The rank coord or piece count can now also be double digit.

updated po files for new release (make distcheck)

updated Changelog, NEWS, etc.

new version number for release of 4.7.0

Let initial setting of Twice checkbox reflect current state

The Twice checkbox in the Tournament dialog of WinBoard was by default
starting unticked. It is more convenient to have it preserve the current
state, however, based on the -lpi or -lgi value. Like the auto-incremet
checkbox already did.

Put GTK warning in about-box

Fix warnings of build server

Fix GTK error auto-raising board

The variable mainwindow is not set in the gtk2 version. Unfortunately
it is still used to determine default board size...

Update Dutch WB translation

Add some new strings to WB translation template

Assign shortcut char to WB menu item

The new Save games as Book item did not have a shortcut yet.