Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes:

| >>>>> "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.

Or... just cache the LyXText at the begining of the method.
  
| Lars> I am testing your patch now, with a couple of minor changes.
| 
| OK. 

I applied your paragraph patch, pluss some changes to
paragraphparmeters.

-- 
        Lgb

Reply via email to