Hello, David. On 6/25/07, David Sitsky <[EMAIL PROTECTED]> 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 ------------------------------------------------------------------------- This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and take control of your XML. No limits. Just data. Click to get it now. http://sourceforge.net/powerbar/db2/ _______________________________________________ Codestriker-user mailing list Codestriker-user@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/codestriker-user
