On Mon, Nov 30, 2015 at 07:46:37PM -0500, Ilia Mirkin wrote:
> On Mon, Nov 30, 2015 at 7:34 PM, Nanley Chery <[email protected]> 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 <[email protected]>
> >> 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.
>
Oh, I thought you were referring to the '(' in the if statement.
I'll perform the indentation you suggested for the function calls.
-Nanley
_______________________________________________
Piglit mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/piglit
