SplitBrush now uses the result of BoundBrush to detect if the split
authorForest Hale <lordhavoc>
Fri, 9 Mar 2007 02:10:22 +0000 (02:10 +0000)
committerForest Hale <lordhavoc>
Fri, 9 Mar 2007 02:10:22 +0000 (02:10 +0000)
brush is invalid rather than using its own code (it already called
BoundBrush to update the bounds, it may as well use the result as well)

git-svn-id: svn://svn.icculus.org/gtkradiant/GtkRadiant/trunk@157 8a3a26a2-13c4-0310-b231-cf6edde360e5

tools/quake3/q3map2/brush.c

index 44e99db..57b5d81 100644 (file)
@@ -906,18 +906,10 @@ void SplitBrush( brush_t *brush, int planenum, brush_t **front, brush_t **back )
        // see if we have valid polygons on both sides
        for (i=0 ; i<2 ; i++)
        {
-               BoundBrush (b[i]);
-               for (j=0 ; j<3 ; j++)
+               if (b[i]->numsides < 3 || !BoundBrush (b[i]))
                {
-                       if (b[i]->mins[j] < MIN_WORLD_COORD || b[i]->maxs[j] > MAX_WORLD_COORD)
-                       {
+                       if (b[i]->numsides >= 3)
                                Sys_FPrintf (SYS_VRB,"bogus brush after clip\n");
-                               break;
-                       }
-               }
-
-               if (b[i]->numsides < 3 || j < 3)
-               {
                        FreeBrush (b[i]);
                        b[i] = NULL;
                }