On Mon Feb 19 10:58:34 2024 +0000, Giovanni Mascellani wrote:
Not that I care that much, but I'd claim that the standard construct is `case XXX: vkd3d_unreachable();` rather than breaking and then calling `vkd3d_unreachable()`.
I need either vkd3d_unreachable() at the end of the function, or in a default case. I prefer the former because using 'default' makes it easier to miss when a new case is added. At that point it seems simpler to break rather than use unreachable() twice, but I'd also be satisfied with the latter.