[cairo] [PATCH 0/7] Bunch of distcheck fixes

Bryce W. Harrington b.harrington at samsung.com
Wed Feb 26 14:29:41 PST 2014


With most of the build warnings squared away, here's a set of fixes for
various problems encountered when running make distcheck.

Largely these are documentation issues.  I've filled in bits as best I
could figure out, and moved things around a bit to make the doc checker
happy, but I would certainly appreciate a close review.  Some of the
docs could benefit from a bit more elaboration by someone more familiar
with the given code.

I also had to make some judgment calls in adding Since fields to various
API bits.  I tried to find when the interface was first mentioned in
NEWS, or lacking that tried to identify the date of introduction into
git, and then assume it was part of the next major cairo release and
used that number.  Those numbers may be off; I don't know how critical
these numbers are though.  There may be cases where I've added versions
but it really should be TBD; if you spot these let me know.

This patch set cleans up nearly all the distcheck issues, but there's a
few things remaining:

  * warning about uninitialized variables

  * warnings about inlining failures

  * warnings about ignored return values (Bugzilla #58727)

  * Lots and lots of test failures

  * rsvg_pixbuf_from_file is deprecated, so svg2png needs recoded a bit.

  * four unused CAIRO_HAS_*_FUNCTIONS declarations aren't included in
    cairo-sections.txt.

  * gtk-doc can't parse "SkBitmap::Config&" args so some skia functions
    aren't getting processed

The first three we've discussed on the list and felt were pretty minor.
The test failures need further analysis, and likely a lot more work.
The gtk-doc issue I suspect is gtk-doc being confused by C++ code, and
maybe not something to worry about; maybe a workaround would be enough.
The other two issues are minor and probably more work to fix than worth.



Bryce Harrington (7):
  doc:  Add missing sections and symbols for public docs
  doc: Drop a couple quartz routines which distcheck claims don't exist
  Mark recently added _cairo_output_stream_print_matrix private symbol
  skia: Add section definitions and code docs for skia backend
  check-def:  Fix code document errors
  surface: Make parameter naming consistent between header and impl
  Document that libglib2.0-doc is needed to avoid some xref warnings

 RELEASING                         |    4 +-
 doc/public/cairo-sections.txt     |   42 +++++++++++++++++++--
 src/cairo-cache-private.h         |    2 +
 src/cairo-output-stream-private.h |    2 +-
 src/cairo-raster-source-pattern.c |    2 -
 src/cairo-region.c                |   16 +-------
 src/cairo-skia-surface.cpp        |   21 +++++++++++
 src/cairo-surface-observer.c      |    5 +++
 src/cairo-surface.c               |   12 +++---
 src/cairo-types-private.h         |   74 ++++++++++++++++++++-----------------
 src/cairo.h                       |   20 ++++++++++
 src/skia/cairo-skia-private.h     |   19 ++++++++++
 12 files changed, 158 insertions(+), 61 deletions(-)

-- 
1.7.9.5


More information about the cairo mailing list