Hello, David.
On 6/25/07, David Sitsky <sits@...> wrote:
> Many thanks for your submission. I actually tried using
> Syntax::Hightlight::Engine::Kate when it first came out, and found it
> to be very slow. It would take something like 30 seconds to parse a
> small java file, so I decided against using it. What is the
> performance of it like now? Hopefully it has been fixed.
If Kate engine is used to highlight text line-by-line as in my code,
it process SQLite 3.3.17 amalgamation (2.1Mb, 41218 SLOCs reported by
sloccount) in about 5 minutes.
I think this speed is acceptable because:
1. Codestriker can't handle such a big files reliable (it creates
Subversion topic that consists of the same amalgamation more than
about 5 minutes)
2. I believe that reviewing such files isn't the common practice.
However if highlighting of our code would be too slow I'll consider to
make alternative highlighting using webc++ external highlighter.
> > I've used to redesign Codestriker::Http::Render.pm internal workflow a
> > little but in my day-to-day work It works just like before. The only
> > thing I haven't check in real life is LXR linking but I'm quite sure
> > it works too.
>
> Please check it out LXR linking if you can - it will save me the
> trouble. Also, once you have finish it, please submit your patch into
> the patches list:
> http://sourceforge.net/tracker/?group_id=41136&atid=429862.
Hm... I've never touched LXR and don't think I'll have a time to until
autumn, sorry.
But a quick test to check which identifiers are passed to lxr_ident
shows the same set for our projects except:
1. identifiers that start with underscore (I've added that to LXR code
as this is common for local C variables)
2. some tab issues related to
/^(\s| ){0,10}\*/
regexp (this is bacause of tab expansion now happens _after_
identifier search)
3. some spaces, as I've used to remove
$token =~ s/(\s| )//g;
code from identifier search code to leave positions untouched.
> Many thanks for your contribution! I'll give it a try later on - it
> would be great to have the code highlighted.
Patch posted to maillist was broken a little (it was not the final
version), sorry.
Fixed patch uploaded:
http://sourceforge.net/tracker/index.php?func=detail&aid=1742781&group_id=41136&atid=429862
--
Nikita V. Borodikhin, System Administrator NIKB-RIPN BNV7-RIPE
Registered Linux user #256562 with the Linux Counter
|