[PATCH weston 07/10] tests: Add screenshot recording capability to weston-test

Pekka Paalanen ppaalanen at gmail.com
Mon May 11 04:32:48 PDT 2015


On Sat, 9 May 2015 14:56:54 +0100
Daniel Stone <daniel at fooishbar.org> wrote:

> Hi,
> 
> On 7 May 2015 at 01:44, Bryce Harrington <bryce at osg.samsung.com> wrote:
> > +enum weston_test_screenshot_outcome {
> > +       WESTON_TEST_SCREENSHOT_SUCCESS,
> > +       WESTON_TEST_SCREENSHOT_NO_MEMORY,
> > +       WESTON_TEST_SCREENSHOT_BAD_BUFFER
> > +       };
> 
> This should be declared as an enum in the protocol definition.

Except it's not actually passed around in the protocol at all. It's
just a part of an internal abstraction, that I don't think is useful.
Removing the abstraction would remove these.


Thanks,
pq


More information about the wayland-devel mailing list