On 04/14/2011 03:59 PM, Peter Kuemmel wrote:
This patch

     http://www.lyx.org/trac/attachment/ticket/7394/resetAnchor.patch

fixes the crash in

     http://www.lyx.org/trac/ticket/7394

But I don't know if it introduces any side effects.

The code before the patch,

     anchor_ = doc_iterator_begin(buffer());
     anchor_.clear();

looks like a NOP.

Couldn't the assignment be removed?

Hmm. It does look that way. But I'm inclined to think that maybe it is anchor_.clear() that ought to be removed. We seem to have no anchor at all after that, whereas the first bit sets it to the beginning of the Buffer.

Richard

Reply via email to