Fix castling rule assignment in shuffle games

The conversion to in-board storage for rights copied directly to boards[0] in steadof
initialPosition, and was then overwritten when the latter was copied with uninitialized
rights to boards[0]. This affected FRC and CRC, where rights are non-standard.

Implement castling in -variant caparandom

CRC should be treated as FRC, but until now it never was. We now test for FRC_TYPE_CASTLING.

Fix default castling rights on reading incomplete FEN

This was overlooked in the conversion to the in-board storage of rights.

Fix NoncompliantFEN defaults.

This must be NULL to get compliant FENs, not "", which would suppress castling and e.p. fields.

Change format of -adapterCommand.

first or f at the start of an option name are now automatically changed into second or s,
rather than needing %% to be recognized. The values of -ec and -ed in the default
-adapterCommand are now quoted (now XBoard understands quoting in the engine command line).

Make WinBoard defaults for -fd and -sd equal to "." rather than empty strings.

Delete old indirection settings-file code

Correct XBoard default debug file name

Suppress saving font settings.

The fonts are size dependent, and might thus not be reusable in the next
session. For now we therefore suppress the saving of them in the
settings file, until we diversify the fonts per boardSize.

added missing sounds files to be able to compile on windows

Changes needed to compile master branch.

implements the eval-graph window for XBoard

improves the XBoard handling of the engine command line

it now understands limited forms of quoting, so engine binaries with spaces in their file name can be handled.

A better and more flexible way of invoking Polyglot

rewrote uci.c. No more use of temporary files, all data passed through the engine command line. Adds a new command-line option -adapterCommand for this.

remove trailing \r in xboard output

there is an annoying carriage return (displayed as ^M) at the end of the PV in the engine-output window, not
only with Fairy-Max, but also with UCI engine running under Polyglot. This patch replaces the trailing CR that Linux programs send after the PV line y a space.

double buffer size to prevent overflow

This patch doubles the size of the ICS input buffer to 16KB, to prevent
the suspected overflow with -keepLineBreaksICS false when using
"inchannel 1" on FICS and over 500 people are tuned in.

fix problem with empty string in -firstNeedsNoncompliantFEN

This fixes a bug in the use of the -firstNeedsNoncompliantFEN option with
an empty string as argument. (Which led to truncation of the FEN.)

updates makefiles to include dependency on config.h

adds the dependency of wbres.o on config.c introduced by having the VersionInfo in the resource

correctly apply some check boxes from the option menu

Usng the check-boxes in the Option->Adjudications menu to indicate the
engine reports absolute scores is now copied directly to the ChessProgramState,
from where it is used, in stead of only to appData. This makes the change take
effect immediately, in stead of after restarting.

score sign in analysis mode

this does not alter any of the current behavior with compliant engines,
but does extend the correction for non-compliant score reporting under
control of the -first(second)ScoreIsAbs option to cases where the engine
is analyzing or playing on an ICS.

fix for bug #28077: xboard needs to link against x11

hopefully fixed automake to take care of this

Fix castling rights when copying FEN to clipboard (again)

missed a spot last time

restoring windows (EngineOutput and MoveHistory) on startup

restoring the open / closed status of the
auxiliary windows (EngineOutput and MoveHistory) on startup
in XBoard like they do in WinBoard, which makes sense now that
we aso remember their coordinates between sessions.

fixed some outstanding pixmaps

fixed transparency issues

use linux style options for config file and add a system config file for XBoard

This patch makes XBoard write the options to its conf file in
Linux style, "-option value" rather than Windows style "/option=value".

The WinBoard option parsing code has always understood both formats
(including any hybrids), but the format it preferred for writing might
look quite strange to Linux users. I also attached a file to be installed
as /etc/xboard/xboard.conf on "make install".

fix window positioning

This patch adds stuff in the header files that allow the various front-end files
to share information on which windows exist, are up, and how they are positioned.
The EngineOutput, MoveHistory and GameList windows now all remember their
position and size, even when they are closed when XBoard exits.

same argDescriptor parsing for Xboard and Winbaord

This patch makes XBoard and WinBoard use the same argDescriptor
list of options, and (largely) use the same code for parsing and saving them.
This is acheived by moving this code to a new file args.h, and #including this
in both winboard.c and xboard.c (so that tiny differences, e.g. in default
values in the table, are decided by the context). XBoard has new menu items
"Save Settings Now" and "Save Settings on Exit" in the "Option" menu.
XBoard uses /etc/xboard/xboard.conf as default settings file.
A new option, -saveSettingsFile, can specify where settings should be saved,
without parsing the file. (So that it also works if the file does not yet exist.)

