<div dir="ltr">On Thu, Nov 20, 2014 at 5:50 PM, Chris Wilson <span dir="ltr"><<a href="mailto:chris@chris-wilson.co.uk" target="_blank">chris@chris-wilson.co.uk</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">On Thu, Nov 20, 2014 at 05:22:31PM +0100, Andrea Canciani wrote:<br>
> The pattern created by cairo_pattern_create_rgb() is owned by the<br>
> caller, hence it needs to be released.<br>
><br>
> Fixes:<br>
><br>
> malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)<br>
> _cairo_pattern_create_solid (cairo-pattern.c:605)<br>
> _cairo_pattern_create_in_error (cairo-pattern.c:628)<br>
> cairo_pop_group (cairo.c:552)<br>
> test_cairo_push_group (api-special-cases.c:157)<br>
> preamble (api-special-cases.c:1766)<br>
> main (cairo-test-runner.c:228)<br>
> ---<br>
>  test/api-special-cases.c | 4 +++-<br>
>  1 file changed, 3 insertions(+), 1 deletion(-)<br>
><br>
> diff --git a/test/api-special-cases.c b/test/api-special-cases.c<br>
> index e861152..c4a754d 100644<br>
> --- a/test/api-special-cases.c<br>
> +++ b/test/api-special-cases.c<br>
> @@ -181,7 +181,9 @@ test_cairo_set_operator (cairo_t *cr)<br>
>  static cairo_test_status_t<br>
>  test_cairo_set_source (cairo_t *cr)<br>
>  {<br>
> -    cairo_set_source (cr, cairo_pattern_create_rgb (0, 0, 0));<br>
<br>
</span>cairo_set_source_rgb (cr, 0, 0, 0) ?<br></blockquote><div><br></div><div>I know that it would be more natural to code it the way you suggest, but I think that the purpose of that part of the test is exactly to check cairo_set_source().<br></div><div>The cairo_set_source_rgb() function is tested in test_cairo_set_source_rgb(), exactly with the code you proposed.<br><br></div><div>Andrea<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div class=""><div class="h5"><br>
> +    cairo_pattern_t *source = cairo_pattern_create_rgb (0, 0, 0);<br>
> +    cairo_set_source (cr, source);<br>
> +    cairo_pattern_destroy (source);<br>
<br>
</div></div><span class=""><font color="#888888">--<br>
Chris Wilson, Intel Open Source Technology Centre<br>
</font></span></blockquote></div><br></div></div>