[Therion] Bug#751471: Therion hanging after compile

[Stacho Mudrak]

I am sorry, I was never able replicate this bug.

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.

Adding "return" just behind

proc xth_cp_show_errors {} {

in cp_procs.tcl should work.

S.



On 12 January 2015 at 03:14, Wookey <wookey at wookware.org> wrote:

> +++ Olly Betts [2014-09-14 11:28 -0300]:
> > # Having to wait 10 minutes counts as a major effect on usability
> > Control: severity -1 important
> >
> > On Sat, Jun 14, 2014 at 12:16:27PM +0100, Olly Betts wrote:
> > > On Fri, Jun 13, 2014 at 11:15:26AM +0100, Wookey wrote:
> > > > +++ Jenny Black [2014-06-11 19:12 +1200]:
> > > > > Olly has just had a look (thank you Olly!), and in xtherion he
> changed:
> > > > > set rx {\S*[^\]\s]\s+\[\d+\]}
> > > > > to:
> > > > > set rx {\s\[\d+\]}
> > > > > It now no longer hangs, and runs as well as ever. We're not sure
> > > > > what the regular expression search was for, or whether changing it
> > > > > matters. As far as I can see it doesn't, as the pdf and log files
> > > > > have already been made.
> > > >
> > > > Just noticed this. Well fettled. I've filed a bug in Debian BTS about
> > > > this so it doesn't get forgotten.
> > > >
> > > > I'll do a 5.3.15 upload with this and ol's other fixes in very soon.
> > >
> > > I'm not at all confident this fix is correct - my grasp of Tcl is
> > > insufficient to understand what the code is actually doing with this
> > > regex.
> > >
> > > All I did was to prune the bit off the start of the pattern which was
> > > causing the regex engine to do insane amount of backtracking (which
> > > with a large log file could easily result in it taking 10 minutes to
> > > run this regex over the whole log).
> > >
> > > So you might want to wait for comments from Stacho or Martin before
> > > shipping this.
> >
> > After staring some more, I'm fairly sure my patch doesn't actually
> > constitute a fix (it was never really intended to - I just meant it to
> > show what was taking all the time).
> >
> > But it would be good to fix this before the Debian release freeze, so in
> > the absence of any clues as to what this code is trying to do (as far as
> > I can make out so far, it seems to be parsing a log to get a list of
> > errors to show), I'll sit down with the tcl docs and figure it out.
>
> This is the only remaining (filed) therion bug (in Debian). Some
> comment from the authors would be helpful on the right way to fix this,
> and from users on whether it still exists in the current releas.
>
> I'm not sure whether to apply Ol's attached patch at least for the
> time being or whether that will actually break something (as well as
> speeding it up). Currently it's not applied.
>
> Wookey
> --
> Principal hats:  Linaro, Debian, Wookware, ARM
> http://wookware.org/
> _______________________________________________
> Therion mailing list
> Therion at speleo.sk
> http://mailman.speleo.sk/mailman/listinfo/therion
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.speleo.sk/pipermail/therion/attachments/20150112/797623aa/attachment.htm>