Merge pull request #155 from jaebeom-kim/master

fix memory leak

Author: jaebeom-kim

src/commons/Classifier.cpp

@@ -37,6 +37,7 @@ Classifier::~Classifier() {
     delete kmerExtractor;
     delete kmerMatcher;
     delete reporter;
+    delete geneticCode;
 }
 
 void Classifier::startClassify(const LocalParameters &par) {

src/commons/KmerMatcher.cpp

@@ -19,6 +19,7 @@ constexpr uint16_t KmerMatcher::HAMMING_LUT7[64];
 KmerMatcher::KmerMatcher(
     const LocalParameters & par,
     TaxonomyWrapper * taxonomy,
+
     int kmerFormat) 
     : par(par), 
       kmerFormat(kmerFormat) 
@@ -38,6 +39,7 @@ KmerMatcher::KmerMatcher(
 
 
 KmerMatcher::~KmerMatcher() {
+    delete geneticCode;
 }
 
 void KmerMatcher::loadTaxIdList(const LocalParameters & par) {

src/commons/KmerScanner.h

@@ -56,6 +56,10 @@ class KmerScanner {
         this->dnaMask = (1ULL << 24) - 1;    
     }
 
+    virtual ~KmerScanner() {
+        // std::cout << "KmerScanner destroyed." << std::endl;
+    }
+
     const GeneticCode &getGeneticCode() const {
         return geneticCode;
     }
@@ -119,6 +123,10 @@ class OldKmerScanner : public KmerScanner {
         std::deque<size_t> dq;
     public:
         OldKmerScanner(const GeneticCode &geneticCode) : KmerScanner(geneticCode) {}
+
+        ~OldKmerScanner() {
+            // std::cout << "OldKmerScanner destroyed." << std::endl;
+        }
 
         void initScanner(const char * seq, size_t seqStart, size_t seqEnd, bool isForward) override {
             KmerScanner::initScanner(seq, seqStart, seqEnd, isForward);