diff options
author | Uli Schlachter <psychon@znc.in> | 2014-03-14 13:47:38 +0100 |
---|---|---|
committer | Uli Schlachter <psychon@znc.in> | 2014-03-21 14:09:37 +0100 |
commit | 372c8c314a93e0efba53319a120a9ccc2c99658b (patch) | |
tree | 1a92f2d179e7917c15b57563d8acf4c7f0477453 /src/cairo-image-mask-compositor.c | |
parent | bc05dbccd7dd2f137038479b30ed71c98ff81da6 (diff) |
mask compositor: Set a check_composite method
Commit 503b6b9e2ea6 added a check_composite method to the mask compositor, but
only added it to one of the existing implementations. This commit fixes that.
In cairo-image-compositor.c, there is already a check_composite method which
just returns success for the traps compositor. This commit makes the mask
compositor use that one.
I don't want to say much about cairo-image-mask-compositor.c except that I
wondered why this file and the file above both define a non-static function
called _cairo_image_mask_compositor_get(). In my opinion, that file should just
be deleted, since it confuses e.g. ctags, but I'll let someone else clean this
up.
Fixes 493 crashes in the test suite for the test-mask target.
Signed-off-by: Uli Schlachter <psychon@znc.in>
Tested-by: Bryce Harrington <b.harrington@samsung.com>
Diffstat (limited to 'src/cairo-image-mask-compositor.c')
-rw-r--r-- | src/cairo-image-mask-compositor.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/cairo-image-mask-compositor.c b/src/cairo-image-mask-compositor.c index 33fd6dd87..bb990dd65 100644 --- a/src/cairo-image-mask-compositor.c +++ b/src/cairo-image-mask-compositor.c @@ -48,6 +48,8 @@ #include "cairo-compositor-private.h" #include "cairo-region-private.h" +#error This file isn't included in any Makefile + static cairo_int_status_t acquire (void *abstract_dst) { @@ -398,6 +400,7 @@ _cairo_image_mask_compositor_get (void) compositor.draw_image = draw_image; compositor.fill_rectangles = fill_rectangles; compositor.fill_boxes = fill_boxes; +#error check_composite must never be NULL, because it gets called without a NULL pointer check //compositor.check_composite = check_composite; compositor.composite = composite; //compositor.check_composite_boxes = check_composite_boxes; |