Skip Menu |
 

This queue is for tickets about the Search-WuManber CPAN distribution.

Report information
The Basics
Id: 120299
Status: new
Priority: 0/
Queue: Search-WuManber

People
Owner: Nobody in particular
Requestors: bkb [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: Suggest making struct WuManber into an object
MIME-Version: 1.0
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
Message-ID: <rt-4.0.18-11936-1487558015-1685.0-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 178
Download (untitled) / with headers
text/plain 178b
Looking at the code, there are quite a few memory leaks. I'd like to suggest changing the struct WuManber into a Search::WuManber object and handle the hash outside the XS code.
MIME-Version: 1.0
In-Reply-To: <rt-4.0.18-11936-1487558015-1685.0-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
References: <rt-4.0.18-11936-1487558015-1685.0-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf-8"
Message-ID: <rt-4.0.18-31164-1487562895-1073.120299-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 381
Download (untitled) / with headers
text/plain 381b
On Sun Feb 19 21:33:35 2017, BKB wrote: Show quoted text
> Looking at the code, there are quite a few memory leaks. I'd like to > suggest changing the struct WuManber into a Search::WuManber object > and handle the hash outside the XS code.
I've gone ahead and done these changes in a git repository here: https://github.com/benkasminbullock/search-wumanber/commits/master I hope this is useful.


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.