Skip Menu |

This queue is for tickets about the Text-SpellChecker CPAN distribution.

Report information
The Basics
Id: 41685
Status: resolved
Priority: 0/
Queue: Text-SpellChecker
People
Owner: Nobody in particular
Requestors: yanick+cpan [...] babyl.dyndns.org
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: 0.03
Fixed in: (no value)
Attachments
patch

History Show all quoted text
  Sun Dec 14 00:22:24 2008 yanick+cpan [...] babyl.dyndns.org - Ticket created
Subject: lang selection + suggestions + single Text::Aspell object
Hi! I've reviewed this mode for the Perl Advent Calendar (http://perladvent.pm.org/2008/13/), and ended up tweaking it a little bit. Attached is the resulting patch that contains 3 fixes/improvements: 1. Added the possibility to pass a 'lang' attribute to the object creation, which will then be passed to the underlaying Text::Aspell object. 2. Cache the Text::Aspell object instead of creating it for each new word. 3. Modified suggestions() so that it returns the array directly if wantarray() swings that way. Please feel free to use 'em or toss 'em as you see fit. :-)
Subject: patch
Download patch
application/octet-stream 3.4k

Message body not shown because it is not plain text.

  Sun Dec 14 17:02:45 2008 bduggan [...] matatu.org - Correspondence added
On Sun Dec 14 00:22:24 2008, YANICK wrote: Show quoted text
> 1. Added the possibility to pass a 'lang' attribute to the object > creation, which will then be passed to the underlaying Text::Aspell
object. This can also be set in an aspell config file or in an environment variable (e.g. $ENV{ASPELL_CONF} = 'lang fr'). Show quoted text
> > 2. Cache the Text::Aspell object instead of creating it for each new word. > > 3. Modified suggestions() so that it returns the array directly if > wantarray() swings that way. > > Please feel free to use 'em or toss 'em as you see fit. :-)
Great, thanks. I've applied all three, and version 0.04 is on its way to CPAN now. Brian
  Sun Dec 14 17:02:45 2008 The RT System itself - Status changed from 'new' to 'open'
  Sun Dec 14 17:02:46 2008 bduggan [...] matatu.org - Status changed from 'open' to 'resolved'