From: Simon Tatham <anakin@pobox.com>
Date: Mon, 14 Apr 2008 11:32:06 +0000 (+0000)
Subject: Shamelessly pander to compilers whose data flow warning systems
X-Git-Url: https://www.chiark.greenend.org.uk/ucgi/~ian/git?a=commitdiff_plain;h=83121fb826f8ce669aa1d02816bb6adaeee1257b;p=sgt-puzzles.git

Shamelessly pander to compilers whose data flow warning systems
insist that a variable should be initialised in all branches of an
if, instead of just all the non-assertion-failing ones.

[originally from svn r7989]
---

diff --git a/solo.c b/solo.c
index 1879a98..8ec0296 100644
--- a/solo.c
+++ b/solo.c
@@ -2779,10 +2779,10 @@ static game_state *new_game(midend *me, game_params *params, char *desc)
 
 	    if (*desc == '_')
 		c = 0;
-	    else if (*desc >= 'a' && *desc <= 'z')
+	    else {
+                assert(*desc >= 'a' && *desc <= 'z');
 		c = *desc - 'a' + 1;
-	    else
-		assert(!"Shouldn't get here");
+            }
 	    desc++;
 
 	    adv = (c != 25);	       /* 'z' is a special case */