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)



Subject: lang selection + suggestions + single Text::Aspell object
Download (untitled) / with headers
text/plain 580b
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.

Download (untitled) / with headers
text/plain 645b
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


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.