Skip Menu |

This queue is for tickets about the Tk CPAN distribution.

Report information
The Basics
Id: 51552
Status: resolved
Priority: 0/
Queue: Tk
People
Owner: Nobody in particular
Requestors: keszler [...] srkconsulting.com
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: 804.029_500

History Show all quoted text
  Sun Nov 15 05:24:01 2009 keszler [...] srkconsulting.com - Ticket created
Subject: Tk::Text bug report
Date: Sun, 15 Nov 2009 01:50:54 +0000
To: slaven [...] rezic.de
From: "Scott R. Keszler" <keszler [...] srkconsulting.com>
Full write-up: http://perlmonks.org/index.pl?node_id=807208 In short, the Tk::Text documentation for $text->FindNext says to use -forward or -reverse to control search direction. The parameter is passed by Tk/Text.pm to $text->search, which is documented to accept -forwards or -backwards. Using -reverse in FindNext produces an error. Simple fix: change Tk::Text documentation for FindNext to show -forward and -backward as the direction parameters. Full fix: change Tk/Text.pm (and the docs) to use -forwards and -backwards, matching the C for the TextSearchCmd function. -- Scott R. Keszler Consultant keszler@srkconsulting.com 701-361-8406 (cell)
  Tue Feb 02 08:29:34 2010 SREZIC [...] cpan.org - Correspondence added
On Sun Nov 15 05:24:01 2009, KESZLER wrote: Show quoted text
> Full write-up: http://perlmonks.org/index.pl?node_id=807208 > > In short, the Tk::Text documentation for $text->FindNext says to use > -forward > or -reverse to control search direction. The parameter is passed by > Tk/Text.pm > to $text->search, which is documented to accept -forwards or > -backwards. Using > -reverse in FindNext produces an error. > > Simple fix: change Tk::Text documentation for FindNext to show > -forward and > -backward as the direction parameters. > > Full fix: change Tk/Text.pm (and the docs) to use -forwards and > -backwards, > matching the C for the TextSearchCmd function. >
Thanks for the pointer. Patches in the Perl/Tk subversion repo as r13796. Regards, Slaven
  Tue Feb 02 08:29:36 2010 The RT System itself - Status changed from 'new' to 'open'
  Tue Feb 02 08:29:36 2010 SREZIC [...] cpan.org - Status changed from 'open' to 'patched'
  Tue Jun 14 02:47:01 2011 SREZIC [...] cpan.org - Status changed from 'patched' to 'resolved'
  Tue Jun 14 02:47:02 2011 SREZIC [...] cpan.org - Fixed in 804.029_500 added