shithub: puzzles

Info  •  Files  •  Log  •  Branches

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]

--- a/solo.c

+++ b/solo.c

@@ -2779,10 +2779,10 @@

 	    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 */