smithlab_utils.hpp: in the kmer_counts function, removing variable length array to silence warnings. This function needs work.

Author: andrewdavidsmith

smithlab_utils.hpp

@@ -380,13 +380,13 @@ kmer_counts(const std::vector<std::string> &seqs,
   counts.resize(nwords, 0);
   size_t total = 0;
   for (size_t i = 0; i < seqs.size(); ++i) {
-    char seq[seqs[i].length() + 1];
-    seq[seqs[i].length()] = '\0';
-    copy(seqs[i].begin(), seqs[i].end(), seq);
+    std::vector<char> seq(seqs[i].length() + 1, '\0');
+    auto seq_data = seq.data();
+    copy(cbegin(seqs[i]), cend(seqs[i]), seq_data);
     for (size_t j = 0; j < seqs[i].length() - k + 1; ++j)
-      if (std::count_if(seq + j, seq + j + k, &valid_base) ==
+      if (std::count_if(seq_data + j, seq_data + j + k, &valid_base) ==
           static_cast<int>(k)) {
-        counts[mer2index(seq + j, k)]++;
+        counts[mer2index(seq_data + j, k)]++;
         ++total;
       }
   }