Attachment 9292 Details for Bug 10036 – reverting to a previous version of patch after bugfix

[patch] reverting to a previous version of patch after bugfix

line-width.patch (text/plain), 7.28 KB, created by Brian Ewins on 2007-03-25 14:49:28 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Brian Ewins

Created: 2007-03-25 14:49:28 UTC

Size: 7.28 KB

patch

obsolete

>commit c7a799f4796671158f35abfbf118b9a468f2423d
>Author: Brian Ewins <Brian.Ewins@gmail.com>
>Date:   Sun Mar 25 23:33:10 2007 +0100
>
>    [quartz] ensure that line widths are scaled.
>    
>    cairo-gstate applies the ctm to the coordinates used in paths,
>    but not to the line width. In quartz this ends up drawing unscaled
>    lines. This is a minimal fix - it undoes the scaling applied to the
>    points and then draws the path scaled correctly.
>
>diff --git a/src/cairo-quartz-surface.c b/src/cairo-quartz-surface.c
>index 3617b1e..c57fee8 100644
>--- a/src/cairo-quartz-surface.c
>+++ b/src/cairo-quartz-surface.c
>@@ -101,6 +101,11 @@ static void quartz_image_to_png (CGImageRef, char *dest);
>  * Cairo path -> Quartz path conversion helpers
>  */
> 
>+typedef struct _quartz_stroke {
>+    CGContextRef cgContext;
>+    cairo_matrix_t *ctm_inverse;
>+} quartz_stroke_t;
>+
> /* cairo path -> mutable path */
> static cairo_status_t
> _cairo_path_to_quartz_path_move_to (void *closure, cairo_point_t *point)
>@@ -140,8 +145,14 @@ static cairo_status_t
> _cairo_path_to_quartz_context_move_to (void *closure, cairo_point_t *point)
> {
>     //ND((stderr, "moveto: %f %f\n", _cairo_fixed_to_double(point->x), _cairo_fixed_to_double(point->y)));
>-    CGContextMoveToPoint ((CGContextRef) closure,
>-			  _cairo_fixed_to_double(point->x), _cairo_fixed_to_double(point->y));
>+    quartz_stroke_t *stroke = (quartz_stroke_t *)closure;
>+    double x = _cairo_fixed_to_double (point->x);
>+    double y = _cairo_fixed_to_double (point->y);
>+
>+    if (stroke->ctm_inverse)
>+	cairo_matrix_transform_point (stroke->ctm_inverse, &x, &y);
>+
>+    CGContextMoveToPoint (stroke->cgContext, x, y);
>     return CAIRO_STATUS_SUCCESS;
> }
> 
>@@ -149,12 +160,17 @@ static cairo_status_t
> _cairo_path_to_quartz_context_line_to (void *closure, cairo_point_t *point)
> {
>     //ND((stderr, "lineto: %f %f\n",  _cairo_fixed_to_double(point->x), _cairo_fixed_to_double(point->y)));
>-    if (CGContextIsPathEmpty ((CGContextRef) closure))
>-	CGContextMoveToPoint ((CGContextRef) closure,
>-			      _cairo_fixed_to_double(point->x), _cairo_fixed_to_double(point->y));
>+    quartz_stroke_t *stroke = (quartz_stroke_t *)closure;
>+    double x = _cairo_fixed_to_double (point->x);
>+    double y = _cairo_fixed_to_double (point->y);
>+    
>+    if (stroke->ctm_inverse)
>+	cairo_matrix_transform_point (stroke->ctm_inverse, &x, &y);
>+
>+    if (CGContextIsPathEmpty (stroke->cgContext))
>+	CGContextMoveToPoint (stroke->cgContext, x, y);
>     else
>-	CGContextAddLineToPoint ((CGContextRef) closure,
>-				 _cairo_fixed_to_double(point->x), _cairo_fixed_to_double(point->y));
>+	CGContextAddLineToPoint (stroke->cgContext, x, y);
>     return CAIRO_STATUS_SUCCESS;
> }
> 
>@@ -165,11 +181,22 @@ _cairo_path_to_quartz_context_curve_to (void *closure, cairo_point_t *p0, cairo_
>     //		   _cairo_fixed_to_double(p0->x), _cairo_fixed_to_double(p0->y),
>     //		   _cairo_fixed_to_double(p1->x), _cairo_fixed_to_double(p1->y),
>     //		   _cairo_fixed_to_double(p2->x), _cairo_fixed_to_double(p2->y)));
>+    quartz_stroke_t *stroke = (quartz_stroke_t *)closure;
>+    double x0 = _cairo_fixed_to_double (p0->x);
>+    double y0 = _cairo_fixed_to_double (p0->y);
>+    double x1 = _cairo_fixed_to_double (p1->x);
>+    double y1 = _cairo_fixed_to_double (p1->y);
>+    double x2 = _cairo_fixed_to_double (p2->x);
>+    double y2 = _cairo_fixed_to_double (p2->y);
>+
>+    if (stroke->ctm_inverse) {
>+	cairo_matrix_transform_point (stroke->ctm_inverse, &x0, &y0);
>+	cairo_matrix_transform_point (stroke->ctm_inverse, &x1, &y1);
>+	cairo_matrix_transform_point (stroke->ctm_inverse, &x2, &y2);
>+    }
> 
>-    CGContextAddCurveToPoint ((CGContextRef) closure,
>-			      _cairo_fixed_to_double(p0->x), _cairo_fixed_to_double(p0->y),
>-			      _cairo_fixed_to_double(p1->x), _cairo_fixed_to_double(p1->y),
>-			      _cairo_fixed_to_double(p2->x), _cairo_fixed_to_double(p2->y));
>+    CGContextAddCurveToPoint (stroke->cgContext,
>+			      x0, y0, x1, y1, x2, y2);
>     return CAIRO_STATUS_SUCCESS;
> }
> 
>@@ -177,7 +204,8 @@ static cairo_status_t
> _cairo_path_to_quartz_context_close_path (void *closure)
> {
>     //ND((stderr, "closepath\n"));
>-    CGContextClosePath ((CGContextRef) closure);
>+    quartz_stroke_t *stroke = (quartz_stroke_t *)closure;
>+    CGContextClosePath (stroke->cgContext);
>     return CAIRO_STATUS_SUCCESS;
> }
> 
>@@ -1108,6 +1136,7 @@ _cairo_quartz_surface_fill (void *abstract_surface,
>     cairo_quartz_surface_t *surface = (cairo_quartz_surface_t *) abstract_surface;
>     cairo_int_status_t rv = CAIRO_STATUS_SUCCESS;
>     cairo_quartz_action_t action;
>+    quartz_stroke_t stroke;
> 
>     ND((stderr, "%p _cairo_quartz_surface_fill op %d source->type %d\n", surface, op, source->type));
> 
>@@ -1126,7 +1155,10 @@ _cairo_quartz_surface_fill (void *abstract_surface,
>     }
> 
>     CGContextBeginPath (surface->cgContext);
>-    _cairo_quartz_cairo_path_to_quartz_context (path, surface->cgContext);
>+
>+    stroke.cgContext = surface->cgContext;
>+    stroke.ctm_inverse = NULL;
>+    _cairo_quartz_cairo_path_to_quartz_context (path, &stroke);
> 
>     if (action == DO_SOLID || action == DO_PATTERN) {
> 	if (fill_rule == CAIRO_FILL_RULE_WINDING)
>@@ -1169,6 +1201,8 @@ _cairo_quartz_surface_stroke (void *abstract_surface,
>     cairo_quartz_surface_t *surface = (cairo_quartz_surface_t *) abstract_surface;
>     cairo_int_status_t rv = CAIRO_STATUS_SUCCESS;
>     cairo_quartz_action_t action;
>+    quartz_stroke_t stroke;
>+    CGAffineTransform strokeTransform;
> 
>     ND((stderr, "%p _cairo_quartz_surface_stroke op %d source->type %d\n", surface, op, source->type));
> 
>@@ -1184,6 +1218,8 @@ _cairo_quartz_surface_stroke (void *abstract_surface,
>     CGContextSetLineCap (surface->cgContext, _cairo_quartz_cairo_line_cap_to_quartz (style->line_cap));
>     CGContextSetLineJoin (surface->cgContext, _cairo_quartz_cairo_line_join_to_quartz (style->line_join));
>     CGContextSetMiterLimit (surface->cgContext, style->miter_limit);
>+    _cairo_quartz_cairo_matrix_to_quartz (ctm, &strokeTransform);
>+    CGContextConcatCTM (surface->cgContext, strokeTransform);
> 
>     if (style->dash && style->num_dashes) {
> #define STATIC_DASH 32
>@@ -1211,7 +1247,10 @@ _cairo_quartz_surface_stroke (void *abstract_surface,
>     }
> 
>     CGContextBeginPath (surface->cgContext);
>-    _cairo_quartz_cairo_path_to_quartz_context (path, surface->cgContext);
>+
>+    stroke.cgContext = surface->cgContext;
>+    stroke.ctm_inverse = ctm_inverse;
>+    _cairo_quartz_cairo_path_to_quartz_context (path, &stroke);
> 
>     if (action == DO_SOLID || action == DO_PATTERN) {
> 	CGContextStrokePath (surface->cgContext);
>@@ -1402,6 +1441,7 @@ _cairo_quartz_surface_intersect_clip_path (void *abstract_surface,
> 					    cairo_antialias_t antialias)
> {
>     cairo_quartz_surface_t *surface = (cairo_quartz_surface_t *) abstract_surface;
>+    quartz_stroke_t stroke;
> 
>     ND((stderr, "%p _cairo_quartz_surface_intersect_clip_path path: %p\n", surface, path));
> 
>@@ -1417,7 +1457,9 @@ _cairo_quartz_surface_intersect_clip_path (void *abstract_surface,
> 	CGContextSaveGState (surface->cgContext);
>     } else {
> 	CGContextBeginPath (surface->cgContext);
>-	_cairo_quartz_cairo_path_to_quartz_context (path, surface->cgContext);
>+	stroke.cgContext = surface->cgContext;
>+	stroke.ctm_inverse = NULL;
>+	_cairo_quartz_cairo_path_to_quartz_context (path, &stroke);
> 	if (fill_rule == CAIRO_FILL_RULE_WINDING)
> 	    CGContextClip (surface->cgContext);
> 	else

Actions: View | Splinter Review

Attachments on bug 10036: 8781 | 8795 | 9001 | 9282 | 9292

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.