// 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);
+ 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.
- return toPar.insertInset(toPos, tmpInset, tmpFont, tmpChange);
+ 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);
}