summaryrefslogtreecommitdiff
path: root/src/cairo-paginated-surface.c
diff options
context:
space:
mode:
authorChris Wilson <chris@chris-wilson.co.uk>2010-03-23 16:43:39 +0000
committerChris Wilson <chris@chris-wilson.co.uk>2010-03-23 16:43:39 +0000
commit1ddcd5cf31bb47e9ff18ddf94d0a4648fa70a617 (patch)
tree1f507a9cd24dc52138c01ba1a8730c7a02f734af /src/cairo-paginated-surface.c
parent61ad28fe7d334c63197ae3881d5edd074d63cfec (diff)
downloadcairo-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.c18
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);