[cairo-commit] Branch '1.8' - src/cairo.c

Chris Wilson ickle at kemper.freedesktop.org
Fri Feb 13 02:08:32 PST 2009


 src/cairo.c |    5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

New commits:
commit de7270cb1e9510cb230e80045c812fa68fcfa585
Author: Chris Wilson <chris at chris-wilson.co.uk>
Date:   Fri Feb 13 10:03:22 2009 +0000

    [cairo] Describe the restrictions upon cairo_set_tolerance()
    
    Truc Troung reported that the behaviour of
    cairo_set_tolerance()/cairo_get_tolerance() was inconsistent with the
    documentation, i.e. we failed to mention that the tolerance would be
    restricted to the smalled fixed-point value.
    
    Add a sentence to the documentation that describes the restriction without
    mentioning what that is... Hopefully that is sufficient detail to
    accommodate the reporter, without exposing internal implementation details.
    
    Fixes https://bugs.freedesktop.org/show_bug.cgi?id=20095
    Bug 20095 - The cairo_set_tolerance() function behavior is inconsistency
    with the spec

diff --git a/src/cairo.c b/src/cairo.c
index cefc925..5f8d266 100644
--- a/src/cairo.c
+++ b/src/cairo.c
@@ -816,7 +816,10 @@ cairo_get_source (cairo_t *cr)
  * is less than @tolerance. The default value is 0.1. A larger
  * value will give better performance, a smaller value, better
  * appearance. (Reducing the value from the default value of 0.1
- * is unlikely to improve appearance significantly.)
+ * is unlikely to improve appearance significantly.)  The accuracy of paths
+ * within Cairo is limited by the precision of its internal arithmetic, and
+ * the prescribed @tolerance is restricted to the smallest
+ * representable internal value.
  **/
 void
 cairo_set_tolerance (cairo_t *cr, double tolerance)


More information about the cairo-commit mailing list