[Piglit] [PATCH] khr_texture_compression_astc: Enable subtest reports

[Ilia Mirkin]

On Mon, Nov 30, 2015 at 7:34 PM, Nanley Chery <nanleychery at gmail.com> wrote:
> On Mon, Nov 30, 2015 at 03:50:04PM -0500, Ilia Mirkin wrote:
>> On Wed, Oct 28, 2015 at 7:55 PM, Nanley Chery <nanleychery at gmail.com> wrote:
>> > +
>> > +               /* Load texture for current submode and block size */
>> > +               load_texture("compressed", tests[subtest],
>> > +                       block_dim_str[block_dims],
>> > +                       &tex_compressed);
>> > +               if (!check_error) {
>> > +                       load_texture("decompressed", tests[subtest],
>> > +                       block_dim_str[block_dims],
>>
>> Here and elsewhere, this needs to be indented to the (, or past it...
>> otherwise it's unreadable.
>>
>
> Isn't this the coding style of Piglit? See this snippet in HACKING:
> "* Indent with 8-column tabs"

By that logic, having 0 indentation on every line would be fine. If
you have a function call, and you have arguments on the next line,
those arguments need to be indented deeper than that function call.

  -ilia