>>>>> "Juergen" == Juergen Spitzmueller <[EMAIL PROTECTED]> writes:

Juergen> Georg Baum wrote:
>> The patch has been tested by Simon, and it works. Is it OK for
>> trunk and 1.4?

Juergen> I can confirm that it works. And I think it should go in
Juergen> 1.4.1. It's also a prerequisite for a patch I have to make
Juergen> pasteselection of tabulator-separated contents (e.g. from
Juergen> spreadsheets or a second LyX instance) from the clipboard to
Juergen> tabular work again.

Hmm, considering that Georg's solution is not particularly pretty (not
your fault Georg, it is due to the need for this viewcache), wouldn't
the following trivial patch be simpler?

Or is it really too trivial to be good? I understands it sprinkles
more of this setViewCache badness, but at least it is something easy
to spot and nuke when/if we find a better way.

JMarc

Index: src/insets/insettabular.C
===================================================================
--- src/insets/insettabular.C	(revision 13424)
+++ src/insets/insettabular.C	(working copy)
@@ -1937,6 +1937,7 @@ bool InsetTabular::insertAsciiString(Buf
 	if (cell < cells && op < len) {
 		shared_ptr<InsetText> inset = loctab->getCellInset(cell);
 		Paragraph & par = inset->text_.getPar(0);
+		inset->setViewCache(&bv);
 		LyXFont const font = inset->text_.getFont(par, 0);
 		inset->setText(buf.substr(op, len - op), font);
 	}

Reply via email to