Re^2: A graphical regular expression tester.

Weird, your example tested fine until I clicked one of the switches, then got a similar error to you. However, I noticed the error didn't occur when clicking the checkbox for 'g'. The way subroutine call for this event is handled differently, so I swapped the following....


< $check_switch_i->bind('<Button-1>', \&button_run('now'));
< $check_switch_m->bind('<Button-1>', \&button_run('now'));
< $check_switch_s->bind('<Button-1>', \&button_run('now'));

> $check_switch_i->bind('<Button-1>', sub { button_run('now') });
> $check_switch_m->bind('<Button-1>', sub { button_run('now') });
> $check_switch_s->bind('<Button-1>', sub { button_run('now') });
[download]

...and the bug has gone. I've no idea why though. Anyone?

Cheers, Rob

---
my name's not Keith, and I'm not reasonable.

Comment on Re^2: A graphical regular expression tester. Download Code

Replies are listed 'Best First'.
Re^3: A graphical regular expression tester. by liverpole (Monsignor) on Dec 07, 2006 at 19:00 UTC
Hi reasonablekeith, Yes, it's because you're not doing what you think you're doing with: `$check_switch_i->bind('<Button-1>', \&button_run('now'));` [download] If you run `perl -MO=Deparse` on that, it'll show you: `$check_switch_i->bind('<Button-1>', \(&button_run('now')));` [download] Which means that you're calling `bind` with a reference to the results of a call to the subroutine `button_run`. What you want, instead (if you're going to use that style of notation), is the anonymous list syntax: `$check_switch_i->bind('<Button-1>', [ \&button_run, 'now' ]);` [download] s''(q.S:$/9=(T1';s;(..)(..);$..=substr+crypt($1,$2),2,3;eg;print$..$/	[reply] [d/l] [select]

Replies are listed 'Best First'.

Re^3: A graphical regular expression tester.
by liverpole (Monsignor) on Dec 07, 2006 at 19:00 UTC

reasonablekeith

Yes, it's because you're not doing what you think you're doing with:

$check_switch_i->bind('<Button-1>', \&button_run('now'));
[download]

If you run perl -MO=Deparse on that, it'll show you:

$check_switch_i->bind('<Button-1>', \(&button_run('now')));
[download]

Which means that you're calling bind with a reference to the results of a call to the subroutine button_run.

What you want, instead (if you're going to use that style of notation), is the anonymous list syntax:

$check_switch_i->bind('<Button-1>', [ \&button_run, 'now' ]);
[download]

s''(q.S:$/9=(T1';s;(..)(..);$..=substr+crypt($1,$2),2,3;eg;print$..$/

[reply]
[d/l]
[select]