Re: [PATCH 0/14] MR38: vkd3d-shader/hlsl: Add field-level documentation to the most important structs in hlsl.h.

[Giovanni Mascellani (＠giomasce)]

Giovanni Mascellani (@giomasce) commented about libs/vkd3d-shader/hlsl.h:
>  HLSL_ROW_MAJOR
>
> };
>
> +/* An HLSL data type. */
>  struct hlsl_type
>  {
> - /* Linked list entry to store this hlsl_type in the context's linked list of hlsl_types. */
> struct list entry;

From the field declaration it's already quite obvious that we are dealing with a linked list of objects of type `hlsl_type`, so I wouldn't repeat that piece of information. The only useful thing this I think should be mentioned here is which `struct list` this field chains to, and everything else descends from that. So I would write something like "Chained with hlsl_ctx->types". Though I am not sure if "chained" is really the best possible word.

The same mostly applies to all other `struct list`, `struct rb_entry` and `struct rb_tree`.