From 9e21d00fb1053897420f80d05a9815c5b2bbf312 Mon Sep 17 00:00:00 2001
From: mc <>
Date: Wed, 18 Feb 2015 11:25:23 +0100
Subject: I can't really understand why, but i can now launch inkscape without
 it segfaulting. That's an improvement. Next thing: code cleaning, replacing
 containers with vectors

(bzr r13922.1.4)
---
 src/selection.cpp | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

(limited to 'src/selection.cpp')

diff --git a/src/selection.cpp b/src/selection.cpp
index b509f4272..cbde9a799 100644
--- a/src/selection.cpp
+++ b/src/selection.cpp
@@ -43,7 +43,7 @@ namespace Inkscape {
 
 Selection::Selection(LayerModel *layers, SPDesktop *desktop) :
     _objs(SelContainer()),
-    _reprs(SelContainer()),
+    _reprs(std::vector<XML::Node*>()),
     _items(SelContainer()),
     _layers(layers),
     _desktop(desktop),
@@ -296,15 +296,13 @@ SelContainer const &Selection::itemList() {
     return _items;
 }
 
-SelContainer const &Selection::reprList() {
+std::vector<XML::Node*> const &Selection::reprList() {
     if (!_reprs.empty()) { return _reprs; }
     SelContainer list = itemList();
     for ( SelContainer::const_iterator iter=list.begin();iter!=list.end();iter++ ) {
         SPObject *obj=reinterpret_cast<SPObject *>(*iter);
-        _reprs.push_front(dynamic_cast<SPObject*>(obj->getRepr()));
+        _reprs.push_back(obj->getRepr());
     }
-    _reprs.reverse();
-
     return _reprs;
 }
 
@@ -343,7 +341,7 @@ SPObject *Selection::single() {
 
 SPItem *Selection::singleItem() {
 	SelContainer const items=itemList();
-    if ( !items.size()==1) {
+    if ( items.size()==1) {
         return reinterpret_cast<SPItem *>(items.front());
     } else {
         return NULL;
-- 
cgit v1.2.3