Implement -viewerOptions

A mechanism is added to allow the user to configure persistently if the
game viewer that starts on clicking PGN files will use -ncp mode, or
load an egine for analysis (or, in fact, do anything). This by introducing
a persistent string option -viewerOptions, which will be parsed as a
command line after the real command line when the option -viewer was present.

Fix crash in auto-comment

Trailing linefeeds in AppendComment were not properly stripped off,
leading to a mismatch between the allocted and used memory, and thus
possible buffer overruns.

Fix crash at match end

The -afterTourney option had broken old-style matches, because the ranking
string that was tested for "busy" is not set there at all, and remains a
NULL pointer, which strcmp doesn't like.

Add desktop files for PGN viewer and tournament player

The desktop files define startup commands for handling PGN, FEN and
tourney files, calling xboard with the proper command-line options.

Add xml file for defining mime types

The file xboard.xml defines application/x-chess-pgn, aplication/x-chess-fen
and application/x-xboard-trn as mime types for *.pgn, *.fen and *.trn files,
so these files can be associated with XBoard in various modes.

Add icons for PGN and tourney files

Icons are provided in two sizes: 32x32 and 48x48.

Clear square markers on new game

Now that the markers can stay on after a first click, they need to be
explicitly cleared when moving to another board, including a new game.

Abandon single-letter menus in tinyLayout

Now that the items on the main menu bar are clipped, it iso longer
necessary to abbreviate them.

Implement peeking previous position in WinBoard

The backspace key is used to temporarily display the previous position.

Fix parsing of faulty PGN tags

A closing bracket could be missed in a malformed tag

Fix parser for variant seirawan

The new parser didnot understand gating suffixes with /, and did not
allow any suffix at all on SAN castling moves.

Try to not confuse ICS rating adustments as shouts

An explicit pattern match for the rating adjustment is added before
matching the shouts.

Describe -overrideLineGap option in texi file

Use <Enter> in stead of Ctrl+. for peeking last move

I commented out the explicit detection of the Ctrl key, as this
is not needed when there are no 'unnatural' Ctrl keystrokes used
for binding.

Next try for menu-bar sizing

This time request the width needed to just fit everything, and then start
shaving off pixels from the currently largest until the entire thing fits.

Clip texts on menu bar

The width of the items on the main menu bar is restricted so the bar
will never exceed board width.

Fix format in temporary timing printf

Don't test drops as mate evasions in variant seirawan

Fix ICS castling rights

Copying the castling rights together with CopyBoard seems to have
broken handling of ICS rights, where ParseBoard12 prepares a board
in a local variable, an copies it before parsing the rights.

Fix initial holdings ICS seirawan games

Fix sending of S-Chess gating moves to ICS

Normal moves did not get the gating suffix sent.

Fix menu grayout after illegal move

TheThinkingEnables were not reset to user state after the engine rejected
the entered move as illegal.

Allow setting of holdings with edit command

For the benefit of Fairy-Max' Seirawan Chess implementation, the edit command
no also sends holdings contents (if the engine sent feature xedit=1), encoded
as pieceType + '@' + holdingsCount (if the count is non-zero).

Fix out-of-bounds access in check test

Thepath to make the check test work with drops had broken the test
after null move (both have rf<0), which led to corruption of the high
word of the black clock.

Fix legality of Spartan promotion to king

In the check test promotion to King is only allowed if there are not
yet 2 kings.

Fix switching back from mono-mode XB

Suppressing unnecesary X-server calls in InitDrawingSizes had broken
switching back from monoMode, because the animVars have to be recreated
for that, and this was only done when the variant changed.

Fix showTargetSquares in mono-mode XB

Make the marker dots black (captures) and white, and draw a circle
in the opposite color around them in monoMode.

Fix arrow highlighting in mono-mode XB

Draw a line around it in darkSquare color in monoMode.

Fix alignment of generic browse button XBoard

The width ofthe browse button is now read back after creating it,
and subtracted from the preceding text edit.

Add -afterTourney option

Only the instance pinting the tourney result will execute this.

Fix buffer possible overflow when writings tags

PGNTagsStatic() could overflow its buffer and is removed. PGNTags() do the work instead of calling it.
It starts by calculation the required buffer size and then allocate the buffer.
PrintPGNTags() now prints directly to the file without having a buffer.

One more string marked for translation

And it is moved inside the if statement that uses it.

Fix checking of return value from snprintf()

snprintf() returns the needed space excluding the trailing 0 byte, so
it has truncated if the returned valued is greater than or EQUAL TO the buffersize

Fix unportable memory initialisation

Memory sat to zero by calloc() cannot safely be assumed to have the pointer value NULL

Add 2 strings for translation

Remove the last 2 compile warnings in gamelist.h

Add #include "moves.h" to gamelist.c for def. of CopyBoard()

Definition of TimeMark moved from 3 c files to backend.h

Transfer a pointer to the promoChar from PromotionPopUp() to PromotionCallback(). Fixes #34980

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 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!

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.

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.