new developer release

changed release name to <branch>-<date>

fix the irritating wandering off of the MoveHistory window in XBoard on opening/closing.

use xtell for talking to handles, but tell for talking into a channel.

by changing the chatboxes to use xtell in stead of tell, we broke
the possibility to use the chat-box for chatting into a channel (which is
apparently not possible with xtell).

fix bug in bughouse drop menu

This patch fixes the bughouse drop menu, which was broken by the
new mouse driver, and could no longer drop white Pawns.

2nd step for moving option parsing from winboard to the backend

This is the next step in option-setting refactoring: All options that are
defined in XBoard are now included in the WinBoard ArgDescriptor list.
The handling of sound option has been harmonized: the sound names
now all initially go to the appData array, (as in XB) in stead of directly to the
WB-specific variables, and LoadAllSounds now takes care of the copying.
A new routine ExportSounds copies them back before saving settings
(which, in XBoard, could be a no-op).

The definition of defaults for the XB-only options has been moved to common.h.

1st step for moving option parsing from winboard to the backend

This is a preparatory step in moving option parsing of WinBoard to the back-end.
The table of command-line options now contains a fifth field, holding the compiled-in default.
This obviates most of the front-end routine InitAppData, which now is replaced by a table-driven
initialization routine SetDefaultsFromList(), which could be back-end.

refactoring evalgraph code

This is the refactored EvalGraph code: the back-end part is separated off,
and put in a file evalgraph.c, which in the future can also be used with XBoard.
Only the WinBoard front-end code is left in the file wevalgraph.c. A new
header file defines the variables and routines they share.

clean-up

some file permissions where set up wrong
(e.g. execute permissions for text files)

refactoring of engineoutput

This refactoring patch separates the back-end part out of
xengineoutput.c, and puts it in a separate file engineoutput.c.
A new header engineoutput.h included by both defines the routines
for cross calling. Similarly wengineo.c will be stripped from its
back-end code, (and renamed to wengineoutput.c), so WinBoard and
XBoard now both use engineoutput.c rather than maintaining duplicate
code in their front-end part.

I also let the back-end now call EngineOutputUpdate directly, rather then
through a relay in the front-end, by renaming it to SetProgramStats
(the former name of the relay in winboard.c / xboard.c).

Moved some templates of functions in engineoutput.c from winboard.h
to frontend.h

fixed build on openbsd

added missing header file

improve thinking-output for mulit-variant

Improve multi-variant support by sorting the Thinking-Output lines of the most recent depth by score in the Engine-Output window.

fix castling rights when copying FEN to clipboard

This patch forces leaving of Edit-Position mode before copying a FEN to the clipboard, to make sure castling rights are faked in a consistent way. Before you could get FENs that had castling rights for non-existent Rooks.

This patch gives a better handling of comments in PGN files, and adds the variation as comment to the main line on using Revert in local mode.

fixed prototype for AppendComment

cleaned up some debug messages and typos

variation-support patch

It allows you to Revert to the main line (or previous variation) when
playing variations on an existing game in EditGame or Analyze mode.

fix for new way of saving castling and e.p. information

I forgot to make some conversions to the new-style encoding of castling there
(using the enum constant NoRights instead of -1 to indicate
the absence of rights, which was the price of storing the
rights in the board, which is an unsigned enum type.)

I also was a bit worried about portability, because I casted the enum type ChessSquare
to (int) before comparing it with (possibly negative) constants EP_NONE etc.
Now on my system enum types are (unsigned int), and this works, but I could
imagine there are systems where this would be an unsigned char. (Not sure
if there are rules for this). So to be safe I cast ChessSquare to (signed char),
which should always work to make small negative ints assigned to it read
back as themselves.

removed files that should only be in the windboard directory

add Winboard source files into tar-ball

forgot to add them when rewriting the Makefile.am

Integrate castling and e.p. rights into board array; bugfix for EditPosition

