From 7fa24021c8c16150b333d8dcbfb4e45509b6012e Mon Sep 17 00:00:00 2001 From: Kevin Shanahan Date: Wed, 6 Mar 2013 22:44:12 +1030 Subject: [PATCH] qbsp: move the hint split check to the correct place Check that the hint face would actually get split before we discount the candidate split plane... Signed-off-by: Kevin Shanahan --- qbsp/solidbsp.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/qbsp/solidbsp.c b/qbsp/solidbsp.c index 4607eb9f..859fdd59 100644 --- a/qbsp/solidbsp.c +++ b/qbsp/solidbsp.c @@ -342,12 +342,12 @@ ChoosePlaneFromList(surface_t *surfaces, vec3_t mins, vec3_t maxs) /* Don't penalize for splitting skip faces */ if (flags & TEX_SKIP) continue; - /* Never split a hint face except with a hint */ - if (!hintsplit && (flags & TEX_HINT)) { - splits = INT_MAX; - break; - } if (FaceSide(face, plane) == SIDE_ON) { + /* Never split a hint face except with a hint */ + if (!hintsplit && (flags & TEX_HINT)) { + splits = INT_MAX; + break; + } splits++; if (splits >= minsplits) break;