Merge branch 'master' into v4.6.x

Conflicts:
po/uk.po -- used latest version to resolve merge conflict

Duh! Initialise the argument to sscanf() inside the loop, so it always gets a known value when sscanf() fails.

Give numeric options the value 0 if a non-numeric text is entered (one more place). Before the value ended up undefined.

Give numeric options the value 0 if a non-numeric text is entered. Before the value ended up undefined.

Fix possible crash on 64-bit systems when copying game or position

The two convertproc's for X selections caused reads out of bounds on 64-bit systems.
I hope that this is the right solution. I am not experienced with X stuff, but it
follows that I have seen in other programs, testing shows that copying still works,
and the reading out of bounds have disappearinged with this patch.

Fix a possible crash in the file browser

If the OK button was selected in the file browser when no file was selected
there was a read out of bounds which could crash xboard

Fix a memory leak in Sound Options

Fix crash when selecting user soundfile due to free() of not malloc()'ed memory

Make GenericPopUp() more readable by using more named flags instead of numerals

Defines the flags SAME_ROW (value 1) and NO_OK (value 2) for use with buttons and endmarks.
There should no functional change.
Also add a code comment where a condition depends on an uninitialised value. There is no direct harm, as both
branches do the same (that is nothing) when the value is uninitialised.

Add NO_GETTEXT flag for combobox. Fix bug #34991

Define NO_GETTEXT flag to avoid calling gettext() for comboboxes, and define COMBO_CALLBACK
flag to call comboCallback() when an item is selected in a combobox (no new code for this,
just a symbolic value instead of a number). The flags are used for the min field of struct Option.
NO_GETTEXT is used for the Select engine combobixes in the Match Options and Load engine windows.

only translate entries in CreateComboPopup if strlen>0; fixes #34991

fix for repeating key issue, with this allissues this bug should be closed; fixes #35000

also don't translate selection in ComboSelect; fixes #34991

make entries in CreateComboPopup not translateable; fixes #34991

fix translation for engine list; fixes #34991

* click on browse button should be recognized in translated versions
* width of button should auto-scale

change keybindings that don't use ctrl, make MoveTypeInProc ctrl aware; fixes #35000

two keybindings were using keys without modifiers, adjusted these to use the ctrl key.
Furthermore when typing in moves we need to check if ctrl was pressed and ignore those inputs.

Also added documentation for these keybindings.

replaced unicode character for "'" in xboard.texi

removed Iconify function. Should be handled by the window manager.

marked an error messages for translation

translation: updated uk.po

fixed size of XBoard icon to 48x48

the icon gets installed in the 48x48 directory, but was >200 pixel wide. This gave a warning during rpm creation.

updated version numnber in pot file

new developer release

Merge branch 'master' into v4.6.x

updated pot file

marked a lot of new strings for translation

added new files to po/POTFILES.in

marked more strings for translation

translation: added danish translation

Change encoding of seirawan ICS gating-castlings

In stead of using upper/lower-case promoChar, the ICS now uses
RxK notation, like WinBoard writes in SAN, for better consistency.

Put promo-suffix on ICS move in variant seirawan castlings

The distinction upper-case / lowercase determines gating at R / K

Refrain from making unnecessary resize X request

The window resizing code in InitDrawingSizes is suppressed when no
change is needed. Similarly, changing piece bitmaps is suppressed when
we stay in the same variant (and InitDrawingSizes is called e.g. because
of chaging lineGap through the Board dialog).

Fix parsing of O-O castling in variant janus

The new parser did not realize it meant e1i1 rather than wild e1h1.

Show 50-move counter also when observing ICS games

To not cheat 50-move display was suppressed in ICS mode,
but for observing a game this made no sense.

Ignore ICS game starts when already in game

The advance switching of the variant on game-start messages was
disruptive for users with gin=1, which also sends such messages
for other games than your own. For now fixed by only reacting on
them when idle (detected by invalid game number).

Fix WinBoard  clock-click bug

Clicking on the clock in WinBoard also called the LeftClick routine
for handling clicks on the board. This led to the from-square being set
to some inalid value, so that the next genuine board click was interpreted
as a to-click, with undesired effects, especially with legality testing off
or in EditPosition mode, where weird things are accepted.

Add feature-override options

A few (dangerous) options are added to allow a user to override
engine features, or change their defaults, by supplying a string
with feature commands (e.g. "sigint=0") which tricks XBoard into
thinking the engine sent that. There is a -featureDefaults which is
processed before the engine features, and -first/secondFeatures
which is parsed just before engine initialization.
  This goes horribly wrong when the user wouldtry silly things like
