Code

fix bbox calculation for groups that contain groups with nothing in them (zero bbox...
[inkscape.git] / src / display / inkscape-cairo.cpp
index 027a89e828e0c326eade6a4aa177a9690a09bd79..49d47ec77bda5369b3c694749b93d74b954fa7dd 100644 (file)
@@ -20,6 +20,7 @@
 #include <libnr/nr-convert2geom.h>
 #include "../style.h"
 #include "nr-arena.h"
+#include "sp-canvas.h"
 #include <2geom/pathvector.h>
 #include <2geom/matrix.h>
 #include <2geom/point.h>
 
 /** Creates a cairo context to render to the given pixblock on the given area */
 cairo_t *
-nr_create_cairo_context (NRRectL *area, NRPixBlock *pb)
+nr_create_cairo_context_for_data (NRRectL *area, NRRectL *buf_area, unsigned char *px, unsigned int rowstride)
 {
-    if (!nr_rect_l_test_intersect (&pb->area, area))
+    if (!nr_rect_l_test_intersect (buf_area, area))
         return NULL;
 
     NRRectL clip;
-    nr_rect_l_intersect (&clip, &pb->area, area);
-    unsigned char *dpx = NR_PIXBLOCK_PX (pb) + (clip.y0 - pb->area.y0) * pb->rs + NR_PIXBLOCK_BPP (pb) * (clip.x0 - pb->area.x0);
+    nr_rect_l_intersect (&clip, buf_area, area);
+    unsigned char *dpx = px + (clip.y0 - buf_area->y0) * rowstride + 4 * (clip.x0 - buf_area->x0);
     int width = area->x1 - area->x0;
     int height = area->y1 - area->y0;
     // even though cairo cannot draw in nonpremul mode, select ARGB32 for R8G8B8A8N as the closest; later eliminate R8G8B8A8N everywhere
     cairo_surface_t* cst = cairo_image_surface_create_for_data
         (dpx,
-         ((pb->mode == NR_PIXBLOCK_MODE_R8G8B8A8P || pb->mode == NR_PIXBLOCK_MODE_R8G8B8A8N) ? CAIRO_FORMAT_ARGB32 : (pb->mode == NR_PIXBLOCK_MODE_R8G8B8? CAIRO_FORMAT_RGB24 : CAIRO_FORMAT_A8)),
+         CAIRO_FORMAT_ARGB32,
          width,
          height,
-         pb->rs);
+         rowstride);
     cairo_t *ct = cairo_create (cst);
 
     return ct;
 }
 
+/** Creates a cairo context to render to the given SPCanvasBuf on the given area */
+cairo_t *
+nr_create_cairo_context_canvasbuf (NRRectL *area, SPCanvasBuf *b)
+{
+    return nr_create_cairo_context_for_data (&(b->rect), &(b->rect), b->buf, b->buf_rowstride);
+}
+
+
+/** Creates a cairo context to render to the given NRPixBlock on the given area */
+cairo_t *
+nr_create_cairo_context (NRRectL *area, NRPixBlock *pb)
+{
+    return nr_create_cairo_context_for_data (area, &(pb->area), NR_PIXBLOCK_PX (pb), pb->rs);
+}
+
 /** Feeds path-creating calls to the cairo context translating them from the SPCurve, with the given transform and shift */
 void
 feed_curve_to_cairo (cairo_t *ct, NArtBpath const *bpath, NR::Matrix trans, NR::Maybe<NR::Rect> area, bool optimize_stroke, double stroke_width)
