https://bugs.kde.org/show_bug.cgi?id=465841

--- Comment #19 from Waqar Ahmed <waqar....@gmail.com> ---
Should now be fast without dynamic wrapping too

> with the C style indenter - about 20 seconds (and also there are no 
> unexpected indentation

This was a bug/perf issue in Cstyle indenter which I have fixed now. It's now
7-8 times faster than before according to my measurements with a C++ code of
about 5K lines. It was taking 9+ seconds and has now dropped to 1.4 seconds.
There is about a ~900 ms call penalty i.e., the call overhead of calling into
the Javascript indentation functions. So the actual work is being done in
~500ms. Even though this is a big improvement its still very slow in my eyes
but there is not much we can do to squeeze out a bigger gain. 

I will take a look at the xml one when I find some time. 

Besides that, I think the important problems in this bug report are now all
addressed. Thanks for the report. If you find more issues, feel free to open
new bugs.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to