shithub: puzzles

Info  •  Files  •  Log  •  Branches

Ben Hutchings reports that gcc 4 gives an optimiser warning because
it can't tell that one branch of a particular switch is always
taken. Adding a default clause with an automatic assertion failure
apparently fixes it.

[originally from svn r6148]

--- a/rect.c

+++ b/rect.c

@@ -1348,6 +1348,8 @@

                                 r1.x++;

                             r1.w--;

                             break;

+                          default:     /* should never happen */

+                            assert(!"invalid direction");

                         }

                         if (r1.h > 0 && r1.w > 0)

                             place_rect(params2, grid, r1);