[cairo] [PATCH] minor document fixes

Behdad Esfahbod behdad at behdad.org
Mon May 28 14:08:08 PDT 2007


On Mon, 2007-05-28 at 08:00 -0400, Kouhei Sutou wrote:
> Hi,
> 
> I found some typos in documentation. I'll attach a patch to
> fix them and improve markup.

Thanks Kouhei.  Reviewed and committed most.  Read below for comments.

> Thanks,
> --
> kou
> plain text document attachment (cairo-doc-fix.diff)
> diff --git a/src/cairo-matrix.c b/src/cairo-matrix.c
> index 40828f0..872bfc1 100644
> --- a/src/cairo-matrix.c
> +++ b/src/cairo-matrix.c
> @@ -218,7 +218,7 @@ slim_hidden_def(cairo_matrix_scale);
>   * @radians: angle of rotation, in radians. The direction of rotation
>   * is defined such that positive angles rotate in the direction from
>   * the positive X axis toward the positive Y axis. With the default
> - * axis orientation of cairo, positive angles rotate in a clockwise
> + * axis orientation of cairo, positive angles rotate in a counterclockwise
>   * direction.
>   *
>   * Initialized @matrix to a transformation that rotates by @radians.
> @@ -246,7 +246,7 @@ slim_hidden_def(cairo_matrix_init_rotate);
>   * @radians: angle of rotation, in radians. The direction of rotation
>   * is defined such that positive angles rotate in the direction from
>   * the positive X axis toward the positive Y axis. With the default
> - * axis orientation of cairo, positive angles rotate in a clockwise
> + * axis orientation of cairo, positive angles rotate in a counterclockwise
>   * direction.
>   *
>   * Applies rotation by @radians to the transformation in

No, the current docs in these cases is correct.  Cairo uses a
y-grows-down coordinate system by default, which means positive angles
rotate clockwise.


> diff --git a/src/cairo.c b/src/cairo.c
> index 2abc073..abebfd3 100644
> --- a/src/cairo.c
> +++ b/src/cairo.c
> @@ -3117,7 +3117,7 @@ cairo_get_antialias (cairo_t *cr)
>   *
>   * cairo_new_path(), cairo_move_to(), cairo_line_to(),
>   * cairo_curve_to(), cairo_arc(), cairo_rel_move_to(),
> - * cairo_rel_line_to(), cairo_rel_curve_to(), cairo_arc(),
> + * cairo_rel_line_to(), cairo_rel_curve_to(), cairo_rel_arc(),
>   * cairo_text_path(), cairo_stroke_to_path()
>   **/
>  void

There is no such function as cairo_rel_arc(), but I updated that list to
include all path-modifying functions as well as some others..


Thanks,
-- 
behdad
http://behdad.org/

"Those who would give up Essential Liberty to purchase a little
 Temporary Safety, deserve neither Liberty nor Safety."
        -- Benjamin Franklin, 1759





More information about the cairo mailing list