Proposals / #1124

Fix decompiler type confusion in RENDERER_D3D_BuildWallTileVerticesNoPerspective loop offset

rejected · by VTSTech · submitted 2026-11-03, rejected 2026-11-04

The decompiler incorrectly reused the `byte` parameter `nParam4` as the loop offset accumulator `_nParam4`, showing it as `byte` (uint8_t) which wraps at 256. The actual assembly uses a 32-bit stack slot (the reused `pTexColorTable` parameter slot at EBP+0x18) incremented by 0x14 per iteration. The database already has a local variable `nSubtileByteOffset` at Stack[-0x10] typed as `undefined4` (4 bytes). Retyping it to `uint32_t` makes the correct type explicit and prevents the decompiler from confusing it with the `byte` parameter.

Changes 1, applied together

1. Retype variable win 0x006b87a0 Stack[-0x10]

   typeundefined4→uint32_t