The branch, master, has been updated.

- Log -----------------------------------------------------------------

commit 3a99e351d9f2678b0d8274ae7c03a73f6bea03eb
Merge: 6a936d6 c235e73
Author: Jean-Marc Lasgouttes <lasgout...@lyx.org>
Date:   Mon Aug 20 17:39:25 2012 +0200

    Merge branch 'master' of git.lyx.org:lyx


commit 6a936d63d3be55a9a553b252e51f773749777e44
Author: Jean-Marc Lasgouttes <lasgout...@lyx.org>
Date:   Mon Aug 20 17:20:59 2012 +0200

    Fix crash with undo in nested insets introduced by commit 6ec1683
    
    The code in DocIterator::sanitize now follows more closely the
    previous StableDocIterator::asDocIterator code. In particular, it adds
    the slices one by one, since fixIfBroken will chop the cursor
    otherwise.

diff --git a/src/DocIterator.cpp b/src/DocIterator.cpp
index 5cd1c0e..420c782 100644
--- a/src/DocIterator.cpp
+++ b/src/DocIterator.cpp
@@ -532,12 +532,14 @@ bool DocIterator::fixIfBroken()
 
 void DocIterator::sanitize()
 {
-       // this function re-creates the cache of inset pointers
-       //lyxerr << "converting:\n" << *this << endl;
+       // keep a copy of the slices
+       vector<CursorSlice> const sl = slices_;
+       slices_.clear();
        if (buffer_)
                inset_ = &buffer_->inset();
        Inset * inset = inset_;
-       for (size_t i = 0, n = slices_.size(); i != n; ++i) {
+       // re-add the slices one by one, and adjust the inset pointer.
+       for (size_t i = 0, n = sl.size(); i != n; ++i) {
                if (inset == 0) {
                        // FIXME
                        LYXERR0(" Should not happen, but does e.g. after "
@@ -548,14 +550,13 @@ void DocIterator::sanitize()
                        fixIfBroken();
                        break;
                }
-               slices_[i].inset_ = inset;
+               push_back(sl[i]);
+               top().inset_ = inset;
                if (fixIfBroken())
                        break;
                if (i + 1 != n)
-                       inset = slices_[i].inset().inMathed() ? 
slices_[i].cell()[slices_[i].pos()].nucleus() 
-                               : slices_[i].paragraph().getInset(pos());
+                       inset = nextInset();
        }
-       //lyxerr << "convert:\n" << *this << " to:\n" << dit << endl;
 }
 
 
@@ -626,7 +627,7 @@ StableDocIterator::StableDocIterator(DocIterator const & 
dit)
 
 DocIterator StableDocIterator::asDocIterator(Buffer * buf) const
 {
-       DocIterator dit = DocIterator(buf);
+       DocIterator dit(buf);
        dit.slices_ = data_;
        dit.sanitize();
        return dit;

-----------------------------------------------------------------------

Summary of changes:
 src/DocIterator.cpp |   17 +++++++++--------
 1 files changed, 9 insertions(+), 8 deletions(-)


hooks/post-receive
-- 
The LyX Source Repository

Reply via email to