Hello,

I propose to put this patch in order to make tables usable again. I have been using this patch for some time and I didn't see any obvious side effect for now.

Objection?

Of course, this is not the right fix but I spend hours debugging through this code without success. So unless someone knows that code better the attached patch is good enough.

Abdel.

Index: BufferView.C
===================================================================
--- BufferView.C        (revision 15367)
+++ BufferView.C        (working copy)
@@ -1261,8 +1261,15 @@
 
 void BufferView::updateMetrics(bool singlepar)
 {
+       // FIXME (Abdel 19/10/2006):
+       // There something fishy in tabular. The coord_cache_ is not
+       // correctly reconstructed when a character is trying to be inserted.
+       // Not clearing out the coord_cache_ fixes the crash but I don't know
+       // what side effect this could have on other insets.
+       //
        // Remove old position cache
-       coord_cache_.clear();
+       // coord_cache_.clear();
+
        LyXText & buftext = buffer_->text();
        lyx::pit_type size = int(buftext.paragraphs().size());
 

Reply via email to