Quoting Caio Marcelo de Oliveira Filho (2018-09-21 16:05:37) > > > -add_shader_test('tests/shaders/glsl-algebraic-add-zero.shader_test') > > > -add_shader_test('tests/spec/glsl-1.50/execution/geometry-basic.shader_test') > > > -add_shader_test('tests/spec/arb_tessellation_shader/execution/sanity.shader_test') > > > -add_shader_test('tests/spec/arb_compute_shader/execution/basic-ssbo.shader_test') > > > +shader_tests = ''' > > > +tests/shaders/glsl-algebraic-add-zero.shader_test > > > +tests/spec/arb_compute_shader/execution/basic-ssbo.shader_test > > > +tests/spec/arb_tessellation_shader/execution/sanity.shader_test > > > +tests/spec/glsl-1.50/execution/geometry-basic.shader_test > > > +''' > > > > How about a list of strings? > > One string + split() gives less boilerplate per line. In this case > either should be fine.
4 characters per line isn't that much boilerplate; and you're not writing what you mean as-is. I'm not NAKing, but I still don't like it :) Dylan
signature.asc
Description: signature
_______________________________________________ Piglit mailing list Piglit@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/piglit