Update to trunk

(bzr r11950.1.141)

1 files changed, 148 insertions, 160 deletions

diff --git a/src/sp-symbol.cpp b/src/sp-symbol.cpp
index a56de2e5a..b2346ac91 100644
--- a/src/sp-symbol.cpp
+++ b/src/sp-symbol.cpp
@@ -26,116 +26,125 @@
 #include "sp-symbol.h"
 #include "document.h"
 
-static void sp_symbol_build (SPObject *object, SPDocument *document, Inkscape::XML::Node *repr);
-static void sp_symbol_release (SPObject *object);
-static void sp_symbol_set (SPObject *object, unsigned int key, const gchar *value);
-static void sp_symbol_child_added (SPObject *object, Inkscape::XML::Node *child, Inkscape::XML::Node *ref);
-static void sp_symbol_update (SPObject *object, SPCtx *ctx, guint flags);
-static void sp_symbol_modified (SPObject *object, guint flags);
-static Inkscape::XML::Node *sp_symbol_write (SPObject *object, Inkscape::XML::Document *doc, Inkscape::XML::Node *repr, guint flags);
-
-static Inkscape::DrawingItem *sp_symbol_show (SPItem *item, Inkscape::Drawing &drawing, unsigned int key, unsigned int flags);
-static void sp_symbol_hide (SPItem *item, unsigned int key);
-static Geom::OptRect sp_symbol_bbox(SPItem const *item, Geom::Affine const &transform, SPItem::BBoxType type);
-static void sp_symbol_print (SPItem *item, SPPrintContext *ctx);
-
-G_DEFINE_TYPE(SPSymbol, sp_symbol, SP_TYPE_GROUP);
-
-static void sp_symbol_class_init(SPSymbolClass *klass)
-{
-    SPObjectClass *sp_object_class = (SPObjectClass *) klass;
-    SPItemClass *sp_item_class = (SPItemClass *) klass;
-
-    sp_object_class->build = sp_symbol_build;
-    sp_object_class->release = sp_symbol_release;
-    sp_object_class->set = sp_symbol_set;
-    sp_object_class->child_added = sp_symbol_child_added;
-    sp_object_class->update = sp_symbol_update;
-    sp_object_class->modified = sp_symbol_modified;
-    sp_object_class->write = sp_symbol_write;
-
-    sp_item_class->show = sp_symbol_show;
-    sp_item_class->hide = sp_symbol_hide;
-    sp_item_class->bbox = sp_symbol_bbox;
-    sp_item_class->print = sp_symbol_print;
-}
+#include "sp-factory.h"
 
