Bug 50412

Summary:	[patch] add comment to explain _cairo_edge_compute_intersection_*, improve code readability
Product:	cairo	Reporter:	Homer Hsing <dongsheng.xing>
Component:	general	Assignee:	Bryce Harrington <bryce>
Status:	RESOLVED FIXED	QA Contact:	cairo-bugs mailing list <cairo-bugs>
Severity:	normal
Priority:	medium
Version:	1.12.2
Hardware:	All
OS:	All
Whiteboard:
i915 platform:		i915 features:

Description Homer Hsing 2012-05-27 23:43:00 UTC

diff --git a/src/cairo-fixed-private.h b/src/cairo-fixed-private.h
index b6cc6be..1956cf9 100644
--- a/src/cairo-fixed-private.h
+++ b/src/cairo-fixed-private.h
@@ -311,7 +311,7 @@ _cairo_fixed_mul_div_floor (cairo_fixed_t a, cairo_fixed_t b, cairo_fixed_t c)
     return _cairo_int64_32_div (_cairo_int32x32_64_mul (a, b), c);
 }
 
-
+/* compute y from x so that (x,y), p1, and p2 are collinear */
 static inline cairo_fixed_t
 _cairo_edge_compute_intersection_y_for_x (const cairo_point_t *p1,
                                          const cairo_point_t *p2,
@@ -332,6 +332,7 @@ _cairo_edge_compute_intersection_y_for_x (const cairo_point_t *p1,
     return y;
 }
 
+/* compute x from y so that (x,y), p1, and p2 are collinear */
 static inline cairo_fixed_t
 _cairo_edge_compute_intersection_x_for_y (const cairo_point_t *p1,
                                          const cairo_point_t *p2,

Comment 1 Bryce Harrington 2014-02-27 02:45:21 UTC

commit 273210683f182d72789f88df80fec8c4d08bb30f
Author: Bryce Harrington <b.harrington@samsung.com>
Date:   Wed Feb 26 18:44:46 2014 -0800

    Add comment to explain _cairo_edge_compute_intersection_*

    Patch by Homer Hsing

    Bugzilla:  https://bugs.freedesktop.org/show_bug.cgi?id=50412

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.