Re: [PATCH v19 0/4] MR346: tests/shader-runner: Test DXIL shaders.

[Giovanni Mascellani (＠giomasce)]

Giovanni Mascellani (@giomasce) commented about tests/shader_runner.c:
> for (;;)
>      {
> -        char *ret = fgets(line, sizeof(line), f);
> +        char *ret = fgets(line_buffer, sizeof(line_buffer), f);
> +        const char *line = line_buffer;
>
> ++line_number;
>
> -        if (!ret || line[0] == '[')
> +        if (!ret || *line == '[')

I'm pretty sure this ship has sailed and I won't insist on this, but to me `line[0]` feels much more expressive and appropriate than `*line` for accessing an array element.

-- 
https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/346#note_47465