[cairo-bugs] [Bug 50416] New: [patch] add explaination to enum _cairo_int_status

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Mon May 28 01:25:27 PDT 2012


             Bug #: 50416
           Summary: [patch] add explaination to enum _cairo_int_status
    Classification: Unclassified
           Product: cairo
           Version: 1.12.2
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: medium
         Component: general
        AssignedTo: cworth at cworth.org
        ReportedBy: dongsheng.xing at intel.com
         QAContact: cairo-bugs at cairographics.org

In the source code, the "int" in "_cairo_int_status" means "internal". But the
code readers maybe do not interpret "int" as "internal". The patch clarifies
the meaning.

diff --git a/src/cairo-error-private.h b/src/cairo-error-private.h
index ea9c2ea..798b82b 100644
--- a/src/cairo-error-private.h
+++ b/src/cairo-error-private.h
@@ -46,7 +46,8 @@


-/* Sure wish C had a real enum type so that this would be distinct
+/* _cairo_int_status: internal status
+ *
+ * Sure wish C had a real enum type so that this would be distinct
  * from #cairo_status_t. Oh well, without that, I'll use this bogus 100
  * offset.  We want to keep it fit in int8_t as the compiler may choose
  * that for #cairo_status_t */

Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA Contact for the bug.

More information about the cairo-bugs mailing list