Merge pull request #165 from rhpvorderman/pedanticcompile2

Heed clang compile warnings.
rhpvorderman · May 27, 2024 · 2751024 · 2751024
2 parents 0079498 + 0861513
commit 2751024
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/src/sequali/function_dispatch.h b/src/sequali/function_dispatch.h
@@ -206,7 +206,7 @@ static int64_t sequence_to_canonical_kmer_default(uint8_t *sequence, uint64_t k)
         kmer <<= 8;
         kmer |= kchunk;
     }
-    for (i=i; i<(int64_t)k; i++) {
+    for (; i<(int64_t)k; i++) {
         size_t nuc = NUCLEOTIDE_TO_TWOBIT[sequence[i]];
         all_nucs |= nuc;
         kmer <<= 2;
@@ -378,4 +378,4 @@ static int64_t (*sequence_to_canonical_kmer)(
 static inline int64_t sequence_to_canonical_kmer(uint8_t *sequence, uint64_t k) {
     return sequence_to_canonical_kmer_default(sequence, k);
 }
-#endif
+#endif
diff --git a/tox.ini b/tox.ini
@@ -35,8 +35,10 @@ deps=
 skip_install=True
 setenv=
     CFLAGS=-Wall -Werror -Wpedantic
+allowlist_externals = bash
 commands =
-    python setup.py build_ext -f
+    bash -c 'CC=gcc python setup.py build_ext -f'
+    bash -c 'CC=clang python setup.py build_ext -f'
 
 [flake8]
 max-line-length=88