[PATCH wayland] tests: rename NO_ASSERT_LEAK_CHECK
Bill Spitzak
spitzak at gmail.com
Wed Nov 26 11:01:46 PST 2014
On 11/26/2014 06:25 AM, Marek Chalupa wrote:
>
>
> On 20 November 2014 at 23:59, Bryce Harrington <bryce at osg.samsung.com
> <mailto:bryce at osg.samsung.com>> wrote:
>
> On Wed, Nov 19, 2014 at 12:00:10PM +0100, Marek Chalupa wrote:
> > This env variable is used for turning off the leak checks in tests.
> > Rename it to WAYLAND_TESTS_NO_LEAK_CHECK, so that it is consistent
> > with WAYLAND_TESTS_NO_TIMEOUTS.
>
> Should these both be 'WAYLAND_TEST_'? In weston there is already a
> WESTON_TEST_CLIENT_PATH.
>
> If WAYLAND_TESTS_* is better, then maybe WESTON_TEST_CLIENT_PATH should
> be changed for consistency.
The plural seems wrong to me. Calling them WAYLAND_TEST_* I think is
much more what I would expect. The reason is that this is not
controlling "tests" (a plural), but instead controlling "testing" (a
singular).
More information about the wayland-devel
mailing list