Skip to content

Commit

Permalink
Merge pull request #3332 from eseiler/fix/codeql
Browse files Browse the repository at this point in the history
[FIX] CodeQL warnings
  • Loading branch information
eseiler authored Jan 27, 2025
2 parents 735763b + b4c2655 commit 666b876
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions test/unit/search/search_scheme_algorithm_test.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,7 @@ inline void test_search_hamming(auto index,
while (std::adjacent_find(error_positions.begin(), error_positions.end()) != error_positions.end());

// Construct query sequence with chosen error positions.
for (uint8_t error = 0; error < error_positions.size(); ++error)
for (size_t error = 0; error < error_positions.size(); ++error)
{
uint64_t const pos = error_positions[error] + current_blocks_length;
// Decrease alphabet size by one because we don't want to replace query[pos], with the same character.
Expand Down Expand Up @@ -206,7 +206,7 @@ test_search_scheme_hamming(search_scheme_t const & search_scheme, size_t const s

// Calculate all error distributions and sort each of them (from left to right).
uint8_t max_error = 0;
for (uint8_t search_id = 0; search_id < search_scheme.size(); ++search_id)
for (size_t search_id = 0; search_id < search_scheme.size(); ++search_id)
{
ordered_search_scheme[search_id] = search_scheme[search_id];
seqan3::search_error_distribution(error_distributions[search_id], search_scheme[search_id]);
Expand All @@ -228,7 +228,7 @@ test_search_scheme_hamming(search_scheme_t const & search_scheme, size_t const s
for (uint64_t query_length = query_length_min; query_length < query_length_max; ++query_length)
{
auto const block_info = search_scheme_block_info(search_scheme, query_length);
for (uint8_t search_id = 0; search_id < search_scheme.size(); ++search_id)
for (size_t search_id = 0; search_id < search_scheme.size(); ++search_id)
{
auto const & [blocks_length, start_pos] = block_info[search_id];

Expand Down

0 comments on commit 666b876

Please sign in to comment.