sezero / quakespasm

QuakeSpasm -- A modern, cross-platform Quake game engine based on FitzQuake.
https://sourceforge.net/projects/quakespasm/
GNU General Public License v2.0
244 stars 97 forks source link

Applying FOV gun scaling for the triangle renderer as well #99

Closed Diordany closed 5 months ago

Diordany commented 6 months ago

Issue

The FOV gun scaling wasn't applied to the triangle renderer (with r_showtris = 1), resulting in geometry that differs from the output of the regular renderer:

spasm0000

Patch

I copied the scaling procedure to the triangle renderer which makes them match now:

spasm0001

The scaling procedure was copied from R_DrawAliasModel() into R_DrawAliasModel_ShowTris.

Edit: I've set fov 130, cl_gun_fovscale 1 and r_showtris 1 when I made the screenshots.

sezero commented 6 months ago

@andrei-drexler , @ericwa : This looks good?

sezero commented 5 months ago

OK, I pulled this in. Thanks.