[cairo] [PATCH] test : Maintain consistency in the usage of xcalloc

Petr Kobalíček kobalicek.petr at gmail.com
Tue May 6 03:36:08 PDT 2014


Will this compile everywhere?

Best regards
Petr Kobalicek


On Tue, May 6, 2014 at 12:25 PM, RAVI NANJUNDAPPA <nravi.n at samsung.com>wrote:

> Hi,
>
> Kindly ignore the previous mail. That was sent in the HTML format, by
> mistake.
> Sorry for the inconvenience caused.
>
> I've done the basic build test with the changes and the build works fine.
> I've also tried to execute "negative-stride-image" and "stride-12-image"
> tests with and without these changes.
> In both the scenarios the test results are same.
>
> Thanks and Best Regards,
> N Ravi
>
> From: cairo [mailto:cairo-bounces at cairographics.org] On Behalf Of RAVI
> NANJUNDAPPA
> Sent: Tuesday, May 06, 2014 3:52 PM
> To: cairo at cairographics.org
> Subject: Re: [cairo] [PATCH] test : Maintain consistency in the usage of
> xcalloc
>
> Hi,
>
> I've done the basic build test with the changes and the build works fine.
> I've also tried to execute "negative-stride-image" and "stride-12-image"
> tests with and without these changes.
> In both the scenarios the results are same.
>
> Thanks and Best Regards,
> N Ravi
>
> ------- Original Message -------
> Sender : RAVI NANJUNDAPPA<nravi.n at samsung.com> Technical
> Lead/SRI-Bangalore-TZN/Samsung Electronics
> Date : May 06, 2014 15:44 (GMT+05:30)
> Title : [PATCH] test : Maintain consistency in the usage of xcalloc
>
> xcalloc is a wrapper for calloc. If it can't allocate memory then it
> terminates the program. The current changes replace the usages of calloc
> by xcalloc in tests.
>
> Signed-off-by: Ravi Nanjundappa
> ---
> test/negative-stride-image.c |    2 +-
> test/stride-12-image.c       |    2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/test/negative-stride-image.c b/test/negative-stride-image.c
> index a407b8c..dfc8d64 100644
> --- a/test/negative-stride-image.c
> +++ b/test/negative-stride-image.c
> @@ -41,7 +41,7 @@ draw (cairo_t *cr, int width, int height)
>      png = cairo_test_create_surface_from_png (ctx, png_filename);
>
>      stride = cairo_format_stride_for_width (format, width);
> -    data = calloc (stride, height);
> +    data = xcalloc (stride, height);
>      inv = cairo_image_surface_create_for_data (data + stride * (height -
> 1),
>        format, width, height, -stride);
>
> diff --git a/test/stride-12-image.c b/test/stride-12-image.c
> index 21bd8b0..5650e98 100644
> --- a/test/stride-12-image.c
> +++ b/test/stride-12-image.c
> @@ -41,7 +41,7 @@ draw (cairo_t *cr, int width, int height)
>      png = cairo_test_create_surface_from_png (ctx, png_filename);
>
>      stride = cairo_format_stride_for_width (format, width) + 12;
> -    data = calloc (stride, height);
> +    data = xcalloc (stride, height);
>      src = cairo_image_surface_create_for_data (data, format,
>        width, height, stride);
>
> --
> 1.7.9.5
>
>
>
>
> --
> cairo mailing list
> cairo at cairographics.org
> http://lists.cairographics.org/mailman/listinfo/cairo
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.cairographics.org/archives/cairo/attachments/20140506/b9547021/attachment.html>


More information about the cairo mailing list