>>>>> "Lars" == Lars Gullik Bjønnes <[EMAIL PROTECTED]> writes:

Lars> Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes: | I
Lars> used sometime yesterday evening to run gprof with large tables,
Lars> and | came up with the following patch. I posted it here instead
Lars> of applying | because it seems so obvious that I suspect I may
Lars> have misunderstood | something. | | The two main problems are: |
Lars> | - InsetTabular::clone() duplicates its LyXTabular twice

Lars> yes, your change should be good.

I'll apply it to 1.1.6 too, then.
 
Lars> A simple assiative map implemented with std::vector<std::pair<X,
Lars> Y> > would probably be just as good. Would be nice to keep the
Lars> std::map interface though.

_Or_ directly use the LyXText in the methods arguments.
 
Lars> You probably do not need this as it is done two lines below
Lars> also...

Indeed.

Lars> I am testing your patch now, with a couple of minor changes.

OK. 

JMarc

Reply via email to