diff options
author | Chris Wilson <chris@chris-wilson.co.uk> | 2010-03-23 16:43:39 +0000 |
---|---|---|
committer | Chris Wilson <chris@chris-wilson.co.uk> | 2010-03-23 16:43:39 +0000 |
commit | 1ddcd5cf31bb47e9ff18ddf94d0a4648fa70a617 (patch) | |
tree | 1f507a9cd24dc52138c01ba1a8730c7a02f734af /src/cairo-paginated-surface.c | |
parent | 61ad28fe7d334c63197ae3881d5edd074d63cfec (diff) | |
download | cairo-1ddcd5cf31bb47e9ff18ddf94d0a4648fa70a617.tar.gz |
clip: Remove the redundant _cairo_clip_init_rectangle()
As _cairo_clip_init_rectangle() is equivalent and more importantly more
clearly written as:
_cairo_clip_init(&clip);
if (status = _cairo_clip_rectangle(&clip, &rect)) {
_cairo_clip_fini(&fini);
return status;
}
perform the transformation and in the process catch a few mistakes along
error paths.
Diffstat (limited to 'src/cairo-paginated-surface.c')
-rw-r--r-- | src/cairo-paginated-surface.c | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/src/cairo-paginated-surface.c b/src/cairo-paginated-surface.c index 4d7dc8fe4..f8c99f83a 100644 --- a/src/cairo-paginated-surface.c +++ b/src/cairo-paginated-surface.c @@ -277,17 +277,17 @@ _paint_fallback_image (cairo_paginated_surface_t *surface, * filtering (if possible) to avoid introducing potential artifacts. */ pattern.base.filter = CAIRO_FILTER_NEAREST; - status = _cairo_clip_init_rectangle (&clip, rect); - if (unlikely (status)) - goto CLEANUP_IMAGE; - - status = _cairo_surface_paint (surface->target, - CAIRO_OPERATOR_SOURCE, - &pattern.base, &clip); - - _cairo_clip_reset (&clip); + _cairo_clip_init (&clip); + status = _cairo_clip_rectangle (&clip, rect); + if (likely (status == CAIRO_STATUS_SUCCESS)) { + status = _cairo_surface_paint (surface->target, + CAIRO_OPERATOR_SOURCE, + &pattern.base, &clip); + } + _cairo_clip_fini (&clip); _cairo_pattern_fini (&pattern.base); + CLEANUP_IMAGE: cairo_surface_destroy (image); |