On Sun May 26 12:43:21 2024 +0000, Connor McAdams wrote:
changed this line in [version 2 of the diff](/wine/wine/-/merge_requests/5666/diffs?diff_id=115190&start_sha=04c51c5eac6f69386870f2ea96e616838cda38be#8fffeae09861a99974e55842964630933974cdc8_2027_1978)
The reason for the int/uint mix is because some of the functions calling this are pulling those values from a `RECT` structure which contains signed ints. I wrote this under the assumption that negative values in e.g the source rectangle argument `D3DXLoadSurfaceFromMemory` were expected to work, but after writing tests it seems like that's more of an unintended thing that only works on 32-bit. I've switched them all to uints in the most recent revision.
Initially I did pack these into a box structure, but that started to feel a bit messy to me, in particular because there were 3 different types of structures in use (`RECT` for d3d9 surfaces, `D3DBOX` for d3d9 volumes, and then `D3D10_BOX`/`D3D11_BOX` for d3d10/d3d11. Rather than add a call to pack a structure before each `d3dx_pixels_init` call, I figured it'd make more sense to just have the values as arguments instead. I can change that if you'd like. :)