Re: [PATCH v8 0/7] MR198: vkd3d-shader: Add more TPF validations.

[Giovanni Mascellani (＠giomasce)]

Giovanni Mascellani (@giomasce) commented about libs/vkd3d-shader/tpf.c:
>       * other values up one slot. Normalize to SM5.1. */
>      param->idx[2] = param->idx[1];
>      param->idx[1] = param->idx[0];
> -
>    ++param->idx_count;

I don't really know what I'm talking about here, but after reading that my first though is: "Shouldn't we check that `idx_count` is not already three? And shouldn't we zero `idx[0]`?".