// Note: moveItem() does not honour change tracking!
// Therefore, it should only be used for breaking and merging paragraphs
- Font const tmpFont = fromPar.getFontSettings(params, fromPos);
- Change const tmpChange = fromPar.lookupChange(fromPos);
-
- Inset * tmpInset = fromPar.getInset(fromPos);
- if (tmpInset) {
- // the inset is not in the paragraph any more
- tmpInset = fromPar.releaseInset(fromPos);
- return toPar.insertInset(toPos, tmpInset, tmpFont, tmpChange);
+ Font const & tmpFont = fromPar.getFontSettings(params, fromPos);
+ Change const & tmpChange = fromPar.lookupChange(fromPos);
+
+ if (Inset * tmpInset = fromPar.getInset(fromPos)) {
+ fromPar.releaseInset(fromPos);
+ // The inset is not in fromPar any more.
+ if (!toPar.insertInset(toPos, tmpInset, tmpFont, tmpChange)) {
+ delete tmpInset;
+ return false;
+ }
+ return true;
}
char_type const tmpChar = fromPar.getChar(fromPos);
// breaking paragraph.
if (tmp->empty()) {
Font changed = tmp->getFirstFontSettings(bparams);
- Font old = par.getFontSettings(bparams, par.size());
+ Font const & old = par.getFontSettings(bparams, par.size());
changed.setLanguage(old.language());
tmp->setFont(0, changed);
}