A large-scale operation that I have been postponing since the very first 4.4.0:
The castling and e.p. rights are now integrated into the board array, as
boards[moveNr][CASTLING][i] replacing castlingRights[moveNr][i] and
boards[moveNr][EP_STATUS] replacing epStatus[moveNr]
These quantities are now automatically copied with the position by CopyBoards
(allowing the removal of lots of dedicated copying code), and passed as
function arguments with the position (so that the moves.c functions like
TestLegality, MateTest, CoordsToAlgebraic, Disambiguate, GenLegal
no longer need explicit e.p. and castling parameters.

Also fixed a bug introduced by a recent previous fix: by adding fake castling
rights to a position set up through the EditPosition menu, I destroyed the
castling rights of a pasted FEN. Faking castling rights is now made conditional
in EditPositionDone(), depending on the caller. (I had not realized it was also
called after FEN pasting, to send the position to the engine.)

added missing library for build on OS X

Xmu seems to be missing

Some code refactoring and cleanup; one small bug fix

Some code refactoring, mainly to bring definitions of routines and variables
shared between winboard.c and one of the auxiliary-windows files
whistory.c, wevalgraph.c, wgamelist.c, wengineo.c somewere in a header file
(winboard.h). Unfortunately this made winboard.h dependent on data types
defined in frontend.h, so that the #include order had to be changed in many
other .c files.

The operation revealed one potentially bad bug: engineOutputDialogUp was
written as int in engineo.c, but declared as char in winboard.c.

I also changed the storage mechanism of the the comment and tags window
positions to use WindowPlacement structs rather than simple variables,
so that in the future they can be docked, like the others.

I also removed some remnants of the Analysis window.

The wgamelist.h and wedittags.h header files contained only 3 lines, and
I absorbed them in winboard.h, rather than creating other such nearly empty
headers engineo.h, whistory.h and wevalgraph.h. This affects the make files.

let the clocks run in -searchTime mode

add fixed time per move to the WinBoard time-control menu dialog

updated version number to unstable

release of version 4.4.1

cleaned up ChangeLog and NEWS

use xtell instead of tell in wchat

updated files for a new release, added a shortlog and a diffstat

prevent buffer overflow

This is a buffer-size adjustments to prevent the JAWS version from
crashing when it has to read out loud very long comments.

Further copy/paste fixes

We needed to implement the XA_TARGETS target in order for more
sophisticated apps to really believe we put anything on the clipboard
they can use.

Make copy/paste position and game use clipboard, bug #27810

Copy actually sets both the clipboard and the selection for
convenience and compatibility with the old way of doing things.  Paste
pastes from the clipboard by default, but the new -pasteSelection
option lets you get back the old behavior of pasting from the
selection.

switch focus to the board after loading a game

fix for bug #27826: fixed autoDisplayComment

apparently the -autoDisplayComment option was not working,
not even in WinBoard. This patch should fix it,both for WB and XB.
(Unfortunately xboard.c and winboard.c were using a different name
for the same front-end variable I needed, so I had to change one of them.)

fix for bug #27826: ported two options to xboard

also updated the documentation

fixed jaws version

turns out the new mouse handler broke the JAWS patch, as I discarded
the function IsPromotion. I adapted it to call HasPromotionChoice instead.
Also fixed two arnings by removing a no-longer needed debug print.

reformated html to be correctly validated

quick fix for "White Mates" in parser.l

fixed a regression

changed stderr to debug output, since stderr is closed in winboard

removing empty lines from ICS play

this patch causes removal of spurious prompts during ICS play also if the prompt is preceeded by a number of empty lines (as the board for the opponent move is on FICS).

another bug in VriantSwitch: an unitialized board was printed.

Must have been there a long time, but this time it crashed WinBoard.

more work on variant switch

the previous solution to the cosmetic problem of displaying a wrong shuffle
in an ICS game that was observed from the very start unfortunately turned
out to cause a real error in diplaying a stored game through the smoves
command, for variants not using the FIDE setup as standard.

This patch displays the position from before setting up the start position
of the new variant, but does setup boards[0] to the initial position of that
variant. (If this is a wrong shuffle, it is thus ot displayed, and a new board
will immediately be sent from the ICS to correct the shuffle.)

force full redraw in winboard

Full redraw is now always forced when highlightMovesWithArrow is on,
rather than only when highlight info is available. (As in drop moves the
latter is invalid, but we must still erase the arrow from the previous move.)

fix for bug #27790 and 277772.

Suppressing the display of a wrong shuffle when observing shuffle games from the first move(#27790 bis), and the sounding of the FICS bell on crazyhouse captures (#27772 bis).

fixed bug related to unsigned char in convert.c

I also fixed the unsigned-char bug in the convert.c for bitmaps that caused the
129x129 XBoard bitmaps to be defective, and a bug that required fixup with sed
afterwards (so I could use it to create a new m33s.bm)

unguarded debug printf. added the appropiate if statement

fix for two compiler warnings

fix for bug #27799: fix for nested-nested-nested variations

The book-probe code forgot to close the book file after opening it.

fixed by adding an fclose at two points.

removed unused v54?.bm bitmaps from resource file

it seems the v54?.bmp bitmaps were actually included in the WinBoard build
(in the resource file), although they were never actually used. So I have
removed their inclusion in the resource file now.

replaced defective bitmaps with copies from 4.2.7

removed bitmaps files that are not needed any more

fixed some bitmaps

Merge branch 'master' of git://git.sv.gnu.org/xboard

Drop an obsolete script that was only used to update my personal web site

Silence warnings when compiling 64-bit xboard

Some harmless warnings were caused by 64-bit mode having 32-bit
int but 64-bit long and pointer.  Add casts (etc.) to silence them.
This might require adding a definition of intptr_t (a signed int
type that is the same width as a pointer) on old systems that don't
have it.

Fix up man page some more

Sigh, the project history was still partly in reverse chronological
order and partly in forward order.  Swapped the order of some
sentences to help straighten it out.  I don't really care about this,
so other project members should feel free to do what they like here,
but the messed-up order was bugging me.

Fix some issues in the XBoard man page

Correct places where the XBoard man page was calling XBoard
"WinBoard".  Change references to "WinBoard engines" and "WinBoard
protocol" to "XBoard/WinBoard ...".  Update bug reporting guidelines.
Reorganize project history to something more like chronological order
-- it had gotten pretty randomly ordered.

new pre-release version; updated version numbers

This patch adds <Enter> to the characters that cause an automatic switch to the ICS console when typed to the board window.

fix to the minor graphics issue contained some typos, as was remarked in the bugs reports

improved mouse handler

The improved mouse handler still needed an extra call to DrawPosition to
work in XBoard.

NPS plays and pondering

Clock updating did not work correctly in node-based time controls when the
engine was pondering. (It was not really meant for use with pondering. Oh well...)
This patch checks if the engine reporting the node count is indeed thinking
(as opposed to pondering).

holdings update and regression fix

The holdings-update patch was not yet satisfactory, because FICS seems to send
holdings both before and after the board! And those received before did overwrite
the holdings received after the previous board. I know tried to fix that in a kludgey
way, by hiding a flag in the board that tells if the holdings have already been written
by the ICS or not, and ignore new holdings when they are. (Except in bughouse,
where holdings updates cannot be tied to a particular board and multiple updates
between boards are normal. (I hope?).)

I also undid a regression in my previous fix for the minor graphics problem,
and solved it in a slightly different way to prevent flicker.

improved mouse handler

I improved the mouse handler in a way which will preclude disappearence of pieces
on off-board or ambiguous (in-between squares) drag & drop moves. (As was reported
in the "premove bug?" thread. Also the command-line premove problem reported there
is now fixed, by removing the prefixing.

fix for bug #8847: moving backward while examining on FICS not reported to engine

I moved Daniel's code that backs up the engine when a bord with a lower number is received to a place where it is always done in zippy mode, rather than only when the engine is analyzing.

Apart from taking back moves, there was also the case where the
user moves forward by more than one move (usually immediately to
end-of-game). In this case a new movelist is fetched from the ICS
to recover the moves that were skipped. But this assumes that the
engine is at the start of the game and feeds all these moves to the
engine. So the engine had to be reset in this case to stay in sync.

better init for random number generator

fix for bug #27772: holdings not updated

holdings lines from the ICS nor update the holdings of boards[forwardMostMove] in stead of boards[currentMove].

allowe parsing / disambiguation of SAN moves like Xe4 in certain situations

This patch allows parsing / disambiguation of SAN moves like Xe4, if there
is only one wild-card piece X on the board. Moves with wild-card pieces
are now considered illegal if they leave the King in check by a piece of
known gait.

fixed bug when switching to variantsuper

fix for bug #27715: 2 (minor) graphic issues

I forced a total redraw when Reset is called from the ICS code in case a
new game is encountered, in the hope it will make the reported
(irreproducible) problem go away. As game switches are not frequent, I
guess there is hardly any downside to this.

fix for bug #27667: PV line missing in analysis window, part 3

"3. Wish for Engine state string (right to the icon): Can you add the actual search depth in analyzing mode? A change like "

I also added the requested display of depth of the analysis move in the
EngineOutput window. I guess a similar change could be made to wengineo.c (as this is in the
code they share in common), but no one ever requested that in the 5 years
since Alessandro wrote it. So for now I leave it alone.