Prevent unnecessary loading of engines

For engines that can be reused, the unloading at the end of a pairing
is in some cases wasteful, as the same engine will be used in the next
pairing as well. In simple gauntlets the gauntlet engine will always
stay first engine, while in multi-gauntlets the oppent engine will stay
the same for an entire round. In these cases unloading is now suppressed.

Reset protocol version before loading new engine

The protocol version was not reset, so if the previous engine was v1,
the newly loaded engine would not be sent a protover command, so
that its features would remain unknown!

Give focus to board window after ICS login

toprevent that the focus remains on any popped up chat boxes, it is
explicitlly set otherwise. The board window is preferred over the
ICS interaction window, because you can type from there to the ICS anyway,
and it makes all accelerator keys available.

Fix seek-graph popup and  popdown

Clicking outside the board area in WB (e.g.on the clocks) would also
trigger seek-graph 'popup'. This is now suppressed by testing for
y-coord > 0. In addition, the seek graph is automaticaly taken down
when we step to another position in the current game (using buttons,
arrow keys or whatever).

Suppress empty lines when observing on VICS

Capture of the engine kibitzes when observing an engine game with
-autoKibitz on was still leaking a linefeed on the Variant ICS
(but not FICS or ICC!?).

Fix printing non-numeric kibitz with -autoKibitz

Messages that were not considered kibitzed engine PV info for lack of
numeric content, were kibitzed as coming from "your opponent", which
is nonsense if you were iobserving, and masked the source. Fortunately
star_match[0] is still holding the original name, at that point.

Fix 4 warnings

Use same translation of "vs." everywhere

Add new translations to WB language file

Fix re-allocation of PGN-cache memory

It  allocated 8 times too much!

Fix height board dialog

Enable WB time-control dialog even when no clock mode

The dialog could be called from the tournament dialog to set a TCfor
the tourney even when currently in game-viewer mode, so it'd better work.

Exempt variant seirawan from -disguisePromoted inheritance

This is not a variant where promoted Pawns have to be distingushed
from primordial pieces, as captured pieces do not go into the holdings.

Add control for overrideLineGap in WB Board dialog

Make it a checkbox to choose between 0 and -1.

Add Ctrl-D as alternative to Ctrl+Alt+F12 in WinBoard

The acellerator key Ctrl+Alt+F12 seems reserved on Windows 7.

Fix seek graph for VICS

The sought command on the open-source ICS code is ICC-like (no terminating
line), although the server is FICS like. So also accept prompt as seek-graph
trigger on FICS.

Open WB startup chat boxes only after logged on

The init code for opening the chat boxes ismoved to DisplayIcsTitle().

Install mime types directly

Install desktop files directly

Install icons directly

Rather than letting make-install copy the pixmaps to the XBoard data
directory they are installed directly from the source files. The name
of the TRN icon was also corrected and changed.

Update texi file and remove duplicate control

Some missing option descriptions were added. The highlight-dragging
control was removed from the General Options dialog, as it did the same
as Show Target Squares. -highlightDragging wa changed to a volatile option
with default value true, making it effectively disappear as an XBoard
option. (In WinBoard it is still persistent.)

Remove the direct commands to the engines

forgot to list new desktop files in EXTRA_DIST section in Makefile.am

added some autogenerated po files to .gitignore

added desktop and mime-type association to autoconf install process

This should install and uninstall the mime assocciation and also the icons.
Also renamed the desktop files to avoid possible name collisions.

code cleanup: make function definition confirm to GNU coding style

change everything in the main directory to

void
funcname (int a, int b)

format

Updated copyright notice to 2012

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.