changes from H.G. Muller; version 4.3.8 v4.3.8
authorH.G. Muller <h.g.muller@hccnet.nl>
Sun, 19 Apr 2009 16:48:59 +0000 (09:48 -0700)
committerArun Persaud <arun@nubati.net>
Sun, 19 Apr 2009 17:32:56 +0000 (10:32 -0700)
backend.c
config.h
moves.c
readme_HGM.txt
winboard/resource.h
winboard/winboard.c
winboard/winboard.rc

index 6c79b03..79bdc68 100644 (file)
--- a/backend.c
+++ b/backend.c
@@ -1367,9 +1367,8 @@ StringToVariant(e)
     if (!e) return v;\r
 \r
     /* [HGM] skip over optional board-size prefixes */\r
-    if( sscanf(e, "%dx%d+%d_", &i, &i, &i) == 3 ) {\r
-        while( *e++ != '_');\r
-    } else if( sscanf(e, "%dx%d_", &i, &i) == 2 ) {\r
+    if( sscanf(e, "%dx%d_", &i, &i) == 2 ||\r
+        sscanf(e, "%dx%d+%d_", &i, &i, &i) == 3 ) {\r
         while( *e++ != '_');\r
     }\r
 \r
@@ -1734,6 +1733,65 @@ CopyHoldings(Board board, char *holdings, ChessSquare lowestPiece)
 \r
 }\r
 \r
+void\r
+VariantSwitch(Board board, VariantClass newVariant)\r
+{\r
+   int newHoldingsWidth, newWidth = 8, newHeight = 8, i, j;\r
+   if(gameInfo.variant == newVariant) return;\r
+\r
+   /* [HGM] This routine is called each time an assignment is made to\r
+    * gameInfo.variant during a game, to make sure the board sizes\r
+    * are set to match the new variant. If that means adding or deleting\r
+    * holdings, we shift the playing board accordingly\r
+    */\r
+\r
+  if (appData.debugMode) {\r
+    fprintf(debugFP, "Switch board from %s to %s\n",\r
+               VariantName(gameInfo.variant), VariantName(newVariant));\r
+    setbuf(debugFP, NULL);\r
+  }\r
+    gameInfo.holdingsSize = 5; /* [HGM] prepare holdings */\r
+         switch(newVariant) {\r
+            case VariantShogi:\r
+            case VariantShowgi:\r
+              newWidth = 9;  newHeight = 9;\r
+              gameInfo.holdingsSize = 7;\r
+            case VariantBughouse:\r
+            case VariantCrazyhouse:\r
+              newHoldingsWidth = 2; break;\r
+            default:\r
+              newHoldingsWidth = gameInfo.holdingsSize = 0;\r
+    }\r
+\r
+    if(newWidth  != gameInfo.boardWidth  ||\r
+       newHeight != gameInfo.boardHeight ||\r
+       newHoldingsWidth != gameInfo.holdingsWidth ) {\r
+\r
+        /* shift position to new playing area, if needed */\r
+        if(newHoldingsWidth > gameInfo.holdingsWidth) {\r
+           for(i=0; i<BOARD_HEIGHT; i++) \r
+               for(j=BOARD_RGHT-1; j>=BOARD_LEFT; j--)\r
+                   board[i][j+newHoldingsWidth-gameInfo.holdingsWidth] =\r
+                                                     board[i][j];\r
+           for(i=0; i<newHeight; i++) {\r
+               board[i][0] = board[i][newWidth+2*newHoldingsWidth-1] = EmptySquare;\r
+               board[i][1] = board[i][newWidth+2*newHoldingsWidth-2] = (ChessSquare) 0;\r
+           }\r
+        } else if(newHoldingsWidth < gameInfo.holdingsWidth) {\r
+           for(i=0; i<BOARD_HEIGHT; i++)\r
+               for(j=BOARD_LEFT; j<BOARD_RGHT; j++)\r
+                   board[i][j+newHoldingsWidth-gameInfo.holdingsWidth] =\r
+                                                 board[i][j];\r
+        }\r
+\r
+        gameInfo.boardWidth  = newWidth;\r
+        gameInfo.boardHeight = newHeight;\r
+        gameInfo.holdingsWidth = newHoldingsWidth;\r
+        gameInfo.variant = newVariant;\r
+        InitDrawingSizes(-2, 0);\r
+    } else gameInfo.variant = newVariant;\r
+}\r
+\r
 static int loggedOn = FALSE;\r
 \r
 /*-- Game start info cache: --*/\r
