From: H.G. Muller <h.g.muller@hccnet.nl>
Date: Sun, 21 Oct 2012 13:15:01 +0000 (+0200)
Subject: Fix bug in fallback mechanism
X-Git-Url: http://winboard.nl/cgi-bin?a=commitdiff_plain;h=c2c246be40c585f2d7ad2bed6abbc77fcb3d9cb1;p=xboard.git

Fix bug in fallback mechanism

After failure to load a piece from a specified -pieceImageDirectory
the default theme was not tried, because the cairo function to load
from a pgn file returns a valid handle even if the file does not exist.
---

diff --git a/draw.c b/draw.c
index 625c0bd..84fb6a1 100644
--- a/draw.c
+++ b/draw.c
@@ -289,10 +289,10 @@ ScaleOnePiece (int color, int piece)
   if(!pngPieceImages[color][piece]) { // we don't have cached bitmap (implying we did not have cached svg)
     if(*appData.pieceDirectory) { // user specified piece directory
       snprintf(buf, MSG_SIZ, "%s/%s%s.png", appData.pieceDirectory, color ? "Black" : "White", pngPieceNames[piece]);
-      pngPieceImages[color][piece] = img = cairo_image_surface_create_from_png (buf); // try if there are png pieces there
+      img = cairo_image_surface_create_from_png (buf); // try if there are png pieces there
       if(cairo_surface_status(img) != CAIRO_STATUS_SUCCESS) { // there were not
 	svgPieces[color][piece] = LoadSVG(appData.pieceDirectory, color, piece); // so try if he has svg there
-      }
+      } else pngPieceImages[color][piece] = img;
     }
   }