-static void sp_symbol_init(SPSymbol *symbol)
-{
-    symbol->viewBox_set = FALSE;
+namespace {
+	SPObject* createSymbol() {
+		return new SPSymbol();
+	}
 
-    symbol->c2p = Geom::identity();
+	bool symbolRegistered = SPFactory::instance().registerObject("svg:symbol", createSymbol);
 }
 
-static void sp_symbol_build(SPObject *object, SPDocument *document, Inkscape::XML::Node *repr)
-{
-    object->readAttr( "viewBox" );
-    object->readAttr( "preserveAspectRatio" );
+SPSymbol::SPSymbol() : SPGroup() {
+	this->aspect_align = 0;
+	this->aspect_clip = 0;
+	this->aspect_set = 0;
 
-    if (((SPObjectClass *) sp_symbol_parent_class)->build) {
-        ((SPObjectClass *) sp_symbol_parent_class)->build (object, document, repr);
-    }
+    this->viewBox_set = FALSE;
+    this->c2p = Geom::identity();
 }
 
-static void sp_symbol_release(SPObject *object)
-{
-    if (((SPObjectClass *) sp_symbol_parent_class)->release) {
-        ((SPObjectClass *) sp_symbol_parent_class)->release (object);
-    }
+SPSymbol::~SPSymbol() {
 }
 
-static void sp_symbol_set(SPObject *object, unsigned int key, const gchar *value)
-{
-    SPSymbol *symbol = SP_SYMBOL(object);
+void SPSymbol::build(SPDocument *document, Inkscape::XML::Node *repr) {
+    this->readAttr( "viewBox" );
+    this->readAttr( "preserveAspectRatio" );
+
+    SPGroup::build(document, repr);
+}
+
+void SPSymbol::release() {
+	SPGroup::release();
+}
 
+void SPSymbol::set(unsigned int key, const gchar* value) {
     switch (key) {
     case SP_ATTR_VIEWBOX:
         if (value) {
             double x, y, width, height;
             char *eptr;
+
             /* fixme: We have to take original item affine into account */
             /* fixme: Think (Lauris) */
             eptr = (gchar *) value;
             x = g_ascii_strtod (eptr, &eptr);
-            while (*eptr && ((*eptr == ',') || (*eptr == ' '))) eptr++;
+
+            while (*eptr && ((*eptr == ',') || (*eptr == ' '))) {
+            	eptr++;
+            }
+
             y = g_ascii_strtod (eptr, &eptr);
-            while (*eptr && ((*eptr == ',') || (*eptr == ' '))) eptr++;
+
+            while (*eptr && ((*eptr == ',') || (*eptr == ' '))) {
+            	eptr++;
+            }
+
             width = g_ascii_strtod (eptr, &eptr);
-            while (*eptr && ((*eptr == ',') || (*eptr == ' '))) eptr++;
+
+            while (*eptr && ((*eptr == ',') || (*eptr == ' '))) {
+            	eptr++;
+            }
+
             height = g_ascii_strtod (eptr, &eptr);
-            while (*eptr && ((*eptr == ',') || (*eptr == ' '))) eptr++;
+
+            while (*eptr && ((*eptr == ',') || (*eptr == ' '))) {
+            	eptr++;
+            }
+
             if ((width > 0) && (height > 0)) {
                 /* Set viewbox */
-                symbol->viewBox = Geom::Rect::from_xywh(x, y, width, height);
-                symbol->viewBox_set = TRUE;
+                this->viewBox = Geom::Rect::from_xywh(x, y, width, height);
+                this->viewBox_set = TRUE;
             } else {
-                symbol->viewBox_set = FALSE;
+                this->viewBox_set = FALSE;
             }
         } else {
-            symbol->viewBox_set = FALSE;
+            this->viewBox_set = FALSE;
         }
-        object->requestDisplayUpdate(SP_OBJECT_MODIFIED_FLAG | SP_OBJECT_VIEWPORT_MODIFIED_FLAG);
+
+        this->requestDisplayUpdate(SP_OBJECT_MODIFIED_FLAG | SP_OBJECT_VIEWPORT_MODIFIED_FLAG);
         break;
+
     case SP_ATTR_PRESERVEASPECTRATIO:
         /* Do setup before, so we can use break to escape */
-        symbol->aspect_set = FALSE;
-        symbol->aspect_align = SP_ASPECT_NONE;
-        symbol->aspect_clip = SP_ASPECT_MEET;
-        object->requestDisplayUpdate(SP_OBJECT_MODIFIED_FLAG | SP_OBJECT_VIEWPORT_MODIFIED_FLAG);
+        this->aspect_set = FALSE;
+        this->aspect_align = SP_ASPECT_NONE;
+        this->aspect_clip = SP_ASPECT_MEET;
+        this->requestDisplayUpdate(SP_OBJECT_MODIFIED_FLAG | SP_OBJECT_VIEWPORT_MODIFIED_FLAG);
+
         if (value) {
             int len;
             gchar c[256];
             const gchar *p, *e;
             unsigned int align, clip;
             p = value;
-            while (*p && *p == 32) p += 1;
-            if (!*p) break;
+
+            while (*p && *p == 32) {
+            	p += 1;
+            }
+
+            if (!*p) {
+            	break;
+            }
+
             e = p;
-            while (*e && *e != 32) e += 1;
+
+            while (*e && *e != 32) {
+            	e += 1;
+            }
+
             len = e - p;
-            if (len > 8) break;
+
+            if (len > 8) {
+            	break;
+            }
+
             memcpy (c, value, len);
+
             c[len] = 0;
+
             /* Now the actual part */
             if (!strcmp (c, "none")) {
                 align = SP_ASPECT_NONE;
@@ -160,8 +169,13 @@ static void sp_symbol_set(SPObject *object, unsigned int key, const gchar *value
             } else {
                 break;
             }
+
             clip = SP_ASPECT_MEET;
-            while (*e && *e == 32) e += 1;
+
+            while (*e && *e == 32) {
+            	e += 1;
+            }
+
             if (*e) {
                 if (!strcmp (e, "meet")) {
                     clip = SP_ASPECT_MEET;
@@ -171,32 +185,29 @@ static void sp_symbol_set(SPObject *object, unsigned int key, const gchar *value
                     break;
                 }
             }
-            symbol->aspect_set = TRUE;
-            symbol->aspect_align = align;
-            symbol->aspect_clip = clip;
+
+            this->aspect_set = TRUE;
+            this->aspect_align = align;
+            this->aspect_clip = clip;
         }
         break;
+
     default:
-        if (((SPObjectClass *) sp_symbol_parent_class)->set)
-            ((SPObjectClass *) sp_symbol_parent_class)->set (object, key, value);
+        SPGroup::set(key, value);
         break;
     }
 }
 
-static void sp_symbol_child_added(SPObject *object, Inkscape::XML::Node *child, Inkscape::XML::Node *ref)
-{
-    if (((SPObjectClass *) (sp_symbol_parent_class))->child_added) {
-        ((SPObjectClass *) (sp_symbol_parent_class))->child_added (object, child, ref);
-    }
+void SPSymbol::child_added(Inkscape::XML::Node *child, Inkscape::XML::Node *ref) {
+	SPGroup::child_added(child, ref);
 }
 
-static void sp_symbol_update(SPObject *object, SPCtx *ctx, guint flags)
-{
-    SPSymbol *symbol = SP_SYMBOL(object);
+
+void SPSymbol::update(SPCtx *ctx, guint flags) {
     SPItemCtx *ictx = (SPItemCtx *) ctx;
     SPItemCtx rctx;
 
-    if (object->cloned) {
+    if (this->cloned) {
         /* Cloned <symbol> is actually renderable */
 
         /* fixme: We have to set up clip here too */
@@ -206,12 +217,13 @@ static void sp_symbol_update(SPObject *object, SPCtx *ctx, guint flags)
 
         /* Calculate child to parent transformation */
         /* Apply parent <use> translation (set up as vewport) */
-        symbol->c2p = Geom::Translate(rctx.viewport.min());
+        this->c2p = Geom::Translate(rctx.viewport.min());
 
-        if (symbol->viewBox_set) {
+        if (this->viewBox_set) {
             double x, y, width, height;
+
             /* Determine actual viewbox in viewport coordinates */
-            if (symbol->aspect_align == SP_ASPECT_NONE) {
+            if (this->aspect_align == SP_ASPECT_NONE) {
                 x = 0.0;
                 y = 0.0;
                 width = rctx.viewport.width();
@@ -219,13 +231,14 @@ static void sp_symbol_update(SPObject *object, SPCtx *ctx, guint flags)
             } else {
                 double scalex, scaley, scale;
                 /* Things are getting interesting */
-                scalex = rctx.viewport.width() / symbol->viewBox.width();
-                scaley = rctx.viewport.height() / symbol->viewBox.height();
-                scale = (symbol->aspect_clip == SP_ASPECT_MEET) ? MIN (scalex, scaley) : MAX (scalex, scaley);
-                width = symbol->viewBox.width() * scale;
-                height = symbol->viewBox.height() * scale;
+                scalex = rctx.viewport.width() / this->viewBox.width();
+                scaley = rctx.viewport.height() / this->viewBox.height();
+                scale = (this->aspect_clip == SP_ASPECT_MEET) ? MIN (scalex, scaley) : MAX (scalex, scaley);
+                width = this->viewBox.width() * scale;
+                height = this->viewBox.height() * scale;
+
                 /* Now place viewbox to requested position */
-                switch (symbol->aspect_align) {
+                switch (this->aspect_align) {
                 case SP_ASPECT_XMIN_YMIN:
                     x = 0.0;
                     y = 0.0;
@@ -268,133 +281,108 @@ static void sp_symbol_update(SPObject *object, SPCtx *ctx, guint flags)
                     break;
                 }
             }
+
             /* Compose additional transformation from scale and position */
             Geom::Affine q;
-            q[0] = width / symbol->viewBox.width();
+            q[0] = width / this->viewBox.width();
             q[1] = 0.0;
             q[2] = 0.0;
-            q[3] = height / symbol->viewBox.height();
-            q[4] = -symbol->viewBox.left() * q[0] + x;
-            q[5] = -symbol->viewBox.top() * q[3] + y;
+            q[3] = height / this->viewBox.height();
+            q[4] = -this->viewBox.left() * q[0] + x;
+            q[5] = -this->viewBox.top() * q[3] + y;
+
             /* Append viewbox transformation */
-            symbol->c2p = q * symbol->c2p;
+            this->c2p = q * this->c2p;
         }
 
-        rctx.i2doc = symbol->c2p * (Geom::Affine)rctx.i2doc;
+        rctx.i2doc = this->c2p * (Geom::Affine)rctx.i2doc;
 
         /* If viewBox is set initialize child viewport */
         /* Otherwise <use> has set it up already */
-        if (symbol->viewBox_set) {
-            rctx.viewport = symbol->viewBox;
+        if (this->viewBox_set) {
+            rctx.viewport = this->viewBox;
             rctx.i2vp = Geom::identity();
         }
 
         // And invoke parent method
-        if (((SPObjectClass *) (sp_symbol_parent_class))->update) {
-            ((SPObjectClass *) (sp_symbol_parent_class))->update (object, (SPCtx *) &rctx, flags);
-	}
+        SPGroup::update((SPCtx *) &rctx, flags);
 
         // As last step set additional transform of drawing group
-        for (SPItemView *v = symbol->display; v != NULL; v = v->next) {
+        for (SPItemView *v = this->display; v != NULL; v = v->next) {
         	Inkscape::DrawingGroup *g = dynamic_cast<Inkscape::DrawingGroup *>(v->arenaitem);
-        	g->setChildTransform(symbol->c2p);
+        	g->setChildTransform(this->c2p);
         }
     } else {
         // No-op
-        if (((SPObjectClass *) (sp_symbol_parent_class))->update) {
-            ((SPObjectClass *) (sp_symbol_parent_class))->update (object, ctx, flags);
-	}
+        SPGroup::update(ctx, flags);
     }
 }
 
-static void
-sp_symbol_modified(SPObject *object,
-		   guint     flags)
-{
-    SP_OBJECT_CLASS(sp_symbol_parent_class)->modified (object, flags);
+void SPSymbol::modified(unsigned int flags) {
+	SPGroup::modified(flags);
 }
 
-static Inkscape::XML::Node *
-sp_symbol_write(SPObject                *object,
-		Inkscape::XML::Document *xml_doc,
-		Inkscape::XML::Node     *repr,
-		guint                    flags)
-{
+
+Inkscape::XML::Node* SPSymbol::write(Inkscape::XML::Document *xml_doc, Inkscape::XML::Node *repr, guint flags) {
     if ((flags & SP_OBJECT_WRITE_BUILD) && !repr) {
         repr = xml_doc->createElement("svg:symbol");
     }
 
     //XML Tree being used directly here while it shouldn't be.
-    repr->setAttribute("viewBox", object->getRepr()->attribute("viewBox"));
+    repr->setAttribute("viewBox", this->getRepr()->attribute("viewBox"));
 	
     //XML Tree being used directly here while it shouldn't be.
-    repr->setAttribute("preserveAspectRatio", object->getRepr()->attribute("preserveAspectRatio"));
+    repr->setAttribute("preserveAspectRatio", this->getRepr()->attribute("preserveAspectRatio"));
 
-    SP_OBJECT_CLASS(sp_symbol_parent_class)->write (object, xml_doc, repr, flags);
+    SPGroup::write(xml_doc, repr, flags);
 
     return repr;
 }
 
-static Inkscape::DrawingItem *sp_symbol_show(SPItem *item, Inkscape::Drawing &drawing, unsigned int key, unsigned int flags)
-{
-    SPSymbol *symbol = SP_SYMBOL(item);
+Inkscape::DrawingItem* SPSymbol::show(Inkscape::Drawing &drawing, unsigned int key, unsigned int flags) {
     Inkscape::DrawingItem *ai = 0;
 
-    if (symbol->cloned) {
+    if (this->cloned) {
         // Cloned <symbol> is actually renderable
-        if (((SPItemClass *) (sp_symbol_parent_class))->show) {
-            ai = ((SPItemClass *) (sp_symbol_parent_class))->show (item, drawing, key, flags);
-            Inkscape::DrawingGroup *g = dynamic_cast<Inkscape::DrawingGroup *>(ai);
-            if (g) {
-            	g->setChildTransform(symbol->c2p);
-            }
-        }
+        ai = SPGroup::show(drawing, key, flags);
+        Inkscape::DrawingGroup *g = dynamic_cast<Inkscape::DrawingGroup *>(ai);
+
+		if (g) {
+			g->setChildTransform(this->c2p);
+		}
     }
 
     return ai;
 }
 
-static void sp_symbol_hide(SPItem *item, unsigned int key)
-{
-    SPSymbol *symbol = SP_SYMBOL(item);
-
-    if (symbol->cloned) {
+void SPSymbol::hide(unsigned int key) {
+    if (this->cloned) {
         /* Cloned <symbol> is actually renderable */
-        if (((SPItemClass *) (sp_symbol_parent_class))->hide) {
-            ((SPItemClass *) (sp_symbol_parent_class))->hide (item, key);
-	}
+        SPGroup::hide(key);
     }
 }
 
-static Geom::OptRect sp_symbol_bbox(SPItem const *item, Geom::Affine const &transform, SPItem::BBoxType type)
-{
-    SPSymbol const *symbol = SP_SYMBOL(item);
+
+Geom::OptRect SPSymbol::bbox(Geom::Affine const &transform, SPItem::BBoxType type) {
     Geom::OptRect bbox;
 
     // We don't need a bounding box for Symbols dialog when selecting
     // symbols. They have no canvas location. But cloned symbols are.
-    if (symbol->cloned) {
-        // Cloned <symbol> is actually renderable
-
-        if (((SPItemClass *) (sp_symbol_parent_class))->bbox) {
-            Geom::Affine const a( symbol->c2p * transform );
-            bbox = ((SPItemClass *) (sp_symbol_parent_class))->bbox(item, a, type);
-        }
+    if (this->cloned) {
+    	Geom::Affine const a( this->c2p * transform );
+    	bbox = SPGroup::bbox(a, type);
     }
+
     return bbox;
 }
 
-static void sp_symbol_print(SPItem *item, SPPrintContext *ctx)
-{
-    SPSymbol *symbol = SP_SYMBOL(item);
-    if (symbol->cloned) {
+void SPSymbol::print(SPPrintContext* ctx) {
+    if (this->cloned) {
         // Cloned <symbol> is actually renderable
 
-        sp_print_bind(ctx, symbol->c2p, 1.0);
+        sp_print_bind(ctx, this->c2p, 1.0);
 
-        if (((SPItemClass *) (sp_symbol_parent_class))->print) {
-            ((SPItemClass *) (sp_symbol_parent_class))->print (item, ctx);
-        }
+        SPGroup::print(ctx);
 
         sp_print_release (ctx);
     }