Re: [PATCH v2 0/1] MR234: vkd3d-shader/hlsl: Partially implement Load() for the Buffer object.

[Zebediah Figura (＠zfigura)]

On Tue Jun 13 11:34:09 2023 +0000, Giovanni Mascellani wrote:
> Of course I don't want this issue to become too predominant, given that
> it's just a matter of taste. I mentioned it mostly because at some point
> I thought my suggestions was somewhat aligned with Zeb's tastes too (but
> I could be wrong).
> My point, though, is not really about the `else` (which I still prefer
> to have, if the two branches are somewhat symmetrical), but rather the
> fact of "hand-inlining" the default helper. I.e., I consider this a
> little suboptimal:
> if (condition)
>     return f();
> return g();
>
> but I consider this significantly worse:
> if (condition)
>     return f();
> // inline g here
>
> (though, in the end, acceptable)

I don't think I can bring myself to care about this particular issue.