Fix heigth of multi-line labels

The previous height patch overlooked that some labels are multi-line,
and need more height.

Fix popdown seekgraph on forward-event XB

Only the grid was drawn when the > or >> button were operated, because
a full redraw was not forced by ForwardInner. The recent patch to take
the seek graph down on such events did not account for that. (BackwardInner
always seems to do full redraw, however?)

Put front-end support for -recentEngines in XBoard

The engines are added to the menu. Problem is still that in XBoard
startup engines are given by command, rather than selected by combobox,
so the command rather than the nickname ends up in the menu. If this is
then selected, it cannot be found in the list. To limit the damage,
Fairy-Max is now pre-installed without nickname. -recentEngines is now
configured to 4.

Better solution to button-height problem

In stead of adapting the height of text widgets to that of buttons,
(as represented by the message widget), we now do the opposite, and
adapt the button & label heights. This is made safe by obtaining textHeight
from the fontSet's max_logical_extent, though an XExtendsFontSet() call.
Turns out the default height of text widgets is not exactly that, but
4 pixels larger for borders and margin.
  The parameter XtNvertDistance is now used to align the checkboxes with
the following text widget, and their size is derived from textHeight as
well.
  The XtNvertDistance is set to -1 for the lower spin button, so that it
its border coincides with that of the upper spin button. Their height is
set to textHeight/2. This makes their combined height just 1 pixel more
than that of the preceeding text widget. (Uh? 2 x tw/2 -1 > tw?), so for
other buttons we use that too (i.e. textHeight = font_extent + 5).

Fix warning

Deleting debug printing had made a variable unused.

Fix sleeping bug

The text-edit-height patch introduced a bug that did not have any consquences
(yet), as the extra arg erroneously included leftover from a previous
SetValues call happened to be the same as one of the intended args.

Fix unloading of first engine on tourney start

Because NextTourneyGame was loading a new engine before ReserveGame was
called to unload it with dummy game nr = -1), the first engine proc kept
hanging.

Remove some of the most verbose debug output

Delete some unused WinBoard bitmaps

There waere up-side-down knight icons that apparently hid by mistake
in the bitmaps directory: they were not part of WinBoard

Cosmetic: change 4.5 to 4.6 in xboard.conf comment

Fix crash on empty combobox menu XB

XBoard crashed when a combobox without items was opened, as could easily
happen in the Load Engine dialogs when -firstChessProgramNames contained
no items, as it would without configuring. Now the creation of the popup
menu is suppressed in that case.

Fix position of checkbox XB

With the large height the widget get in internationalization, the
checkboxes are position way too high above the text. To shift them down
a dummy borderless label widget is drawn above them (when needed).

Fix heights in Engine Output window XB

The icons did not fit in the space above the memos anymore, so the height
of this area is now left free.

Derive height of text-edits in dialogs from message widget

With internationalization the default height of buttons has inexpliquably
inreased compared to that of text-edits, messing up the layout of the
GenericPopUp. The message widget is now used as a emasure of how high
the widgets should be.

Configure XBoard engines and textures

The new -useBoardTexture option is needed in xboard.conf, as we want to
distribute with the texture on, while the compiled-in default is off.
A list of 4 engines has been configured (fairymax, fruit, crafty, gnuchess),
as the Load Engine and Match Options checkboxes crash XBoard when the list
is empty. The -recentEngines option is set to 0 to make sure the code
associated with it is never executed: the code is still experimental,
and the required front-end support is only present in WinBoard anyway.

Update Dutch WinBoard translation

Create room in some WB dialogs for translations

Add 5 missing strings to WinBoard translation template

new developer release

updated po-filies

updated Changelog, NEWS, etc.

Fix non-bug in WinBoard

The IDM_RecentEngines was not used in symbolic form everywhere.

Fix MAXENGINES in WinBoard

The front-end arrays for expanding the engine list measured 1000 entries,
in stead of MAXENGINES, leading them to overflow.

Implement kludge to set options through Move Type-in

A leading "!!! " will cause the rest of the line to be parsed as
command line. Very dangerous, as an unknown option (e.g. because of a typo)
will cause a fatal error, which makes XBoard exit.

Put move number in Eval Graph title

Implement -pgnNumberTag option

A new (persistent) boolean option controls if the sequence number is to
be written in saved tourney games as a Number tag, to make it easier
to correlate games with debug files.

Create separate debug fil for each tourney game

Users can now apecify a -debugfile with %d in its name, and this will
be replaced by the sequence number of the game. TwoMachinesEvent then
starts by opening a new file when the name has changed, so that each
game will go to a different file, with recognizable name.

Fix bug in WB combobox readout

The previous patch exposed a latent bug in the readout of the generic popup.

Allow grouping of engines in engine list

Lines "# GROUPNAME" and "# end" enclose a group in -firstChessProgramNames.

Fix TidyProgramName

A trailing .exe was not removed if the engine name was quoted.

Put recently used engines in WB menu

A new option -recentEngines specifies how many engines should be listed.
The list of nicknames is stored in the ettings file as -recentEngineList.
The engines are appended to the Engine menu.

Fix registering of EditTagsProg

This was registered with X as EditCommentProc!

Increase max nr of engines to 2000

Move the #define for MAXENGINES also to common.h, so that both the
Load and Tourney dialog use the same maximum.

Fix fix of switch to mono-mode

The previous fix had == in stead of != for testing if mono-mode had changed.

Make New Variant dialog visible in mono-mode

All colored buttons did get a black background, and also black text...
Use timer background color in stead, which is white.

Incease size of WB array of translated items

For the BoardOptions dialog 42 items are now needed

Implement Clone Tourney button XBoard

This makes use of the new GenericUpdate().

Implement GenericUpdate function

This does the opposite from GenericReadout: it copies the current
parameter values into the dialog.

Fix arrow damage with highlightDragging XB

With highlight dragging the highlights of the previous moves are cleared
when lifting the piece. Logically, this should also clear the arrow, but
to do that requires a board redraw, which is not done at this stage. (And
even then this would only be effective after marking the arrow damage,
which is not done yet either). As a temporary fix, we repair the arrow
by redrawing it.
  This patch has moved marking of the arrow damage from DrawArrowHighlight
to a separate routine, so a future better fix can also call that when the
arrow should be erased.

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