Skip to content

Commit 1f7ff1c

Browse files
committed
fix: make pos size_t
1 parent 2022894 commit 1f7ff1c

File tree

1 file changed

+8
-7
lines changed

1 file changed

+8
-7
lines changed

src/matcher.cc

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ std::vector<size_t> computeMatch(const CandidateString &subject, const Candidate
4040

4141
// Traceback matrix
4242
auto trace = std::vector<Direction>(subject_size * query_size, Direction::STOP);
43-
auto pos = -1;
43+
auto pos = 0u;
4444

4545
auto i = 0u;
4646
while (i < subject_size) {//foreach char is of subject
@@ -95,10 +95,11 @@ std::vector<size_t> computeMatch(const CandidateString &subject, const Candidate
9595
score_row[j] = score;
9696
csc_row[j] = csc_score;
9797

98-
++pos;
9998
assert(0u <= pos && pos < trace.size());
10099
trace[pos] = score > 0 ? move : Direction::STOP;
101100

101+
++pos;
102+
102103
++j;
103104
}
104105
assert(0u <= j && j <= query_size);
@@ -115,25 +116,25 @@ std::vector<size_t> computeMatch(const CandidateString &subject, const Candidate
115116

116117
int ii = subject_size - 1;
117118
int jj = query_size - 1;
118-
pos = ii * query_size + jj;
119+
int pos_ = ii * query_size + jj;
119120
auto backtrack = true;
120121
std::vector<size_t> matches;
121122

122123
while (backtrack && ii >= 0 && jj >= 0) {
123-
switch (trace[pos]) {
124+
switch (trace[pos_]) {
124125
case Direction::UP:
125126
ii--;
126-
pos -= query_size;
127+
pos_ -= query_size;
127128
break;
128129
case Direction::LEFT:
129130
jj--;
130-
pos--;
131+
pos_--;
131132
break;
132133
case Direction::DIAGONAL:
133134
matches.emplace_back(ii + offset);
134135
jj--;
135136
ii--;
136-
pos -= query_size + 1;
137+
pos_ -= query_size + 1;
137138
break;
138139
default:
139140
backtrack = false;

0 commit comments

Comments
 (0)