From 09f0e2a0c52750f65ce026410c78c6bdb24d8121 Mon Sep 17 00:00:00 2001
From: Eric Wasylishen <ewasylishen@gmail.com>
Date: Sat, 17 Sep 2022 21:08:44 -0600
Subject: [PATCH] qbsp: preserve brush side texinfo in collision hulls

17k -> 9.8k clipnodes on testmaps\retrojam1_ericw.map

setting the texinfo to 0 was defeating the bsp heuristic, because
texinfo 0 was marked as is_skip, which makes side_t::is_visible()
return false (which makes us not care about splitting those brush sides)
---
 qbsp/brush.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/qbsp/brush.cc b/qbsp/brush.cc
index 8415203e..b6b84faf 100644
--- a/qbsp/brush.cc
+++ b/qbsp/brush.cc
@@ -576,7 +576,7 @@ std::optional<bspbrush_t> LoadBrush(const mapentity_t &src, mapbrush_t &mapbrush
 #endif
 
         auto &dst = brush.sides.emplace_back();
-        dst.texinfo = hullnum.value_or(0) ? 0 : src.texinfo;
+        dst.texinfo = src.texinfo;
         dst.planenum = src.planenum;
         dst.bevel = src.bevel;
         dst.source = &src;