Re: [PATCH v3 0/7] MR66: vkd3d-shader/hlsl: Broaden resources support v2 (PART 1).

[Giovanni Mascellani (＠giomasce)]

Giovanni Mascellani (@giomasce) commented about libs/vkd3d-shader/hlsl.h:
> };
>
> +static inline char hlsl_regset_name(enum hlsl_regset regset)
> +{
> - switch (regset)
> - {
> -
>    case HLSL_REGSET_S:
> -
>        return 's';
> -
>    case HLSL_REGSET_T:
> -
>        return 't';
> -
>    case HLSL_REGSET_U:
> -
>        return 'u';
> -
>    case HLSL_REGSET_NUMERIC:
> -
>    case HLSL_REGSET_VOID:
> -
>        vkd3d_unreachable();
> -
>    break;

You should not need this `break`. The point of `vkd3d_unreachable()` is precisely to free you from the need to handle control flow in such a situation. Also, you're just before the end of the `switch` block anyway.