@@ -141,22 +157,27 @@ feed_curve_to_cairo (cairo_t *ct, NArtBpath const *bpath, NR::Matrix trans, NR::
 
 
 static void
-feed_curve_to_cairo(cairo_t *cr, Geom::Curve const &c, Geom::Rect view, bool optimize_stroke)
+feed_curve_to_cairo(cairo_t *cr, Geom::Curve const &c, Geom::Matrix & trans, Geom::Rect view, bool optimize_stroke)
 {
     if(Geom::LineSegment const* line_segment = dynamic_cast<Geom::LineSegment const*>(&c)) {
+    // same logic can be used for hlinesegment and vlinesegment: just use finalpoint
+        Geom::Point end_tr = (*line_segment)[1] * trans;
         if (!optimize_stroke) {
-            cairo_line_to(cr, (*line_segment)[1][0], (*line_segment)[1][1]);
+            cairo_line_to(cr, end_tr[0], end_tr[1]);
         } else {
-            Geom::Rect swept((*line_segment)[0], (*line_segment)[1]);
+            Geom::Rect swept((*line_segment)[0]*trans, end_tr);
             if (swept.intersects(view)) {
-                cairo_line_to(cr, (*line_segment)[1][0], (*line_segment)[1][1]);
+                cairo_line_to(cr, end_tr[0], end_tr[1]);
             } else {
-                cairo_move_to(cr, (*line_segment)[1][0], (*line_segment)[1][1]);
+                cairo_move_to(cr, end_tr[0], end_tr[1]);
             }
         }
     }
     else if(Geom::QuadraticBezier const *quadratic_bezier = dynamic_cast<Geom::QuadraticBezier const*>(&c)) {
         std::vector<Geom::Point> points = quadratic_bezier->points();
+        points[0] *= trans;
+        points[1] *= trans;
+        points[2] *= trans;
         Geom::Point b1 = points[0] + (2./3) * (points[1] - points[0]);
         Geom::Point b2 = b1 + (1./3) * (points[2] - points[0]);
         if (!optimize_stroke) {
@@ -173,9 +194,14 @@ feed_curve_to_cairo(cairo_t *cr, Geom::Curve const &c, Geom::Rect view, bool opt
     }
     else if(Geom::CubicBezier const *cubic_bezier = dynamic_cast<Geom::CubicBezier const*>(&c)) {
         std::vector<Geom::Point> points = cubic_bezier->points();
+        //points[0] *= trans; // don't do this one here for fun: it is only needed for optimized strokes
+        points[1] *= trans;
+        points[2] *= trans;
+        points[3] *= trans;
         if (!optimize_stroke) {
             cairo_curve_to(cr, points[1][0], points[1][1], points[2][0], points[2][1], points[3][0], points[3][1]);
         } else {
+            points[0] *= trans;  // didn't transform this point yet
             Geom::Rect swept(points[0], points[3]);
             swept.expandTo(points[1]);
             swept.expandTo(points[2]);
@@ -195,7 +221,7 @@ feed_curve_to_cairo(cairo_t *cr, Geom::Curve const &c, Geom::Rect view, bool opt
 
         //recurse to convert the new path resulting from the sbasis to svgd
         for(Geom::Path::iterator iter = sbasis_path.begin(); iter != sbasis_path.end(); ++iter) {
-            feed_curve_to_cairo(cr, *iter, view, optimize_stroke);
+            feed_curve_to_cairo(cr, *iter, trans, view, optimize_stroke);
         }
     }
 }
@@ -215,16 +241,24 @@ feed_path_to_cairo (cairo_t *ct, Geom::Path const &path, Geom::Matrix trans, NR:
     NR::Rect view = *area;
     view.growBy (stroke_width);
     view = view * from_2geom(Geom::Translate(-shift));
-    Geom::Path const path_trans = path * (trans * Geom::Translate(-shift));
+    //  Pass transformation to feed_curve, so that we don't need to create a whole new path.
+    Geom::Matrix transshift(trans * Geom::Translate(-shift));
 
-    cairo_move_to(ct, path_trans.initialPoint()[0], path_trans.initialPoint()[1] );
+    Geom::Point initial = path.initialPoint() * transshift;
+    cairo_move_to(ct, initial[0], initial[1] );
 
-    for(Geom::Path::const_iterator cit = path_trans.begin(); cit != path_trans.end_open(); ++cit) {
-        feed_curve_to_cairo(ct, *cit, to_2geom(view), optimize_stroke);
+    for(Geom::Path::const_iterator cit = path.begin(); cit != path.end_open(); ++cit) {
+        feed_curve_to_cairo(ct, *cit, transshift, to_2geom(view), optimize_stroke);
     }
 
     if (path.closed()) {
-        cairo_line_to(ct, path_trans.initialPoint()[0], path_trans.initialPoint()[1] );
+        cairo_line_to(ct, initial[0], initial[1]);
+        // I think we should use cairo_close_path(ct) here but it doesn't work. (the closing line is not rendered completely)
+        /* according to cairo documentation:
+           The behavior of cairo_close_path() is distinct from simply calling cairo_line_to() with the equivalent coordinate
+           in the case of stroking. When a closed sub-path is stroked, there are no caps on the ends of the sub-path. Instead,
+           there is a line join connecting the final and initial segments of the sub-path. 
+        */
     }
 }