[cairo] [PATCH 2/4] cairo-xcb: gtk-doc doesn't like _ in parameter names
Andrea Canciani
ranma42 at gmail.com
Fri Dec 2 07:20:54 PST 2011
On Fri, Dec 2, 2011 at 3:57 PM, Keith Packard <keithp at keithp.com> wrote:
> Any function documented with gtk-doc must not have _ in any parameter
> names, or at least that's what I've found. This patch simply renames
> parameters as needed to make things work.
The '_' is not the real issue. The problem is that there is a mismatch
between the names used in the header and in the documentation strings.
For a previous (less complete and untested) attempt at fixing the same
problem, see:
http://cgit.freedesktop.org/~ranma42/cairo/commit/?h=wip/xcb-doc&id=2fab3e2ea852bda56998bec3be3bf29b729b97ba
Maybe we can just reuse the commit message from 2fab3e2 and apply your
patches (they're a superset of what I had in wip/xcb-doc).
Andrea
>
> Signed-off-by: Keith Packard <keithp at keithp.com>
> ---
> src/cairo-xcb-surface.c | 48 +++++++++++++++++++++++-----------------------
> 1 files changed, 24 insertions(+), 24 deletions(-)
>
> diff --git a/src/cairo-xcb-surface.c b/src/cairo-xcb-surface.c
> index 77d4e79..6ddc815 100644
> --- a/src/cairo-xcb-surface.c
> +++ b/src/cairo-xcb-surface.c
> @@ -1149,7 +1149,7 @@ _cairo_xcb_screen_from_visual (xcb_connection_t *connection,
>
> /**
> * cairo_xcb_surface_create:
> - * @xcb_connection: an XCB connection
> + * @connection: an XCB connection
> * @drawable: an XCB drawable
> * @visual: the visual to use for drawing to @drawable. The depth
> * of the visual must match the depth of the drawable.
> @@ -1179,7 +1179,7 @@ _cairo_xcb_screen_from_visual (xcb_connection_t *connection,
> * occurs. You can use cairo_surface_status() to check for this.
> **/
> cairo_surface_t *
> -cairo_xcb_surface_create (xcb_connection_t *xcb_connection,
> +cairo_xcb_surface_create (xcb_connection_t *connection,
> xcb_drawable_t drawable,
> xcb_visualtype_t *visual,
> int width,
> @@ -1192,7 +1192,7 @@ cairo_xcb_surface_create (xcb_connection_t *xcb_connection,
> xcb_render_pictformat_t xrender_format;
> int depth;
>
> - if (xcb_connection_has_error (xcb_connection))
> + if (xcb_connection_has_error (connection))
> return _cairo_surface_create_in_error (_cairo_error (CAIRO_STATUS_WRITE_ERROR));
>
> if (unlikely (width > XLIB_COORD_MAX || height > XLIB_COORD_MAX))
> @@ -1200,7 +1200,7 @@ cairo_xcb_surface_create (xcb_connection_t *xcb_connection,
> if (unlikely (width <= 0 || height <= 0))
> return _cairo_surface_create_in_error (_cairo_error (CAIRO_STATUS_INVALID_SIZE));
>
> - xcb_screen = _cairo_xcb_screen_from_visual (xcb_connection, visual, &depth);
> + xcb_screen = _cairo_xcb_screen_from_visual (connection, visual, &depth);
> if (unlikely (xcb_screen == NULL))
> return _cairo_surface_create_in_error (_cairo_error (CAIRO_STATUS_INVALID_VISUAL));
>
> @@ -1223,7 +1223,7 @@ cairo_xcb_surface_create (xcb_connection_t *xcb_connection,
> if (! _pixman_format_from_masks (&image_masks, &pixman_format))
> return _cairo_surface_create_in_error (_cairo_error (CAIRO_STATUS_INVALID_FORMAT));
>
> - screen = _cairo_xcb_screen_get (xcb_connection, xcb_screen);
> + screen = _cairo_xcb_screen_get (connection, xcb_screen);
> if (unlikely (screen == NULL))
> return _cairo_surface_create_in_error (_cairo_error (CAIRO_STATUS_NO_MEMORY));
>
> @@ -1242,11 +1242,11 @@ slim_hidden_def (cairo_xcb_surface_create);
>
> /**
> * cairo_xcb_surface_create_for_bitmap:
> - * @xcb_connection: an XCB connection
> - * @xcb_screen: the XCB screen associated with @bitmap
> + * @connection: an XCB connection
> + * @screen: the XCB screen associated with @bitmap
> * @bitmap: an XCB drawable (a Pixmap with depth 1)
> - * @width: the current width of @drawable
> - * @height: the current height of @drawable
> + * @width: the current width of @bitmap
> + * @height: the current height of @bitmap
> *
> * Creates an XCB surface that draws to the given bitmap.
> * This will be drawn to as a %CAIRO_FORMAT_A1 object.
> @@ -1260,15 +1260,15 @@ slim_hidden_def (cairo_xcb_surface_create);
> * occurs. You can use cairo_surface_status() to check for this.
> **/
> cairo_surface_t *
> -cairo_xcb_surface_create_for_bitmap (xcb_connection_t *xcb_connection,
> - xcb_screen_t *xcb_screen,
> +cairo_xcb_surface_create_for_bitmap (xcb_connection_t *connection,
> + xcb_screen_t *screen,
> xcb_pixmap_t bitmap,
> int width,
> int height)
> {
> - cairo_xcb_screen_t *screen;
> + cairo_xcb_screen_t *cairo_xcb_screen;
>
> - if (xcb_connection_has_error (xcb_connection))
> + if (xcb_connection_has_error (connection))
> return _cairo_surface_create_in_error (_cairo_error (CAIRO_STATUS_WRITE_ERROR));
>
> if (width > XLIB_COORD_MAX || height > XLIB_COORD_MAX)
> @@ -1276,13 +1276,13 @@ cairo_xcb_surface_create_for_bitmap (xcb_connection_t *xcb_connection,
> if (unlikely (width <= 0 || height <= 0))
> return _cairo_surface_create_in_error (_cairo_error (CAIRO_STATUS_INVALID_SIZE));
>
> - screen = _cairo_xcb_screen_get (xcb_connection, xcb_screen);
> - if (unlikely (screen == NULL))
> + cairo_xcb_screen = _cairo_xcb_screen_get (connection, screen);
> + if (unlikely (cairo_xcb_screen == NULL))
> return _cairo_surface_create_in_error (_cairo_error (CAIRO_STATUS_NO_MEMORY));
>
> - return _cairo_xcb_surface_create_internal (screen, bitmap, FALSE,
> + return _cairo_xcb_surface_create_internal (cairo_xcb_screen, bitmap, FALSE,
> PIXMAN_a1,
> - screen->connection->standard_formats[CAIRO_FORMAT_A1],
> + cairo_xcb_screen->connection->standard_formats[CAIRO_FORMAT_A1],
> width, height);
> }
> #if CAIRO_HAS_XLIB_XCB_FUNCTIONS
> @@ -1321,18 +1321,18 @@ slim_hidden_def (cairo_xcb_surface_create_for_bitmap);
> * occurs. You can use cairo_surface_status() to check for this.
> **/
> cairo_surface_t *
> -cairo_xcb_surface_create_with_xrender_format (xcb_connection_t *xcb_connection,
> - xcb_screen_t *xcb_screen,
> +cairo_xcb_surface_create_with_xrender_format (xcb_connection_t *connection,
> + xcb_screen_t *screen,
> xcb_drawable_t drawable,
> xcb_render_pictforminfo_t *format,
> int width,
> int height)
> {
> - cairo_xcb_screen_t *screen;
> + cairo_xcb_screen_t *cairo_xcb_screen;
> cairo_format_masks_t image_masks;
> pixman_format_code_t pixman_format;
>
> - if (xcb_connection_has_error (xcb_connection))
> + if (xcb_connection_has_error (connection))
> return _cairo_surface_create_in_error (_cairo_error (CAIRO_STATUS_WRITE_ERROR));
>
> if (width > XLIB_COORD_MAX || height > XLIB_COORD_MAX)
> @@ -1364,11 +1364,11 @@ cairo_xcb_surface_create_with_xrender_format (xcb_connection_t *xcb_connecti
> if (! _pixman_format_from_masks (&image_masks, &pixman_format))
> return _cairo_surface_create_in_error (_cairo_error (CAIRO_STATUS_INVALID_FORMAT));
>
> - screen = _cairo_xcb_screen_get (xcb_connection, xcb_screen);
> - if (unlikely (screen == NULL))
> + cairo_xcb_screen = _cairo_xcb_screen_get (connection, screen);
> + if (unlikely (cairo_xcb_screen == NULL))
> return _cairo_surface_create_in_error (_cairo_error (CAIRO_STATUS_NO_MEMORY));
>
> - return _cairo_xcb_surface_create_internal (screen,
> + return _cairo_xcb_surface_create_internal (cairo_xcb_screen,
> drawable,
> FALSE,
> pixman_format,
> --
> 1.7.7.3
>
> --
> cairo mailing list
> cairo at cairographics.org
> http://lists.cairographics.org/mailman/listinfo/cairo
More information about the cairo
mailing list