diff --git a/src/text-editing.cpp b/src/text-editing.cpp
index 7a6f5fd15fdef985346060017ed94ebfac35b087..b84361324338c62735017ba15f2407bd624e9d60 100644 (file)
--- a/src/text-editing.cpp
+++ b/src/text-editing.cpp
#include "style.h"
#include "unit-constants.h"
+#include "document.h"
#include "xml/repr.h"
#include "xml/attribute-record.h"
@@ -103,13 +104,14 @@ SPStyle const * sp_te_style_at_position(SPItem const *text, Inkscape::Text::Layo
Inkscape::Text::Layout const *layout = te_get_layout(text);
if (layout == NULL)
return NULL;
- union { SPObject const *op; void *vp; } pos_obj;
- pos_obj.vp = NULL;
- layout->getSourceOfCharacter(position, &pos_obj.vp);
- if (pos_obj.vp == NULL) pos_obj.op = text;
- while (SP_OBJECT_STYLE(pos_obj.op) == NULL)
- pos_obj.op = SP_OBJECT_PARENT(pos_obj.op); // SPStrings don't have style
- return SP_OBJECT_STYLE(pos_obj.op);
+ SPObject const *pos_obj = 0;
+ void *rawptr = 0;
+ layout->getSourceOfCharacter(position, &rawptr);
+ pos_obj = SP_OBJECT(rawptr);
+ if (pos_obj == 0) pos_obj = text;
+ while (SP_OBJECT_STYLE(pos_obj) == NULL)
+ pos_obj = SP_OBJECT_PARENT(pos_obj); // SPStrings don't have style
+ return SP_OBJECT_STYLE(pos_obj);
}
/*
return length;
}
-static Inkscape::XML::Node* duplicate_node_without_children(Inkscape::XML::Node const *old_node)
+static Inkscape::XML::Node* duplicate_node_without_children(Inkscape::XML::Document *xml_doc, Inkscape::XML::Node const *old_node)
{
switch (old_node->type()) {
case Inkscape::XML::ELEMENT_NODE: {
- Inkscape::XML::Node *new_node = sp_repr_new(old_node->name());
+ Inkscape::XML::Node *new_node = xml_doc->createElement(old_node->name());
Inkscape::Util::List<Inkscape::XML::AttributeRecord const> attributes = old_node->attributeList();
GQuark const id_key = g_quark_from_string("id");
for ( ; attributes ; attributes++) {
@@ -235,10 +237,10 @@ static Inkscape::XML::Node* duplicate_node_without_children(Inkscape::XML::Node
}
case Inkscape::XML::TEXT_NODE:
- return sp_repr_new_text(old_node->content());
+ return xml_doc->createTextNode(old_node->content());
case Inkscape::XML::COMMENT_NODE:
- return sp_repr_new_comment(old_node->content());
+ return xml_doc->createComment(old_node->content());
case Inkscape::XML::DOCUMENT_NODE:
return NULL; // this had better never happen
*/
static SPObject* split_text_object_tree_at(SPObject *split_obj, unsigned char_index)
{
+ Inkscape::XML::Document *xml_doc = sp_document_repr_doc(SP_OBJECT_DOCUMENT(split_obj));
if (is_line_break_object(split_obj)) {
- Inkscape::XML::Node *new_node = duplicate_node_without_children(SP_OBJECT_REPR(split_obj));
+ Inkscape::XML::Node *new_node = duplicate_node_without_children(xml_doc, SP_OBJECT_REPR(split_obj));
SP_OBJECT_REPR(SP_OBJECT_PARENT(split_obj))->addChild(new_node, SP_OBJECT_REPR(split_obj));
Inkscape::GC::release(new_node);
split_attributes(split_obj, SP_OBJECT_NEXT(split_obj), char_index);
@@ -285,7 +288,7 @@ static SPObject* split_text_object_tree_at(SPObject *split_obj, unsigned char_in
unsigned char_count_before = sum_sibling_text_lengths_before(split_obj);
SPObject *duplicate_obj = split_text_object_tree_at(SP_OBJECT_PARENT(split_obj), char_index + char_count_before);
// copy the split node
- Inkscape::XML::Node *new_node = duplicate_node_without_children(SP_OBJECT_REPR(split_obj));
+ Inkscape::XML::Node *new_node = duplicate_node_without_children(xml_doc, SP_OBJECT_REPR(split_obj));
SP_OBJECT_REPR(duplicate_obj)->appendChild(new_node);
Inkscape::GC::release(new_node);
@@ -319,30 +322,32 @@ Inkscape::Text::Layout::iterator sp_te_insert_line (SPItem *item, Inkscape::Text
return position;
Inkscape::Text::Layout const *layout = te_get_layout(item);
- union { SPObject *op; void *vp; } split_obj;
+ SPObject *split_obj = 0;
Glib::ustring::iterator split_text_iter;
- if (position == layout->end())
- split_obj.vp = NULL;
- else
- layout->getSourceOfCharacter(position, &split_obj.vp, &split_text_iter);
-
- if (split_obj.vp == NULL || is_line_break_object(split_obj.op)) {
- if (split_obj.vp == NULL) split_obj.op = item->lastChild();
- if (split_obj.vp) {
- Inkscape::XML::Node *new_node = duplicate_node_without_children(SP_OBJECT_REPR(split_obj.op));
- SP_OBJECT_REPR(SP_OBJECT_PARENT(split_obj.op))->addChild(new_node, SP_OBJECT_REPR(split_obj.op));
+ if (position != layout->end()) {
+ void *rawptr = 0;
+ layout->getSourceOfCharacter(position, &rawptr, &split_text_iter);
+ split_obj = SP_OBJECT(rawptr);
+ }
+
+ if (split_obj == 0 || is_line_break_object(split_obj)) {
+ if (split_obj == 0) split_obj = item->lastChild();
+ if (split_obj) {
+ Inkscape::XML::Document *xml_doc = sp_document_repr_doc(SP_OBJECT_DOCUMENT(split_obj));
+ Inkscape::XML::Node *new_node = duplicate_node_without_children(xml_doc, SP_OBJECT_REPR(split_obj));
+ SP_OBJECT_REPR(SP_OBJECT_PARENT(split_obj))->addChild(new_node, SP_OBJECT_REPR(split_obj));
Inkscape::GC::release(new_node);
}
- } else if (SP_IS_STRING(split_obj.op)) {
- Glib::ustring *string = &SP_STRING(split_obj.op)->string;
+ } else if (SP_IS_STRING(split_obj)) {
+ Glib::ustring *string = &SP_STRING(split_obj)->string;
unsigned char_index = 0;
for (Glib::ustring::iterator it = string->begin() ; it != split_text_iter ; it++)
char_index++;
// we need to split the entire text tree into two
- SPString *new_string = SP_STRING(split_text_object_tree_at(split_obj.op, char_index));
+ SPString *new_string = SP_STRING(split_text_object_tree_at(split_obj, char_index));
SP_OBJECT_REPR(new_string)->setContent(&*split_text_iter.base()); // a little ugly
string->erase(split_text_iter, string->end());
- SP_OBJECT_REPR(split_obj.op)->setContent(string->c_str());
+ SP_OBJECT_REPR(split_obj)->setContent(string->c_str());
// TODO: if the split point was at the beginning of a span we have a whole load of empty elements to clean up
} else {
// TODO
@@ -405,49 +410,52 @@ sp_te_insert(SPItem *item, Inkscape::Text::Layout::iterator const &position, gch
}
Inkscape::Text::Layout const *layout = te_get_layout(item);
- union { SPObject *op; void *vp; } source_obj;
+ SPObject *source_obj = 0;
+ void *rawptr = 0;
Glib::ustring::iterator iter_text;
// we want to insert after the previous char, not before the current char.
// it makes a difference at span boundaries
Inkscape::Text::Layout::iterator it_prev_char = position;
bool cursor_at_start = !it_prev_char.prevCharacter();
bool cursor_at_end = position == layout->end();
- layout->getSourceOfCharacter(it_prev_char, &source_obj.vp, &iter_text);
- if (SP_IS_STRING(source_obj.op)) {
+ layout->getSourceOfCharacter(it_prev_char, &rawptr, &iter_text);
+ source_obj = SP_OBJECT(rawptr);
+ if (SP_IS_STRING(source_obj)) {
// the simple case
if (!cursor_at_start) iter_text++;
- SPString *string_item = SP_STRING(source_obj.op);
+ SPString *string_item = SP_STRING(source_obj);
insert_into_spstring(string_item, cursor_at_end ? string_item->string.end() : iter_text, utf8);
} else {
// the not-so-simple case where we're at a line break or other control char; add to the next child/sibling SPString
+ Inkscape::XML::Document *xml_doc = SP_OBJECT_REPR(item)->document();
if (cursor_at_start) {
- source_obj.op = item;
- if (source_obj.op->hasChildren()) {
- source_obj.op = source_obj.op->firstChild();
+ source_obj = item;
+ if (source_obj->hasChildren()) {
+ source_obj = source_obj->firstChild();
if (SP_IS_FLOWTEXT(item)) {
- while (SP_IS_FLOWREGION(source_obj.op) || SP_IS_FLOWREGIONEXCLUDE(source_obj.op))
- source_obj.op = SP_OBJECT_NEXT(source_obj.op);
- if (source_obj.vp == NULL)
- source_obj.op = item;
+ while (SP_IS_FLOWREGION(source_obj) || SP_IS_FLOWREGIONEXCLUDE(source_obj))
+ source_obj = SP_OBJECT_NEXT(source_obj);
+ if (source_obj == NULL)
+ source_obj = item;
}
}
- if (source_obj.op == item && SP_IS_FLOWTEXT(item)) {
- Inkscape::XML::Node *para = sp_repr_new("svg:flowPara");
+ if (source_obj == item && SP_IS_FLOWTEXT(item)) {
+ Inkscape::XML::Node *para = xml_doc->createElement("svg:flowPara");
SP_OBJECT_REPR(item)->appendChild(para);
- source_obj.op = item->lastChild();
+ source_obj = item->lastChild();
}
} else
- source_obj.op = SP_OBJECT_NEXT(source_obj.op);
+ source_obj = SP_OBJECT_NEXT(source_obj);
- if (source_obj.vp) { // never fails
- SPString *string_item = sp_te_seek_next_string_recursive(source_obj.op);
+ if (source_obj) { // never fails
+ SPString *string_item = sp_te_seek_next_string_recursive(source_obj);
if (string_item == NULL) {
// need to add an SPString in this (pathological) case
- Inkscape::XML::Node *rstring = sp_repr_new_text("");
- SP_OBJECT_REPR(source_obj.op)->addChild(rstring, NULL);
+ Inkscape::XML::Node *rstring = xml_doc->createTextNode("");
+ SP_OBJECT_REPR(source_obj)->addChild(rstring, NULL);
Inkscape::GC::release(rstring);
- g_assert(SP_IS_STRING(source_obj.op->firstChild()));
- string_item = SP_STRING(source_obj.op->firstChild());
+ g_assert(SP_IS_STRING(source_obj->firstChild()));
+ string_item = SP_STRING(source_obj->firstChild());
}
insert_into_spstring(string_item, cursor_at_end ? string_item->string.end() : string_item->string.begin(), utf8);
}
@@ -520,7 +528,8 @@ static SPObject* delete_line_break(SPObject *root, SPObject *item, bool *next_is
<p><div></div>*text</p>
<p><div></div></p><p>*text</p>
*/
- Inkscape::XML::Node *new_span_repr = sp_repr_new(span_name_for_text_object(root));
+ Inkscape::XML::Document *xml_doc = SP_OBJECT_REPR(item)->document();
+ Inkscape::XML::Node *new_span_repr = xml_doc->createElement(span_name_for_text_object(root));
if (gchar const *a = this_repr->attribute("dx"))
new_span_repr->setAttribute("dx", a);
@@ -625,33 +634,36 @@ sp_te_delete (SPItem *item, Inkscape::Text::Layout::iterator const &start, Inksc
last = start;
}
Inkscape::Text::Layout const *layout = te_get_layout(item);
- union { SPObject *op; void *vp; } start_item, end_item;
+ SPObject *start_item = 0, *end_item = 0;
+ void *rawptr = 0;
Glib::ustring::iterator start_text_iter, end_text_iter;
- layout->getSourceOfCharacter(first, &start_item.vp, &start_text_iter);
- layout->getSourceOfCharacter(last, &end_item.vp, &end_text_iter);
- if (start_item.vp == NULL)
+ layout->getSourceOfCharacter(first, &rawptr, &start_text_iter);
+ start_item = SP_OBJECT(rawptr);
+ layout->getSourceOfCharacter(last, &rawptr, &end_text_iter);
+ end_item = SP_OBJECT(rawptr);
+ if (start_item == 0)
return first; // start is at end of text
- if (is_line_break_object(start_item.op))
- move_to_end_of_paragraph(&start_item.op, &start_text_iter);
- if (end_item.vp == NULL) {
- end_item.op = item->lastChild();
- move_to_end_of_paragraph(&end_item.op, &end_text_iter);
+ if (is_line_break_object(start_item))
+ move_to_end_of_paragraph(&start_item, &start_text_iter);
+ if (end_item == 0) {
+ end_item = item->lastChild();
+ move_to_end_of_paragraph(&end_item, &end_text_iter);
}
- else if (is_line_break_object(end_item.op))
- move_to_end_of_paragraph(&end_item.op, &end_text_iter);
+ else if (is_line_break_object(end_item))
+ move_to_end_of_paragraph(&end_item, &end_text_iter);
- SPObject *common_ancestor = get_common_ancestor(item, start_item.op, end_item.op);
+ SPObject *common_ancestor = get_common_ancestor(item, start_item, end_item);
- if (start_item.op == end_item.op) {
+ if (start_item == end_item) {
// the quick case where we're deleting stuff all from the same string
- if (SP_IS_STRING(start_item.op)) { // always true (if it_start != it_end anyway)
- erase_from_spstring(SP_STRING(start_item.op), start_text_iter, end_text_iter);
+ if (SP_IS_STRING(start_item)) { // always true (if it_start != it_end anyway)
+ erase_from_spstring(SP_STRING(start_item), start_text_iter, end_text_iter);
}
} else {
- SPObject *sub_item = start_item.op;
+ SPObject *sub_item = start_item;
// walk the tree from start_item to end_item, deleting as we go
while (sub_item != item) {
- if (sub_item == end_item.op) {
+ if (sub_item == end_item) {
if (SP_IS_STRING(sub_item)) {
Glib::ustring *string = &SP_STRING(sub_item)->string;
erase_from_spstring(SP_STRING(sub_item), string->begin(), end_text_iter);
@@ -660,7 +672,7 @@ sp_te_delete (SPItem *item, Inkscape::Text::Layout::iterator const &start, Inksc
}
if (SP_IS_STRING(sub_item)) {
SPString *string = SP_STRING(sub_item);
- if (sub_item == start_item.op)
+ if (sub_item == start_item)
erase_from_spstring(string, start_text_iter, string->string.end());
else
erase_from_spstring(string, string->string.begin(), string->string.end());
@@ -684,7 +696,7 @@ sp_te_delete (SPItem *item, Inkscape::Text::Layout::iterator const &start, Inksc
sub_item = next_item;
if (is_sibling) break;
// no more siblings, go up a parent
- } while (sub_item != item && sub_item != end_item.op);
+ } while (sub_item != item && sub_item != end_item);
}
}
}
@@ -749,10 +761,12 @@ sp_te_get_string_multiline (SPItem const *text, Inkscape::Text::Layout::iterator
Glib::ustring result;
// not a particularly fast piece of code. I'll optimise it if people start to notice.
for ( ; first < last ; first.nextCharacter()) {
- union { SPObject *op; void *vp; } char_item;
+ SPObject *char_item = 0;
+ void *rawptr = 0;
Glib::ustring::iterator text_iter;
- layout->getSourceOfCharacter(first, &char_item.vp, &text_iter);
- if (SP_IS_STRING(char_item.op))
+ layout->getSourceOfCharacter(first, &rawptr, &text_iter);
+ char_item = SP_OBJECT(rawptr);
+ if (SP_IS_STRING(char_item))
result += *text_iter;
else
result += '\n';
g_return_if_fail (text != NULL);
g_return_if_fail (SP_IS_TEXT(text) || SP_IS_FLOWTEXT(text));
+ Inkscape::XML::Document *xml_doc = SP_OBJECT_REPR(text)->document();
Inkscape::XML::Node *repr;
SPObject *object;
bool is_textpath = false;
if (e) *e = '\0';
Inkscape::XML::Node *rtspan;
if (SP_IS_TEXT(text)) { // create a tspan for each line
- rtspan = sp_repr_new ("svg:tspan");
+ rtspan = xml_doc->createElement("svg:tspan");
rtspan->setAttribute("sodipodi:role", "line");
} else { // create a flowPara for each line
- rtspan = sp_repr_new ("svg:flowPara");
+ rtspan = xml_doc->createElement("svg:flowPara");
}
- Inkscape::XML::Node *rstr = sp_repr_new_text(p);
+ Inkscape::XML::Node *rstr = xml_doc->createTextNode(p);
rtspan->addChild(rstr, NULL);
Inkscape::GC::release(rstr);
repr->appendChild(rtspan);
p = (e) ? e + 1 : NULL;
}
if (is_textpath) {
- Inkscape::XML::Node *rstr = sp_repr_new_text(content);
+ Inkscape::XML::Node *rstr = xml_doc->createTextNode(content);
repr->addChild(rstr, NULL);
Inkscape::GC::release(rstr);
}
@@ -833,17 +848,19 @@ text_tag_attributes_at_position(SPItem *item, Inkscape::Text::Layout::iterator c
return NULL; // flowtext doesn't support kerning yet
SPText *text = SP_TEXT(item);
- union { SPObject *op; void *vp; } source_item;
+ SPObject *source_item = 0;
+ void *rawptr = 0;
Glib::ustring::iterator source_text_iter;
- text->layout.getSourceOfCharacter(position, &source_item.vp, &source_text_iter);
+ text->layout.getSourceOfCharacter(position, &rawptr, &source_text_iter);
+ source_item = SP_OBJECT(rawptr);
- if (!SP_IS_STRING(source_item.op)) return NULL;
- Glib::ustring *string = &SP_STRING(source_item.op)->string;
- *char_index = sum_sibling_text_lengths_before(source_item.op);
+ if (!SP_IS_STRING(source_item)) return NULL;
+ Glib::ustring *string = &SP_STRING(source_item)->string;
+ *char_index = sum_sibling_text_lengths_before(source_item);
for (Glib::ustring::iterator it = string->begin() ; it != source_text_iter ; it++)
++*char_index;
- return attributes_for_object(SP_OBJECT_PARENT(source_item.op));
+ return attributes_for_object(SP_OBJECT_PARENT(source_item));
}
void
@@ -876,12 +893,14 @@ sp_te_adjust_rotation_screen(SPItem *text, Inkscape::Text::Layout::iterator cons
gdouble factor = 1 / desktop->current_zoom();
NR::Matrix t = sp_item_i2doc_affine(text);
factor = factor / NR::expansion(t);
- union { SPObject *op; void *vp; } source_item;
Inkscape::Text::Layout const *layout = te_get_layout(text);
if (layout == NULL) return;
- layout->getSourceOfCharacter(std::min(start, end), &source_item.vp);
- if (source_item.op == NULL) return;
- gdouble degrees = (180/M_PI) * atan2(pixels, SP_OBJECT_PARENT(source_item.op)->style->font_size.computed / factor);
+ SPObject *source_item = 0;
+ void *rawptr = 0;
+ layout->getSourceOfCharacter(std::min(start, end), &rawptr);
+ source_item = SP_OBJECT(rawptr);
+ if (source_item == 0) return;
+ gdouble degrees = (180/M_PI) * atan2(pixels, SP_OBJECT_PARENT(source_item)->style->font_size.computed / factor);
sp_te_adjust_rotation(text, start, end, desktop, degrees);
}
@@ -914,18 +933,20 @@ sp_te_adjust_tspan_letterspacing_screen(SPItem *text, Inkscape::Text::Layout::it
Inkscape::Text::Layout const *layout = te_get_layout(text);
gdouble val;
- union { SPObject *op; void *vp; } source_obj;
+ SPObject *source_obj = 0;
+ void *rawptr = 0;
unsigned nb_let;
- layout->getSourceOfCharacter(std::min(start, end), &source_obj.vp);
+ layout->getSourceOfCharacter(std::min(start, end), &rawptr);
+ source_obj = SP_OBJECT(rawptr);
- if (source_obj.vp == NULL) { // end of text
- source_obj.op = text->lastChild();
+ if (source_obj == 0) { // end of text
+ source_obj = text->lastChild();
}
- if (SP_IS_STRING(source_obj.op)) {
- source_obj.op = source_obj.op->parent;
+ if (SP_IS_STRING(source_obj)) {
+ source_obj = source_obj->parent;
}
- SPStyle *style = SP_OBJECT_STYLE (source_obj.op);
+ SPStyle *style = SP_OBJECT_STYLE (source_obj);
// calculate real value
/* TODO: Consider calculating val unconditionally, i.e. drop the first `if' line, and
@@ -943,9 +964,9 @@ sp_te_adjust_tspan_letterspacing_screen(SPItem *text, Inkscape::Text::Layout::it
}
if (start == end) {
- while (!is_line_break_object(source_obj.op)) // move up the tree so we apply to the closest paragraph
- source_obj.op = SP_OBJECT_PARENT(source_obj.op);
- nb_let = sp_text_get_length(source_obj.op);
+ while (!is_line_break_object(source_obj)) // move up the tree so we apply to the closest paragraph
+ source_obj = SP_OBJECT_PARENT(source_obj);
+ nb_let = sp_text_get_length(source_obj);
} else {
nb_let = abs(layout->iteratorToCharIndex(end) - layout->iteratorToCharIndex(start));
}
@@ -955,7 +976,7 @@ sp_te_adjust_tspan_letterspacing_screen(SPItem *text, Inkscape::Text::Layout::it
gdouble const zoom = desktop->current_zoom();
gdouble const zby = (by
/ (zoom * (nb_let > 1 ? nb_let - 1 : 1))
- / NR::expansion(sp_item_i2doc_affine(SP_ITEM(source_obj.op))));
+ / NR::expansion(sp_item_i2doc_affine(SP_ITEM(source_obj))));
val += zby;
if (start == end) {
static void recursively_apply_style(SPObject *common_ancestor, SPCSSAttr const *css, SPObject *start_item, Glib::ustring::iterator start_text_iter, SPObject *end_item, Glib::ustring::iterator end_text_iter, char const *span_object_name)
{
bool passed_start = start_item == NULL ? true : false;
-
+ Inkscape::XML::Document *xml_doc = sp_document_repr_doc(SP_OBJECT_DOCUMENT(common_ancestor));
+
for (SPObject *child = common_ancestor->firstChild() ; child != NULL ; child = SP_OBJECT_NEXT(child)) {
if (start_item == child)
passed_start = true;
@@ -1202,7 +1224,7 @@ static void recursively_apply_style(SPObject *common_ancestor, SPCSSAttr const *
SPString *string_item = SP_STRING(child);
bool surround_entire_string = true;
- Inkscape::XML::Node *child_span = sp_repr_new(span_object_name);
+ Inkscape::XML::Node *child_span = xml_doc->createElement(span_object_name);
sp_repr_css_set(child_span, const_cast<SPCSSAttr*>(css), "style"); // better hope that prototype wasn't nonconst for a good reason
SPObject *prev_item = SP_OBJECT_PREV(child);
Inkscape::XML::Node *prev_repr = prev_item ? SP_OBJECT_REPR(prev_item) : NULL;
@@ -1214,11 +1236,11 @@ static void recursively_apply_style(SPObject *common_ancestor, SPCSSAttr const *
unsigned start_char_index = char_index_of_iterator(string_item->string, start_text_iter);
unsigned end_char_index = char_index_of_iterator(string_item->string, end_text_iter);
- Inkscape::XML::Node *text_before = sp_repr_new_text(string_item->string.substr(0, start_char_index).c_str());
+ Inkscape::XML::Node *text_before = xml_doc->createTextNode(string_item->string.substr(0, start_char_index).c_str());
SP_OBJECT_REPR(common_ancestor)->addChild(text_before, prev_repr);
SP_OBJECT_REPR(common_ancestor)->addChild(child_span, text_before);
Inkscape::GC::release(text_before);
- Inkscape::XML::Node *text_in_span = sp_repr_new_text(string_item->string.substr(start_char_index, end_char_index - start_char_index).c_str());
+ Inkscape::XML::Node *text_in_span = xml_doc->createTextNode(string_item->string.substr(start_char_index, end_char_index - start_char_index).c_str());
child_span->appendChild(text_in_span);
Inkscape::GC::release(text_in_span);
SP_OBJECT_REPR(child)->setContent(string_item->string.substr(end_char_index).c_str());
@@ -1230,7 +1252,7 @@ static void recursively_apply_style(SPObject *common_ancestor, SPCSSAttr const *
unsigned end_char_index = char_index_of_iterator(string_item->string, end_text_iter);
SP_OBJECT_REPR(common_ancestor)->addChild(child_span, prev_repr);
- Inkscape::XML::Node *text_in_span = sp_repr_new_text(string_item->string.substr(0, end_char_index).c_str());
+ Inkscape::XML::Node *text_in_span = xml_doc->createTextNode(string_item->string.substr(0, end_char_index).c_str());
child_span->appendChild(text_in_span);
Inkscape::GC::release(text_in_span);
SP_OBJECT_REPR(child)->setContent(string_item->string.substr(end_char_index).c_str());
@@ -1239,11 +1261,11 @@ static void recursively_apply_style(SPObject *common_ancestor, SPCSSAttr const *
// eg "abcDEF" -> "abc"<span>"DEF"</span>
unsigned start_char_index = char_index_of_iterator(string_item->string, start_text_iter);
- Inkscape::XML::Node *text_before = sp_repr_new_text(string_item->string.substr(0, start_char_index).c_str());
+ Inkscape::XML::Node *text_before = xml_doc->createTextNode(string_item->string.substr(0, start_char_index).c_str());
SP_OBJECT_REPR(common_ancestor)->addChild(text_before, prev_repr);
SP_OBJECT_REPR(common_ancestor)->addChild(child_span, text_before);
Inkscape::GC::release(text_before);
- Inkscape::XML::Node *text_in_span = sp_repr_new_text(string_item->string.substr(start_char_index).c_str());
+ Inkscape::XML::Node *text_in_span = xml_doc->createTextNode(string_item->string.substr(start_char_index).c_str());
child_span->appendChild(text_in_span);
Inkscape::GC::release(text_in_span);
child->deleteObject();
@@ -1487,7 +1509,8 @@ static bool redundant_semi_nesting_processor(SPObject **item, SPObject *child, b
sp_repr_css_attr_unref(css_child_only);
if (!equal) return false;
- Inkscape::XML::Node *new_span = sp_repr_new(SP_OBJECT_REPR(*item)->name());
+ Inkscape::XML::Document *xml_doc = SP_OBJECT_REPR(*item)->document();
+ Inkscape::XML::Node *new_span = xml_doc->createElement(SP_OBJECT_REPR(*item)->name());
if (prepend) {
SPObject *prev = SP_OBJECT_PREV(*item);
SP_OBJECT_REPR(SP_OBJECT_PARENT(*item))->addChild(new_span, prev ? SP_OBJECT_REPR(prev) : NULL);
@@ -1646,17 +1669,20 @@ void sp_te_apply_style(SPItem *text, Inkscape::Text::Layout::iterator const &sta
last = start;
}
Inkscape::Text::Layout const *layout = te_get_layout(text);
- union { SPObject *op; void *vp; } start_item, end_item;
+ SPObject *start_item = 0, *end_item = 0;
+ void *rawptr = 0;
Glib::ustring::iterator start_text_iter, end_text_iter;
- layout->getSourceOfCharacter(first, &start_item.vp, &start_text_iter);
- layout->getSourceOfCharacter(last, &end_item.vp, &end_text_iter);
- if (start_item.vp == NULL)
+ layout->getSourceOfCharacter(first, &rawptr, &start_text_iter);
+ start_item = SP_OBJECT(rawptr);
+ layout->getSourceOfCharacter(last, &rawptr, &end_text_iter);
+ end_item = SP_OBJECT(rawptr);
+ if (start_item == 0)
return; // start is at end of text
- if (is_line_break_object(start_item.op))
- start_item.op = SP_OBJECT_NEXT(start_item.op);
- if (is_line_break_object(end_item.op))
- end_item.op = SP_OBJECT_NEXT(end_item.op);
- if (end_item.op == NULL) end_item.op = text;
+ if (is_line_break_object(start_item))
+ start_item = SP_OBJECT_NEXT(start_item);
+ if (is_line_break_object(end_item))
+ end_item = SP_OBJECT_NEXT(end_item);
+ if (end_item == 0) end_item = text;
/* stage 1: applying the style. Go up to the closest common ancestor of
start and end and then semi-recursively apply the style to all the
@@ -1666,10 +1692,10 @@ void sp_te_apply_style(SPItem *text, Inkscape::Text::Layout::iterator const &sta
eg: <span>abcDEF</span><span>GHI</span><span>JKLmno</span>
The recursion may involve creating new spans.
*/
- SPObject *common_ancestor = get_common_ancestor(text, start_item.op, end_item.op);
- start_item.op = ascend_while_first(start_item.op, start_text_iter, common_ancestor);
- end_item.op = ascend_while_first(end_item.op, end_text_iter, common_ancestor);
- recursively_apply_style(common_ancestor, css, start_item.op, start_text_iter, end_item.op, end_text_iter, span_name_for_text_object(text));
+ SPObject *common_ancestor = get_common_ancestor(text, start_item, end_item);
+ start_item = ascend_while_first(start_item, start_text_iter, common_ancestor);
+ end_item = ascend_while_first(end_item, end_text_iter, common_ancestor);
+ recursively_apply_style(common_ancestor, css, start_item, start_text_iter, end_item, end_text_iter, span_name_for_text_object(text));
/* stage 2: cleanup the xml tree (of which there are multiple passes) */
/* discussion: this stage requires a certain level of inventiveness because