<div dir="ltr"><div class="gmail_default" style="font-family:tahoma,sans-serif">I am sorry, I was never able replicate this bug.</div><div class="gmail_default" style="font-family:tahoma,sans-serif"><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif">If it is this regex that causes problems, it can be removed completely. It is used to enable clicking on error source and opening in in appropriate editor. Feature almost never used.</div><div class="gmail_default" style="font-family:tahoma,sans-serif"><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif">Adding "return" just behind</div><div class="gmail_default" style="font-family:tahoma,sans-serif"><br></div><div class="gmail_default" style><font face="tahoma, sans-serif">proc xth_cp_show_errors {} {</font><br></div><div class="gmail_default" style><font face="tahoma, sans-serif"><br></font></div><div class="gmail_default" style><font face="tahoma, sans-serif">in cp_procs.tcl should work.</font></div><div class="gmail_default" style><font face="tahoma, sans-serif"><br></font></div><div class="gmail_default" style><font face="tahoma, sans-serif">S.</font></div><div class="gmail_default" style="font-family:tahoma,sans-serif"><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif"><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On 12 January 2015 at 03:14, Wookey <span dir="ltr"><<a href="mailto:wookey@wookware.org" target="_blank">wookey@wookware.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">+++ Olly Betts [2014-09-14 11:28 -0300]:<br>
<div><div class="h5">> # Having to wait 10 minutes counts as a major effect on usability<br>
> Control: severity -1 important<br>
><br>
> On Sat, Jun 14, 2014 at 12:16:27PM +0100, Olly Betts wrote:<br>
> > On Fri, Jun 13, 2014 at 11:15:26AM +0100, Wookey wrote:<br>
> > > +++ Jenny Black [2014-06-11 19:12 +1200]:<br>
> > > > Olly has just had a look (thank you Olly!), and in xtherion he changed:<br>
> > > > set rx {\S*[^\]\s]\s+\[\d+\]}<br>
> > > > to:<br>
> > > > set rx {\s\[\d+\]}<br>
> > > > It now no longer hangs, and runs as well as ever. We're not sure<br>
> > > > what the regular expression search was for, or whether changing it<br>
> > > > matters. As far as I can see it doesn't, as the pdf and log files<br>
> > > > have already been made.<br>
> > ><br>
> > > Just noticed this. Well fettled. I've filed a bug in Debian BTS about<br>
> > > this so it doesn't get forgotten.<br>
> > ><br>
> > > I'll do a 5.3.15 upload with this and ol's other fixes in very soon.<br>
> ><br>
> > I'm not at all confident this fix is correct - my grasp of Tcl is<br>
> > insufficient to understand what the code is actually doing with this<br>
> > regex.<br>
> ><br>
> > All I did was to prune the bit off the start of the pattern which was<br>
> > causing the regex engine to do insane amount of backtracking (which<br>
> > with a large log file could easily result in it taking 10 minutes to<br>
> > run this regex over the whole log).<br>
> ><br>
> > So you might want to wait for comments from Stacho or Martin before<br>
> > shipping this.<br>
><br>
> After staring some more, I'm fairly sure my patch doesn't actually<br>
> constitute a fix (it was never really intended to - I just meant it to<br>
> show what was taking all the time).<br>
><br>
> But it would be good to fix this before the Debian release freeze, so in<br>
> the absence of any clues as to what this code is trying to do (as far as<br>
> I can make out so far, it seems to be parsing a log to get a list of<br>
> errors to show), I'll sit down with the tcl docs and figure it out.<br>
<br>
</div></div>This is the only remaining (filed) therion bug (in Debian). Some<br>
comment from the authors would be helpful on the right way to fix this,<br>
and from users on whether it still exists in the current releas.<br>
<br>
I'm not sure whether to apply Ol's attached patch at least for the<br>
time being or whether that will actually break something (as well as<br>
speeding it up). Currently it's not applied.<br>
<span class="HOEnZb"><font color="#888888"><br>
Wookey<br>
--<br>
Principal hats:  Linaro, Debian, Wookware, ARM<br>
<a href="http://wookware.org/" target="_blank">http://wookware.org/</a><br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
Therion mailing list<br>
<a href="mailto:Therion@speleo.sk">Therion@speleo.sk</a><br>
<a href="http://mailman.speleo.sk/mailman/listinfo/therion" target="_blank">http://mailman.speleo.sk/mailman/listinfo/therion</a><br>
</div></div></blockquote></div><br></div>