Georg Baum wrote:
Abdelrazak Younes wrote:Abdelrazak Younes wrote:Abdelrazak Younes wrote:Hello, It took a great deal of step by step debugging but my supposition was right, redoParagraph() is the culprit! The reason why the slowdown (and the csrss.exe) appears only in the line following a big formula is because of this code in rowpainter.C, line 1008: // Cache one paragraph above and one below // Note MV: this cannot be suppressed even for singlepar. // Try viewing the User Guide Mobius figureThe funny thing is that trying to view this figure in UserGuide.lyx crashes LyX even though this code is here...Even funnier is that removing this code solves the crash and the speed problem at the same time! Looks like my metrics cleanup has some nice side effects... I'm doubly good! :-)I don't understand at all. Can you please explain in more detail for somebody who is not so familiar with that code? Why is caching the paragraph below and above not needed anymore? Why was it needed formerly?
I have gone too fast here... it is still needed apparently but it was not not done correctly, hence the crash. I will clean this up at put the CoordCache update at the correct place. I am going to do this on demand instead of everytime. And the correct place is when you change screen display because of a FitCursor.
Do you say that fontmetrics and paragraph breaking in math is too slow?
If you look at my test document, this is no surprise really. Considering what we ask mathed to do, I'd say it's pretty fast. The problem is that we updated the whole formula before at each key stroke whereas it was not needed at all!
I would not understand that, because it is (according to Andre) basically unchanged from 1.3 (and there it was quick enough).
The code that I just removed appeared in 1.4 I think.
It rather looks to me that it was called far too often when it was not needed. But anyway, it really looks like you are a multiple hero, since I am sure that the remining questions can be answered easily.
Yes :-). I know where the path to follow from now on! Abdel.
attachment.lyx
Description: application/lyx
