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. -ilia _______________________________________________ Piglit mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/piglit
