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.

Let WinBoard start in its installation folder

The current directory in WinBoard was not automatically the same as the
installDir returned by SearchPath. Especially not if used without shortcut
by file association.

Fix warning in WinBoard

Fix expose of to-square with grid off

The code relied on drawHighlight to request an expose evet for the entire
square, but when lineGap = 0, drawHighlight is a no-op.

Fix library order

On OS X dialog windows could not get focus if -lXaw appeared after
-lX11 -lXt in the linker command. This was cured by moving
@FRONTEND_FLAGS@ (which expands to -lXaw) first in the LDADD parameter
of the automake file.

Preserve PGN tags when loading engine

Switching to EditGame mode would normally erase the tags, but when analyzing
a loaded game this would be undesirable. When the engine is playing, keeping
the tags is dubious, but it is not obvious what to put in a new name tag,
so we leave that for now.

Preserve mode on engine loading (sometimes)

That the mode always switched to EditGame after engine loading was
particularly annoying at the start of a game (where leaving BeginningOfGame
mode would disablethe Time Control dialog, and the engine would not
automatically reply if you start playing) and AnalyzeMode. These two
modes are now preserved, BeginningOfGame by refraining from calling
EditGameEvent() and sending 'force' to the new engine, and AnalyzeMode
be calling AnalyzeModeEvent() after the engine is loaded and repositioned.

Catch unknown engine in tourney games

When an XBoard instance is playing on a tourney it can happen that it is
using a settings file with an engine list not yet containing a certain
participant, because that participant was just installed in the instance
that created the tourney file, but it did not save those modifications yet.
This error is now caught, and leads to an error popup and stopping of the
tourney play. This error was made non-fatal to allow the user to decide
if he wants to quit with or without saving the settings (which, after all,
does not seem up-to-date).

Adjudicate pawn-drop mate as loss in Shogi

Mating with a Pawn drop is illegal in Shogi, but rather than burdoning
the legality-testing by this complex rule, we simply invert the result
for such a mate. After all, Shogi customs are such that everything illegal
leads to an immediate loss.

Adjudicate perpetual checks as loss also in Shogi

The Xiangqi code for recognizing perpetual checks is now also activated for
Shogi, but in this case we bail out before checking for perpetual chases.
The mini-Shogi case is then recognized as a win for gote.

Stalemate is a win in Shogi

In regular Shogi this never occurs, but in the mini variants it happens
often enough.

Fix engine timeout problem in match mode

Reload of a reuse=0 engine would cause a done=1 event which would be
abort the feature timeout of the other engine in the first game.

Enforce -popupMoveErrors

One move error was using DisplayError(), in stead of DisplayMoveError(),
and thus did not obey the -popupMoveErrors option.

Update texi file

Fix some warnings

Updated German translation

Do delayed board draw also with -stickyWindows false

Only CoDrag needs to be suppressed on a configure event with
-stickyWindows false, not the board redraw.

Let tellothers command add comment to PGN in local mode

Allow use of ~ in pieceToChar for shadow pieces in any variant

The use of shadow-pieces for promoted Pawns was limited to Crazyhouse and
Bughouse. But this is not necessary, and in fact undesirable, as it made
it impossible to implement Crazyhouse960, which combines shadow promotions
with Fischer castling (only available in variant fischerandom)s, and thus
has to be played as 8x8+5_fischerandom, using the Crazyhouse -pieceToChaTable.

Resend engine-defined options after reuse=0 reload

When the engine process is re-spawned for a new game under -xreuse setting,
any alteration of engine settings brought about through the Engine Settings
dialogs would be lost, and revert to the engine's default. This is now fixed
by re-sending all options (known from the previous load of the same engine)
with the current settings immediately after "protover 2", and ignoring the
engine's option features. Rather than clearing the option list, and
redefining it from the option features.

Implement aborting of games on engine request.

A new game-end command is added. For bakward compatibility it is disguised
as an old one, namely "1/2-1/2 {Engine Abort Request}". These resultDetails
are recognized as special, and rather than ending the game in a draw (which
GUIs not implementing this), the game will be completely 'expunged' from
the match or tourney it is part of, leaving no trace in the saveGameFile.

Store some more tourney params in tourney file

The hash and threads settings, as well as ponder and book options are now
also saved in the tourney file, so that tournaments can be played in
parallel where these parameters differ, and no mess up can occur on
restarting a touney after doing something else.

Do not automatically save aborted games in tourney PGN

When the game result is 'unfinished', we prompt for a filename rather
then using the specified saveGameFile if a tourney is in progress.

fix configure bug that showed up on OS X (couldn't find X11/Dialog.h)

X_CFLAGS (set by AC_PATH_XTRA) was not used in Makefile.am. It was used in tests for X11/Dialog.h, etc. so configure worked, but make didn't.

Updated Ukrainian translations

removed trailing whitespace