[PATCH 4/4] tests: Use one shared xzalloc implementation

Marek Chalupa mchqwerty at gmail.com
Fri Nov 28 03:47:21 PST 2014


I've had patch doing exactly the same in my wip branch locally for some
time now, so just:

Reviewed-by: Marek Chalupa <mchqwerty at gmail.com>

On 20 November 2014 at 02:18, Bryce Harrington <bryce at osg.samsung.com>
wrote:

> Since this is an inlined function, move it to a common header file.
>
> Signed-off-by: Bryce Harrington <bryce at osg.samsung.com>
> ---
>  tests/presentation-test.c         | 11 -----------
>  tests/weston-test-client-helper.c | 11 -----------
>  tests/weston-test-client-helper.h | 11 +++++++++++
>  3 files changed, 11 insertions(+), 22 deletions(-)
>
> diff --git a/tests/presentation-test.c b/tests/presentation-test.c
> index f97e5b3..59cca8e 100644
> --- a/tests/presentation-test.c
> +++ b/tests/presentation-test.c
> @@ -31,17 +31,6 @@
>  #include "weston-test-client-helper.h"
>  #include "presentation_timing-client-protocol.h"
>
> -static inline void *
> -xzalloc(size_t size)
> -{
> -       void *p;
> -
> -       p = calloc(1, size);
> -       assert(p);
> -
> -       return p;
> -}
> -
>  static struct presentation *
>  get_presentation(struct client *client)
>  {
> diff --git a/tests/weston-test-client-helper.c
> b/tests/weston-test-client-helper.c
> index eadc29b..e374a96 100644
> --- a/tests/weston-test-client-helper.c
> +++ b/tests/weston-test-client-helper.c
> @@ -32,17 +32,6 @@
>  #include "../shared/os-compatibility.h"
>  #include "weston-test-client-helper.h"
>
> -static inline void *
> -xzalloc(size_t size)
> -{
> -       void *p;
> -
> -       p = calloc(1, size);
> -       assert(p);
> -
> -       return p;
> -}
> -
>  int
>  surface_contains(struct surface *surface, int x, int y)
>  {
> diff --git a/tests/weston-test-client-helper.h
> b/tests/weston-test-client-helper.h
> index 684afc6..2cf3d8f 100644
> --- a/tests/weston-test-client-helper.h
> +++ b/tests/weston-test-client-helper.h
> @@ -101,6 +101,17 @@ struct surface {
>         void *data;
>  };
>
> +static inline void *
> +xzalloc(size_t size)
> +{
> +        void *p;
> +
> +        p = calloc(1, size);
> +        assert(p);
> +
> +        return p;
> +}
> +
>  struct client *
>  client_create(int x, int y, int width, int height);
>
> --
> 1.9.1
>
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20141128/33a9a9ce/attachment-0001.html>


More information about the wayland-devel mailing list