[PATCH vkd3d 1/6] vkd3d-shader/hlsl: Allow mixing "in" and "out" with other modifiers.
Zebediah Figura
zfigura at codeweavers.com
Thu Mar 24 11:41:20 CDT 2022
On 3/23/22 07:44, Giovanni Mascellani wrote:
> Signed-off-by: Giovanni Mascellani <gmascellani at codeweavers.com>
> ---
> I don't think it is wrong as it is, but I would have found more
> idiomatic to put the check in declare_vars() outside of the loop
> iterating over all the variable declarations, given that it doesn't
> depend on the specific declaration. This way if multiple variables are
> declared with in or out modifiers in the same statement (as in "in int
> x, y, z;") the error message is uselessly outputted many times.
I don't know about idiomatic, but yes, they should be moved for the
reason you describe. It's a preëxisting issue, though, for what it's worth.
More information about the wine-devel
mailing list