including 'option' or 'done' features!

Merge branch 'master' into v4.6.x

translation: updated uk.po

translation: updated uk.po

Fix handling of -secondOptions option

The -firstOptions and -secondOptions settigs were not swapped when
-singleEngieList was true, so the second engine's options would never
be set in that case, but used for the first engine.

Accept setup command in variant fairy even with legality testing on

Variant fairy does not really have a defined opening position, but lots
of pieces defined, so we might want to play with legality checking on,
but still rely on the engine for the opening position

Set pieceToCharTable by setup command even when ignoring FEN

When the user starts from a set-up position, the egines default
openingposition has to be ignored, but the pieceToCharTable would
still be relevant.

Change default sweep-promotion choicein queenless variants

The initial scanning of the sweep piece until a defined piece is found
is now in the increasing direction (still starting at Queen), so that
the fairy pieces are suggested before R, B and N.

Clear board markers when moving to other position

Now the markers from -showTargetSquares would stay on after a first click
for the benefit of a click-click move, they would also stay on when not
finishing the move, but operating the < > buttons in stead to call up
other positions. ForwardInner and BackwardInner now clear the markers,
like they also cleared highlights.

new developer release

updated pot file and send to translation project

updated Changelog, NEWS, etc.

Fix two warnings

Don't adjust clock on right-click in EditGame mode WB

This was a change that was already done for the left-clicks, but somehow
never made it togit for the right-clicks

Change chase message in .lng files

Let perpetual-chase message mention square

In stead of a boolean, PerpetualChase now returns the square the chased
piece ends on (encoded as an int), and on adjudication it will be printed.

Add mechanism to translate variable messages in WinBoard

A message prefixed with % in the lng file will be considered a match
if it matches the start of the text to be printed, and the remaining
part of the latter will remain untranslated. This was needed to allow
translation of the perpetual change message, now it indicates the square
of the chased piece.

Use other interpretation of Asia rules for chase detection

The detection of perpetual chases now ignores pre-existing checks.

Fix crash on loading garbage game file

The code to cache games was not resistant to moves before a valid
game start, or a game filewithout any games at all.

Incorporate variant type in book hash key

This prevents booksforone variant being used for another, and allows
multi-variant books.

Update language.txt file

Update Dutch lng file

Fix Load Options dialog WB

Defined fastkeys; reordered elements so that fastkeys focus edit they
are intended for. Changed some of the more obscure texts. Corrected
list of transatble dialog items, adding a few missing ones.

Fix some warnings

Prototypes were added, ormoved from backend.c to backend.h.

Put 'Load Next' button back in WB Game List

The '>' button for loading the next (filtered) game is re-instated in the
WinBoard GameList. The main window has now accelerator keys Ctrl+Up and
Ctrl+Down for loading the previous and next filtered game.

Speed up position search and consider side to move

The position search is made to pay attention to the side to move,
which produces a speedup, because we only have to compare half the
game positions when looking for an exact position match. An addition
we now keep track of the total number of pieces, and abandon a game when
it drops below the number of pieces in the position we seek.

Dynamically allocate move cache

A small statically declared cache good for 250k moves (~3200 games)
will be expanded in steps of a factor 8 when it overflows.

Translate search strings

Implement flipped search

Debug position search cache

profile

Translate search-mode strings

Quickscan

Fix duplicate loading of second engine in tourneys

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

Fix PV walking with -fSAN

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

Refine clock adjusting

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

Fix crash on adding items near book end

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

Print message on wrong use of Clone Tourney

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

Fix invalid combobox entry WB

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

Fix crash on loading garbage game file

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

Add control to set -discourageOwnBooks WB

only require 0.17 of gettext

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

Add option -discourageOwnBooks

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

Reset initString and computerString on engine load

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

Add Romanian lng file

Reactivate Falcon Chess

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

Update texi file for new features

Update texi file for position search

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

Remove paragraph about .Xresources from texi file

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

Add Dutch WinBoard translation

Prevent Edit Book window from stealing focus (WB)

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

Make book-window update part of HistorySet

Fix position loading in tourneys

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

Fix resetting -fSAN on engine change

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

Make too-long game non-fatal

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

Keep book file open

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

Suppress playing of book moves with weight 0

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

added NEWS for release of 4.5.3

better contrast for XBoard icon on a dark background

translationproject.org: updated Ukrainian translation

updated pot file and send to translation project

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

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

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

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

Fix LoadGameOrPosition starting up engine

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

Clear fSAN option before new engine load

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

Fix some warnings (prototypes)