At 10:12 AM 9/5/2002 +0900, Stephen J. Turnbull wrote:
>>>>> "Andy" == Andy Piper
<andyp(a)san-francisco.bea.com> writes:
Andy> *sigh* oh dear. Maybe I will have to revert the change.
I think you should. We have a crash report, we have performance
problems being reported, we have a known reversion of a patch that Ben
Actually it turns out the performance problem is unrelated (to even XEmacs)
and the crash is unrelated also.
said prevents crashes, and we have _many_ reversions that have at best
The Ben thing I will fix.
In the meantime I've reduced your patch to about half its
original
size. I think I can halve it again, at which point it should be
pretty obvious where to start looking for "Matt's molasses".
That's cool. If that's the only good thing I have provoked that's still cool.
One interesting fact that has been identified already is that Matt
substituted parse-partial-sexp for syntactically-sectionalize in the
core font-lock function dealing with syntactic fontification.
Interestingly enough, syntactically-sectionalize has _no_
update_syntax_cache code in it. Why don't you revert your patch and
just reapply the font-lock.el portion, and see what happens to JDE
performance?
font-lock has very little to do with it. Turning off font-lock entirely
still does not make any appreciable difference to indentation slowness
(maybe it shaved a second off the 15, I don't know). I didn't see much
difference in font-lock performance with or without matt's patch, the only
reason I included it was because I don't have time to come up with a proper
patch and messing with it is a sure route to disaster.
andy