From 4183912c60a9e3e8380ec439899c0db9ed15f50b Mon Sep 17 00:00:00 2001 From: toasterbabe <rollerorbital@gmail.com> Date: Tue, 9 Aug 2016 13:01:56 +0100 Subject: [PATCH] Please don't assume indentation carries intent, especially since a break that exclusively follows an I_Error will never run --- src/r_things.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/r_things.c b/src/r_things.c index 5e9d93b70e..2f19af8338 100644 --- a/src/r_things.c +++ b/src/r_things.c @@ -331,11 +331,10 @@ static boolean R_AddSingleSpriteDef(const char *sprname, spritedef_t *spritedef, case SRF_2D: // both Left and Right rotations // we test to see whether the left and right slots are present if ((sprtemp[frame].lumppat[2] == LUMPERROR) || (sprtemp[frame].lumppat[6] == LUMPERROR)) - { I_Error("R_AddSingleSpriteDef: Sprite %s frame %c is missing rotations", sprname, R_Frame2Char(frame)); - break; - } + break; + default: // must have all 8 frames for (rotation = 0; rotation < 8; rotation++) -- GitLab