qbsp: remove treating __TB_empty as skip

ericwa · Nov 23, 2024 · 4e11e3e · 4e11e3e
1 parent 9d6ce6d
commit 4e11e3e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/qbsp/map.cc b/qbsp/map.cc
@@ -511,8 +511,6 @@ static bool IsSkipName(const char *name)
         return true;
     if (!Q_strcasecmp(name, "null")) // zhlt compat
         return true;
-    if (!Q_strcasecmp(name, "__TB_empty"))
-        return true;
     return false;
 }
 

diff --git a/tests/test_qbsp_q2.cc b/tests/test_qbsp_q2.cc
@@ -734,8 +734,8 @@ TEST(testmapsQ2, tbCleanup)
     const auto [bsp, bspx, prt] = LoadTestmapQ2("q2_tb_cleanup.map");
 
     {
-        SCOPED_TRACE("check that __TB_empty was converted to skip");
-        EXPECT_EQ(nullptr, BSP_FindFaceAtPoint(&bsp, &bsp.dmodels[0], {0, 0, 0}));
+        SCOPED_TRACE("check that __TB_empty was not converted to skip");
+        EXPECT_NE(nullptr, BSP_FindFaceAtPoint(&bsp, &bsp.dmodels[0], {0, 0, 0}));
     }
 
     {