@@ -2332,12 +2390,11 @@ read_from_ics(isr, closure, data, count, error)
              "* * match, initial time: * minute*, increment: * second")) {\r
                /* Header for a move list -- second line */\r
                /* Initial board will follow if this is a wild game */\r
-\r
                if (gameInfo.event != NULL) free(gameInfo.event);\r
                sprintf(str, "ICS %s %s match", star_match[0], star_match[1]);\r
                gameInfo.event = StrSave(str);\r
-               gameInfo.variant = StringToVariant(gameInfo.event);\r
-                Reset(TRUE,TRUE); /* [HGM] possibly change board or holdings size */\r
+                /* [HGM] we switched variant. Translate boards if needed. */\r
+                VariantSwitch(boards[currentMove], StringToVariant(gameInfo.event));\r
                continue;\r
            }\r
 \r
@@ -2802,7 +2859,8 @@ read_from_ics(isr, closure, data, count, error)
                    started = STARTED_NONE;\r
                    parse[parse_pos] = NULLCHAR;\r
                    if (appData.debugMode)\r
-                     fprintf(debugFP, "Parsing holdings: %s\n", parse);\r
+                      fprintf(debugFP, "Parsing holdings: %s, currentMove = %d\n",\r
+                                                        parse, currentMove);\r
                    if (sscanf(parse, " game %d", &gamenum) == 1 &&\r
                        gamenum == ics_gamenum) {\r
                        if (gameInfo.variant == VariantNormal) {\r
@@ -2811,22 +2869,7 @@ read_from_ics(isr, closure, data, count, error)
                            * to move the position two files to the right to\r
                            * create room for them!\r
                            */\r
-                          int i, j;\r
-                          if(gameInfo.holdingsWidth == 0) /* to be sure */\r
-                          for(i=0; i<BOARD_HEIGHT; i++)\r
-                            for(j=BOARD_RGHT-1; j>=0; j--)\r
-                              boards[currentMove][i][j+2] = boards[currentMove][i][j];\r
-\r
-  if (appData.debugMode) {\r
-    fprintf(debugFP, "Switch board to Crazy\n");\r
-    setbuf(debugFP, NULL);\r
-  }\r
-                         gameInfo.variant = VariantCrazyhouse; /*temp guess*/\r
-                          gameInfo.boardWidth  = 8;  /* [HGM] guess board size as well */\r
-                          gameInfo.boardHeight = 8;\r
-                          gameInfo.holdingsSize = 5;\r
-                          gameInfo.holdingsWidth = 2;\r
-                          InitDrawingSizes(-2, 0);\r
+                          VariantSwitch(boards[currentMove], VariantCrazyhouse); /* temp guess */\r
                          /* Get a move list just to see the header, which\r
                             will tell us whether this is really bug or zh */\r
                          if (ics_getting_history == H_FALSE) {\r
@@ -2862,7 +2905,7 @@ read_from_ics(isr, closure, data, count, error)
                                    gameInfo.black, black_holding);\r
                        }\r
 \r
-                       DrawPosition(FALSE, NULL);\r
+                        DrawPosition(FALSE, boards[currentMove]);\r
                        DisplayTitle(str);\r
                    }\r
                    /* Suppress following prompt */\r
@@ -3078,27 +3121,13 @@ ParseBoard12(string)
        timeIncrement = increment * 1000;\r
        movesPerSession = 0;\r
        gameInfo.timeControl = TimeControlTagValue();\r
-       gameInfo.variant = StringToVariant(gameInfo.event);\r
+        VariantSwitch(board, StringToVariant(gameInfo.event) );\r
   if (appData.debugMode) {\r
     fprintf(debugFP, "ParseBoard says variant = '%s'\n", gameInfo.event);\r
     fprintf(debugFP, "recognized as %s\n", VariantName(gameInfo.variant));\r
     setbuf(debugFP, NULL);\r
   }\r
 \r
