Merge PR 'Only use BoxTouchingVisibleLeafs when not using r_novis'

hemebond · web-flow · commit c443fbad631c · 2024-08-16T00:09:14.000+10:00
Updates the entity visibility check in `R_View_UpdateEntityVisible` so that with `r_novis 1` entities are rendered when the player is outside of the level. Resolves DarkPlacesEngine/DarkPlaces#188 See DarkPlacesEngine/DarkPlaces#192 Signed-off-by: bones_was_here <bones_was_here@xonotic.au>
diff --git a/gl_rmain.c b/gl_rmain.c
@@ -4007,7 +4007,7 @@ static void R_View_UpdateEntityVisible (void)
 	if (!r_drawexteriormodel.integer)
 		renderimask |= RENDER_EXTERIORMODEL;
 	memset(r_refdef.viewcache.entityvisible, 0, r_refdef.scene.numentities);
-	if (r_refdef.scene.worldmodel && r_refdef.scene.worldmodel->brush.BoxTouchingVisibleLeafs)
+	if (r_refdef.scene.worldmodel && !r_novis.integer && r_refdef.scene.worldmodel->brush.BoxTouchingVisibleLeafs)
 	{
 		// worldmodel can check visibility
 		for (i = 0;i < r_refdef.scene.numentities;i++)

Original file line number	Diff line number	Diff line change
`@@ -4007,7 +4007,7 @@ static void R_View_UpdateEntityVisible (void)`
`4007`	`4007`	`if (!r_drawexteriormodel.integer)`
`4008`	`4008`	`renderimask \|= RENDER_EXTERIORMODEL;`
`4009`	`4009`	`memset(r_refdef.viewcache.entityvisible, 0, r_refdef.scene.numentities);`
`4010`		`- if (r_refdef.scene.worldmodel && r_refdef.scene.worldmodel->brush.BoxTouchingVisibleLeafs)`
	`4010`	`+ if (r_refdef.scene.worldmodel && !r_novis.integer && r_refdef.scene.worldmodel->brush.BoxTouchingVisibleLeafs)`
`4011`	`4011`	`{`
`4012`	`4012`	`// worldmodel can check visibility`
`4013`	`4013`	`for (i = 0;i < r_refdef.scene.numentities;i++)`