search

Ensembl / Bio-DB-HTS

Git repo for Bio::DB::HTS module on CPAN, providing Perl links into HTSlib
Apache License 2.0
24 stars 16 forks source link

Make sure to free the string #18

Closed andrewyatz closed 8 years ago

andrewyatz commented 8 years ago

The string that was returned from htslib needs to be freed otherwise it causes a leak.

andrewyatz commented 8 years ago

Sorry there's been a few commits on here. The first is the main bit. The others are making sure the Tabix perl objects cleanup after themselves. Testing on a Linux server the process' memory usage is flat.

rishidev commented 8 years ago

Thanks Andy

andrewyatz commented 8 years ago

I guess now we just need to make a release 😉 .

rishidev commented 8 years ago

A release is on it's way...

keiranmraine commented 8 years ago

Thanks to both of you... perhaps it's time we had a CGP-IT/Ensembl dev pub lunch

keiranmraine commented 8 years ago

Loop back to #17, huge warning stack on use