-        gameInfo.holdingsSize = 5; /* [HGM] prepare holdings */\r
-        gameInfo.boardWidth = gameInfo.boardHeight = 8;\r
-        switch(gameInfo.variant) {\r
-            case VariantShogi:\r
-            case VariantShowgi:\r
-              gameInfo.boardWidth = 9;  gameInfo.boardHeight = 9;\r
-              gameInfo.holdingsSize = 7;\r
-            case VariantBughouse:\r
-            case VariantCrazyhouse:\r
-              gameInfo.holdingsWidth = 2; break;\r
-            default:\r
-              gameInfo.holdingsWidth = gameInfo.holdingsSize = 0;\r
-        }\r
-        InitDrawingSizes(-2, 0);\r
         gameInfo.outOfBook = NULL;\r
        \r
        /* Do we have the ratings? */\r
@@ -3237,7 +3266,7 @@ ParseBoard12(string)
     if (moveNum > 0) {\r
   if (appData.debugMode) {\r
     fprintf(debugFP, "accepted move %s from ICS, parse it.\n", move_str);\r
-    fprintf(debugFP, "board = %d-d x%d\n", BOARD_LEFT, BOARD_RGHT, BOARD_HEIGHT);\r
+    fprintf(debugFP, "board = %d-%d x %d\n", BOARD_LEFT, BOARD_RGHT, BOARD_HEIGHT);\r
     setbuf(debugFP, NULL);\r
   }\r
        if (moveNum <= backwardMostMove) {\r
@@ -3592,8 +3621,8 @@ AlphaRank(char *move, int n)
     if( !appData.alphaRank ) return;\r
 \r
     while(c = *p) {\r
-        if(c>='0' && c<='9') *p += 'a'-'0'; else\r
-        if(c>='a' && c<='z') *p -= 'a'-'0';\r
+        if(c>='0' && c<='9') *p += AAA-ONE; else\r
+        if(c>='a' && c<'x') *p -= AAA-ONE;\r
         p++;\r
         if(--n < 1) break;\r
     }\r
@@ -5075,7 +5104,7 @@ HandleMachineMove(message, cps)
     if (!strncmp(message, "setboard ", 9)) {\r
         Board initial_position; int i;\r
 \r
-        GameEnds(GameIsDrawn, "Engine aborts game", GE_XBOARD);\r
+        GameEnds(GameUnfinished, "Engine aborts game", GE_XBOARD);\r
 \r
         if (!ParseFEN(initial_position, &blackPlaysFirst, message + 9)) {\r
             DisplayError("Bad FEN received from engine", 0);\r
@@ -11818,8 +11847,8 @@ PositionToFEN(move, useFEN960)
        *p++ = '-';\r
     }\r
 \r
-    /* [HGM] print Crazyhouse holdings */\r
-    if( gameInfo.variant == VariantCrazyhouse ) {\r
+    /* [HGM] print Crazyhouse or Shogi holdings */\r
+    if( gameInfo.holdingsWidth ) {\r
         *p++ = ' '; q = p;\r
         for(i=0; i<gameInfo.holdingsSize; i++) { /* white holdings */\r
             piece = boards[move][i][BOARD_WIDTH-1];\r
index 7f3a083..7b9e8bc 100644 (file)
--- a/config.h
+++ b/config.h
@@ -47,7 +47,7 @@
 \r
 #define LAST_PTY_LETTER 'q'\r
 \r
-#define PATCHLEVEL "7"\r
+#define PATCHLEVEL "8"\r
 \r
 #define PRODUCT "WinBoard"\r
 \r
diff --git a/moves.c b/moves.c
index 9aff915..ddf3ee9 100644 (file)
--- a/moves.c
+++ b/moves.c
@@ -932,6 +932,17 @@ int CheckTest(board, flags, rf, ff, rt, ft, enPassant)
     for (cl.fking = BOARD_LEFT+0; cl.fking < BOARD_RGHT; cl.fking++)\r
        for (cl.rking = 0; cl.rking < BOARD_HEIGHT; cl.rking++) {\r
           if (board[cl.rking][cl.fking] == king) {\r
+              if(gameInfo.variant == VariantXiangqi) {\r
+                  /* [HGM] In Xiangqi opposing Kings means check as well */\r
+                  int i, dir;\r
+                  dir = (king >= BlackPawn) ? -1 : 1;\r
+                  for( i=cl.rking+dir; i>=0 && i<BOARD_HEIGHT &&\r
+                                board[i][cl.fking] == EmptySquare; i+=dir );\r
+                  if(i>=0 && i<BOARD_HEIGHT &&\r
+                      board[i][cl.fking] == (dir>0 ? BlackWazir : WhiteWazir) )\r
+                          cl.check++;\r
+              }\r
+\r
              GenPseudoLegal(board, flags ^ F_WHITE_ON_MOVE, -1,\r
                             CheckTestCallback, (VOIDSTAR) &cl);\r
              goto undo_move;  /* 2-level break */\r
index 5769521..0845114 100644 (file)
@@ -1,3 +1,98 @@
+Winboard_F.4.3.8 release notes\r
+\r
+This Winboard supports the following new options (shown here with their default values):\r
+\r
+/variant=normal \r
+This (already existing) option has been expanded with several new variants, involving non-conventional pieces and deviating board sizes. The board size is automatically adapted to the selected variant, unless explicitly overruled (see below). The new variants are (with default board size, files x ranks, in parentheses): \r
+\r
+variant name    Game           board     description\r
+knightmate    Knightmate        (8x8)  Variant where the King moves as a Knight, and vice versa \r
+capablanca    Capablanca Chess (10x8)  Variant featuring Archbishop and Chancellor as new pieces \r
+gothic        Gothic Chess     (10x8)  Same as Capablanca, with a more interesting opening position \r
+courier       Courier Chess    (12x8)  a Medieval form that combines elements of Shatranj and modern Chess \r
+shogi         Shogi             (9x9)  Japanese Chess \r
+xiangqi       Xiangqi          (9x10)  Chinese Chess \r
+fairy         Fairy Chess       (8x8)  Variant were you can use all pieces of other variants together \r
\r
+The variant can be set from the newly added "File -> New Variant..." sub-menu. \r
+Extra board files are indicated by the letters i, j, k, l, ... For boards with more than 9 ranks, the counting starts at zero! Non-FIDE pieces will be referred to in FENs and PGN by letters that depend on the variant, and might collide with piece designators in other variants. E.g. in Xiangqi 'C' is a Cannon, in Capablanca Chess it is a Chancellor. Pieces that do not belong in a variant cannot be addressed in FEN and PGN either as long as that variant is selected, unless the letter assignment is overruled by the /pieceToCharTable option. The variant is not saved in the winboard.ini file; on start-up we always get variant "normal" unless we use the command-line opton, or have added the option to the winboard.ini file manually (in which case it will disappear when this file is overwritten). \r
+WinBoard_F knows the movement of all pieces occurring in Capablanca Chess (of which FIDE Chess is a subset), Shatranj, Courier, Xiangqi and 9x9 Shogi, so that these games can be played with legality testing enabled. \r
+\r
+/pieceToCharTable="PNBRQFWEMOUHACGSKpnbrqfwemouhacgsk" \r
+Each piece that WinBoard knows (in its legality test) has a letter associated with it, by which it will be referred to in FEN or PGN. The default assignment can be overruled with this option. The value has to be a string of even length, with at least 12 characters. The first half of the string designates the white pieces, the second half the black. \r
+The last letter for each color will be assigned to the King. (This is the piece that moves as an orthodox King; note that Nightmate and Xiangqi have a different royal piece.) All letters before it will be assigned to the other pieces in the order: \r
+\r
+P Pawn                 (move often depends on variant) \r
+N Knight               (move subtly different in Xiangqi (where it is written as H) or Shogi) \r
+B Bishop \r
+R Rook \r
+Q Queen                (Lance L in Shogi) \r
+F Ferz/General         (Silver General S in Shogi) \r
+W Wazir/GrandVizer     (Gold General G in Shogi, in Xiangqi it is royal and denoted by K) \r
+E Alfil/Elephant       (Moves subtly different in Xiangqi vs Shatranj/Courier) \r
+M Commoner/Man \r
+O Cannon/Pao \r
+U Unicorn              (representation of Royal Knight in Knightmate, used as promoted Pawn in Shogi) \r
+H Nightrider           (Promoted Knight in Shogi and CrazyHouse) \r
+A Archbishop/Cardinal  (Promoted Bishop in Shogi and CrazyHouse) \r
+C Chancellor/Marshall  (Promoted Rook   in Shogi and CrazyHouse) \r
+G Grasshopper          (Promoted Queen in Crazyhouse, promoted Lance in Shogi) \r
+S                      (Promoted Silver in Shogi) \r
+K King \r
\r
+Pieces that are not mentioned (because the argument has less than 34 characters) will remain disabled. Mentioned pieces can be disabled by assigning them a '.' (period). They are then not recognized in FEN or PGN input. It is not advisable to disable a piece that is present in the opening position of the selected variant, though. \r
+Promoted pieces that need to be distinguished from original pieces of the same type (because of demotion on capture and transfer to the holdings) will be indicated by the letter for the unpromoted piece with a '+' in front of it (Shogi), or by the letter of the promoted piece with a '~' after it (Crazyhouse, Bughouse, in general everything with holdings that is not Shogi). \r
+All the new pieces have a native biytmap representation in the board sizes 'bulky' and 'middling'. For all window sizes that do not support such fairy bitmaps, promoted NBRQ are represented as a 2-sizes-smaller normal piece symbol, so that Crazyhouse can be played at any size. People disliking the fairy representations might even prefer that. \r
+There is an enhanced 'Edit Position' menu popup (right-clicking on the squares after selecting this mode in the main menu), featuring some common non-FIDE pieces, and 'promote' and 'demote' options to make those not directly in the menu. The promotion popup shows ArchBishop and Chancellor in Capablanca and Gothic, (or in fact in any game where this piece is not disabled), and leaves only the options YES / NO in Shogi. In Xiangqi there are no promotions.\r
+\r
+/fontPieceToCharTable="PNBRQFWEMOUHACGSKpnbrqfwemouhacgsk" \r
+This option is similar to /pieceToCharTable, but sets the font character that is used to display the piece on the screen (when font-based rendering is in use), rather than in the FEN or PGN. The default setting should work with the WinboardF font, which uses the 'intuitive' mapping of font characters to symbols. \r
+Note that UHACGS are also used to represent the promoted versions of PNBRQF, in games like Crazyhouse and Shogi, where the promotion has to be undone on capture.\r
+\r
+/boardWidth=-1 /boardHeight=-1 \r
+Set a number of files and ranks of the playing board to a value that will override the defaults for the variant that is selected. A value of -1 means the variant default board size will be used for the corresponding parameter (and is itself the default value of these options). These parameters can be set in the "Files -> New Variant..." sub-menu, where they are reset to the default -1 is you OK the chosen variant without typing something to overrule it. These parameters are saved in the winboard.ini file. (But unless you saved while a variant with board-size override was selected, they will always be saved as -1.) \r
+A variant with a non-standard board size will be communicated to the engine(s) with the board size prefixed to the variant name, e.g. "variant 12x8_capablanca". In protocol 2 the engine must first enable this feature by sending "boardsizeFxR" amongst the accepted variants, where F is the maximum number of files, and R the maximum number of ranks, as decimal numbers. \r
+\r
+/holdingsSize=-1 \r
+Set the size of the holdings for dropable pieces to a value that will override the default for the variant that is selected. A value of -1 means the variant default holdings size will be used for that parameter (and is itself the default value of this options). This parameter can be set in the Files -> New Variant... sub-menu, where it is reset to the default -1 is you OK the chosen variant without typing something to overrule it. This parameters is saved in the winboard.ini file. \r
+To disable holdings, set their size to 0. They will then not be displayed. For non-zero holding size N, the holdings are displayed left and right of the board, and piece drops can be effected by dragging pieces from the holdings to the drop square. In bughouse, the holdings will be filled by the ICS. In all other variants, captured pieces will go into the holdings (after reversing their color). Only the first N pieces of the /pieceToCharTable argument will go into the holdings. All other pieces will be converted to Pawns. (In Shogi, however they will be demoted in the regular way before determining if they fit.) Pieces that are disabled (per default and per /pieceToCharTable option) might not be counted when determining what are the first N pieces. \r
+Non-standard holdingsize will be communicated to the engine by prefixing it (together with the board size, even if this is standard) to the variant name, e.g. "variant 7x7+5_shogi". In protocol 2 the engine should enable this feature by sending "holdingsH" amongst the variant names, where H is the maximum acceptable holdings size as a decimal number. \r
+\r
+/alphaRank=FALSE \r
+When this parameter is true, a-h are converted to 1-9, and vice versa, in all move output and input (to PGN files or SAN move display as well as in communication with the engine). This might be useful for Shogi, where conventionally one uses letters to designate ranks, and digits to designate files. Engines that want to use this option must make sure pieces are never represented by lower case! This option can be set from the Files -> New Variant... menu, where it defaults to FALSE unless you explicitly set it. It is not saved in the winboard.ini file. \r
+This kludge does not seem to work for reading PGN files. Saving works fine. For now, using it is not recommended.\r
+Note that the PGN format in Shogi also leaves out the trailing '+' as check indicator: In Shogi such a trailing '+' means promotion, while a trailing '=' means defer promotion. Prefix '+' signs are used on moves with promoted pieces, disambiguation is done western SAN style.\r
+\r
+/allWhite=FALSE\r
+Causes the outline of the 'white' pieces to be superimposed onto the 'black' piece symbols as well (as a black outline) when native bitmaps are used (as opposed to font-based rendering). This is useful if we choose a very light color to represent the 'black' pieces. It might be particularly useful in Shogi, where the conventional representation of the 'black' pieces is as upside-down white pieces, so that both colors would be white. This option is saved in the winboard.ini file, and can be set in the "Options -> Board..." sub-menu. \r
+\r
+/flipBlack=FALSE\r
+Setting this option will cause upside-down display of the native piece bitmaps used to represent the pieces of the side that plays black, as would be needed for a traditional representation of Shogi pieces. It can be set from the "Options -> Board..." sub-menu, and it is saved in the winboard.ini file. For now, traditional Shogi bitmaps are not included, though.\r
+\r
+/detectMate=TRUE \r
+/testClaim=TRUE \r
+/materialDraws=TRUE \r
+/trivialDraws=FALSE \r
+/ruleMoves=51 \r
+/repeatsToDraw=6 \r
+These are all options that only affect engine-engine play, and can be set from the "Options -> Engine..." sub-menu. They are all related to adjudication of games by the GUI. Legality checking must be switched on for them to work. \r
+If /detectMate is TRUE, the GUI recognizes checkmate and stalemate (but not in games with holdings!), and ends the game accordingly before the engines can claim. This is convenient for play with engines that fail to claim, and just exit. \r
+With /testClaim set, all result and illegal-move claims by engines that claim more than their own loss are scrutinized for validity, and false claims result in forfeit of the game. Useful with buggy engines. \r
+The option /materialDraws=TRUE causes games with insufficient mating material to be adjudicated immediately as draws, in case the engines would not claim them. \r
+The option /trivialDraws adjudicates KNNK, KBKB, KNKN, KBKN, KRKR and KQKQ to draws after 3 moves (to allow for a quick tactical win. Note that in KQKQ this might not be sound, but that problem would disappear once bitbase probing is added). \r
+The /ruleMoves determine after how many reversible moves the game is adjudicated as a draw. Setting this to 0 turns this option off. Draw claims by the engine are still accepted (by /testClaim) after 50 reversible moves, even if /ruleMoves species a larger number. Note that it is perfectly legal according to FIDE rules to play on after 50 reversible moves, but in tournaments having two engines that want to play on forever is a nuisance in endings like KBNKR, where one of the engines thinks it is ahead and can avoids repeats virtually forever. \r
+The option /repeatsToDraw makes the GUI adjudicate a game as draw after the same position has occurred the specified number of times. If it is set to a value > 3, engines can still claim the draw after 3-fold repeat. \r
+All these options are saved in the winboard.ini file. \r
+\r
+/matchPause=10000 \r
+Determines the number of milliseconds that is paused between two games of a match. Saved in the Winboard.ini.\r
+\r
+Clocks\r
+There is an "Options -> flip Clocks" command, that swaps the position of white and black clocks (convenient in over-the-board matches, where the screen is next to the board, and you want your own time to be displayed on your side of the screen). The clocks can be adjusted in "edit game" mode: right-clicking them adds one minute, left-clicking subtracts one minute. (Also for OTB matches, to keep them synchronized with the official match clock.) The flag-fell condition is now indicated as (!) behind the time.\r
+\r
+Other improvements / changes\r
+Castling rights and e.p. rights are now fully maintained, and considered in legality testing. They are imported from and written to FEN, as is the 50-move counter. The time (in sec, or min:sec) is now always stored together with the PV information to the PGN, if storing the latter was requested (through ticking 'extended PGN info' in "Options -> General..."). The saved time is the Winboard clock time (as opposed to the time reported by the engine).\r
+--------------------------------------------------------------------\r
 Winboard_F.4.3.7 release notes\r
 \r
 This Winboard supports the following new options (shown here with their default values):\r
index 8c4ba3e..32e7081 100644 (file)
 #define PB_Knight                       505\r
 #define PB_Chancellor                   506 /* [HGM] Capablanca pieces */\r
 #define PB_Archbishop                   507\r
+#define IDC_Yes                         520\r
+#define IDC_No                          521\r
 #define EP_White                        600\r
 #define EP_WhitePawn                    601\r
 #define EP_WhiteKnight                  602\r
index 7369520..7302223 100644 (file)
@@ -3319,6 +3319,7 @@ DrawPieceOnDC(HDC hdc, ChessSquare piece, int color, int sqcolor, int x, int y,
 {\r
   HBITMAP oldBitmap;\r
   HBRUSH oldBrush;\r
+  int tmpSize;\r
 \r
   if (appData.blindfold) return;\r
 \r
@@ -3358,6 +3359,7 @@ DrawPieceOnDC(HDC hdc, ChessSquare piece, int color, int sqcolor, int x, int y,
     BitBlt(hdc, x, y, squareSize, squareSize, tmphdc, 0, 0,\r
           sqcolor ? SRCCOPY : NOTSRCCOPY);\r
   } else {\r
+    tmpSize = squareSize;\r
     if(minorSize &&\r
         (piece >= (int)WhiteNightrider && piece <= WhiteGrasshopper ||\r
          piece >= (int)BlackNightrider && piece <= BlackGrasshopper)  ) {\r
@@ -3365,24 +3367,31 @@ DrawPieceOnDC(HDC hdc, ChessSquare piece, int color, int sqcolor, int x, int y,
       /* Bitmaps of smaller size are substituted, but we have to align them */\r
       x += (squareSize - minorSize)>>1;\r
       y += squareSize - minorSize - 2;\r
+      tmpSize = minorSize;\r
     }\r
     if (color || appData.allWhite ) {\r
       oldBitmap = SelectObject(tmphdc, PieceBitmap(piece, WHITE_PIECE));\r
       if( color )\r
               oldBrush = SelectObject(hdc, whitePieceBrush);\r
       else    oldBrush = SelectObject(hdc, blackPieceBrush);\r
-      BitBlt(hdc, x, y, squareSize, squareSize, tmphdc, 0, 0, 0x00B8074A);\r
+      if(appData.upsideDown && !color)\r
+        StretchBlt(hdc, x, y+tmpSize, tmpSize, -tmpSize, tmphdc, 0, 0, tmpSize, tmpSize, 0x00B8074A);\r
+      else\r
+        BitBlt(hdc, x, y, tmpSize, tmpSize, tmphdc, 0, 0, 0x00B8074A);\r
 #if 0\r
       /* Use black piece color for outline of white pieces */\r
       /* Not sure this looks really good (though xboard does it).\r
         Maybe better to have another selectable color, default black */\r
       SelectObject(hdc, blackPieceBrush); /* could have own brush */\r
       SelectObject(tmphdc, PieceBitmap(piece, OUTLINE_PIECE));\r
-      BitBlt(hdc, x, y, squareSize, squareSize, tmphdc, 0, 0, 0x00B8074A);\r
+      BitBlt(hdc, x, y, tmpSize, tmpSize, tmphdc, 0, 0, 0x00B8074A);\r
 #else\r
       /* Use black for outline of white pieces */\r
       SelectObject(tmphdc, PieceBitmap(piece, OUTLINE_PIECE));\r
-      BitBlt(hdc, x, y, squareSize, squareSize, tmphdc, 0, 0, SRCAND);\r
+      if(appData.upsideDown && !color)\r
+        StretchBlt(hdc, x, y+tmpSize, tmpSize, -tmpSize, tmphdc, 0, 0, tmpSize, tmpSize, SRCAND);\r
+      else\r
+        BitBlt(hdc, x, y, tmpSize, tmpSize, tmphdc, 0, 0, SRCAND);\r
 #endif\r
     } else {\r
 #if 0\r
@@ -3393,15 +3402,18 @@ DrawPieceOnDC(HDC hdc, ChessSquare piece, int color, int sqcolor, int x, int y,
         Maybe better to have another selectable color, default medium gray? */\r
       oldBitmap = SelectObject(tmphdc, PieceBitmap(piece, BLACK_PIECE));\r
       oldBrush = SelectObject(hdc, whitePieceBrush); /* could have own brush */\r
-      BitBlt(hdc, x, y, squareSize, squareSize, tmphdc, 0, 0, 0x00B8074A);\r
+      BitBlt(hdc, x, y, tmpSize, tmpSize, tmphdc, 0, 0, 0x00B8074A);\r
       SelectObject(tmphdc, PieceBitmap(piece, SOLID_PIECE));\r
       SelectObject(hdc, blackPieceBrush);\r
-      BitBlt(hdc, x, y, squareSize, squareSize, tmphdc, 0, 0, 0x00B8074A);\r
+      BitBlt(hdc, x, y, tmpSize, tmpSize, tmphdc, 0, 0, 0x00B8074A);\r
 #else\r
       /* Use square color for details of black pieces */\r
       oldBitmap = SelectObject(tmphdc, PieceBitmap(piece, SOLID_PIECE));\r
       oldBrush = SelectObject(hdc, blackPieceBrush);\r
-      BitBlt(hdc, x, y, squareSize, squareSize, tmphdc, 0, 0, 0x00B8074A);\r
+      if(appData.upsideDown)\r
+        StretchBlt(hdc, x, y+tmpSize, tmpSize, -tmpSize, tmphdc, 0, 0, tmpSize, tmpSize, 0x00B8074A);\r
+      else\r
+        BitBlt(hdc, x, y, tmpSize, tmpSize, tmphdc, 0, 0, 0x00B8074A);\r
 #endif\r
     }\r
     SelectObject(hdc, oldBrush);\r
@@ -4706,6 +4718,12 @@ Promotion(HWND hDlg, UINT message, WPARAM wParam, LPARAM lParam)
     ShowWindow(GetDlgItem(hDlg, PB_Bishop), \r
        gameInfo.variant != VariantShogi ?\r
               SW_SHOW : SW_HIDE);\r
+    ShowWindow(GetDlgItem(hDlg, IDC_Yes), \r
+       gameInfo.variant == VariantShogi ?\r
+              SW_SHOW : SW_HIDE);\r
+    ShowWindow(GetDlgItem(hDlg, IDC_No), \r
+       gameInfo.variant == VariantShogi ?\r
+              SW_SHOW : SW_HIDE);\r
     return TRUE;\r
 \r
   case WM_COMMAND: /* message: received a command */\r
index 0f85a93..baf18bd 100644 (file)
@@ -185,8 +185,6 @@ STYLE DS_MODALFRAME | WS_POPUP | WS_VISIBLE | WS_CAPTION | WS_SYSMENU
 CAPTION "Promote to:"\r
 FONT 8, "MS Sans Serif"\r
 BEGIN\r
-    LTEXT           "YES ",IDC_STATIC,49,27,40,14\r
-    RTEXT           "NO ",IDC_STATIC,94,27,40,14\r
     PUSHBUTTON      "Chance&llor",PB_Chancellor,49,4,40,14,NOT WS_TABSTOP\r
     PUSHBUTTON      "&Archbishop",PB_Archbishop,94,4,40,14,NOT WS_TABSTOP\r
     PUSHBUTTON      "&Queen",PB_Queen,4,23,40,14,WS_GROUP\r
@@ -195,6 +193,8 @@ BEGIN
     PUSHBUTTON      "K&night",PB_Knight,139,23,40,14,NOT WS_TABSTOP\r
     PUSHBUTTON      "&King",PB_King,94,4,40,14,NOT WS_TABSTOP\r
     PUSHBUTTON      "&Cancel",IDCANCEL,139,4,40,14,WS_GROUP\r
+    LTEXT           "      YES",IDC_Yes,6,25,36,10\r
+    LTEXT           "       NO",IDC_No,141,25,36,10\r
 END\r
 \r
 ABOUTBOX2 DIALOG DISCARDABLE  22, 17, 281, 223\r