Re: [PATCH 2/7] lib/drmtest: Add drmtest_subtest_block macro

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Aug 12, 2013 at 12:20:24PM +0200, Daniel Vetter wrote:
> On Mon, Aug 12, 2013 at 11:26:49AM +0200, Daniel Vetter wrote:
> > On Mon, Aug 12, 2013 at 10:15:06AM +0100, Chris Wilson wrote:
> > > On Mon, Aug 12, 2013 at 11:09:56AM +0200, Daniel Vetter wrote:
> > > > Doesn't do more than an if (drmtest_run_test(name)) right now, but
> > > > as soon as we get a bit of infrastructure to handle test failures and
> > > > skipping, this will get more interesting.
> > > 
> > > Just use drm_subtest("name") { }
> > > 
> > > drm_subtest_block() reads like a function and so one expects "block" to
> > > be a verb.
> > 
> > The intention was to make clear that the macro expects a C code block
> > afterwards. See the later patches for some of the fun I add ... I see that
> > the _block suffix can be misread, but tbh I don't have any other ideas
> > that might properly convey what this thing is. The 2nd best I could think
> > of is do_subtest("name") similar to do {} while (). But that's not great
> > either.
> 
> Ok, I've applied the following three bikesheds onto my tree:
> - s/drmtest_run_subtest/__drmtest_run_subtest/ since that fuction is now
>   internal to drmtest.c (but needs to be in the header due to the fancy
>   macro).
> - s/drmtest_subtest_block/drmtest/subtest/ ... see above.
> - s/drmtest_/igt_/ just because (suggested by Chris on irc).

One more on top:
- s/return igt_retval()/igt_exit()/

I'll push this one the US has woken up an had a chance to comment. There's
too much tree-wide sed and sed-like stuff in this series now. Latest
patches pushed to my private repo.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux