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.