On Thu, Dec 05, 2013 at 04:38:19PM +0100, Marek Ch wrote: > Hi, > > I did similar thing like is in Weston for testing static functions. Also, I > wrote > few wl_buffer unit tests. > I appreaciate any comments :) > > P. S.: based on mailing here: > http://lists.freedesktop.org/archives/wayland-devel/2013-November/012299.html
Looks good to me! I reviewed the three patches plus the docs patch. Reviewed-by: Bryce Harrington <[email protected]> > Thanks, > Marek Ch > > > Marek Ch (3): > wayland-private: add define for testing static functions > wayland-private: move struct wl_buffer into wayland-private.h > tests: add tests for wl_buffer_* > > src/connection.c | 18 +--- > src/wayland-private.h | 24 +++++ > tests/Makefile.am | 12 ++- > tests/buffer-test.c | 247 > ++++++++++++++++++++++++++++++++++++++++++++++++++ > 4 files changed, 286 insertions(+), 15 deletions(-) > create mode 100644 tests/buffer-test.c > > -- > 1.8.4.2 > > _______________________________________________ > wayland-devel mailing list > [email protected] > http://lists.freedesktop.org/mailman/listinfo/wayland-devel _______________________________________________ wayland-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/wayland-devel
