Skip to content

Commit 509ee95

Browse files
committed
IntegerMatcher: Fix data type of loop counters
ClassTemplate->ProtoLengths[n] is of type uint8_t, so use that for the related loop counters, too. Signed-off-by: Stefan Weil <sw@weilnetz.de>
1 parent f4f34a8 commit 509ee95

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

src/classify/intmatcher.cpp

+5-5
Original file line numberDiff line numberDiff line change
@@ -621,7 +621,7 @@ int IntegerMatcher::FindGoodProtos(
621621
for (int proto = 0; proto < ClassTemplate->NumProtos; proto++) {
622622
/* Compute Average for Actual Proto */
623623
int Temp = 0;
624-
for (int i = 0; i < ClassTemplate->ProtoLengths[proto]; i++)
624+
for (uint8_t i = 0; i < ClassTemplate->ProtoLengths[proto]; i++)
625625
Temp += tables->proto_evidence_[proto][i];
626626

627627
Temp /= ClassTemplate->ProtoLengths[proto];
@@ -894,7 +894,7 @@ int IntegerMatcher::UpdateTablesForFeature(
894894

895895
uint8_t* UINT8Pointer =
896896
&(tables->proto_evidence_[ActualProtoNum + proto_offset][0]);
897-
for (int ProtoIndex =
897+
for (uint8_t ProtoIndex =
898898
ClassTemplate->ProtoLengths[ActualProtoNum + proto_offset];
899899
ProtoIndex > 0; ProtoIndex--, UINT8Pointer++) {
900900
if (Evidence > *UINT8Pointer) {
@@ -997,7 +997,7 @@ void IntegerMatcher::DebugFeatureProtoError(
997997
ProtoNum++, ActualProtoNum++) {
998998
cprintf ("P %3d =", ActualProtoNum);
999999
int temp = 0;
1000-
for (int j = 0; j < ClassTemplate->ProtoLengths[ActualProtoNum]; j++) {
1000+
for (uint8_t j = 0; j < ClassTemplate->ProtoLengths[ActualProtoNum]; j++) {
10011001
uint8_t data = tables.proto_evidence_[ActualProtoNum][j];
10021002
cprintf(" %d", data);
10031003
temp += data;
@@ -1071,7 +1071,7 @@ void IntegerMatcher::DisplayProtoDebugInfo(
10711071
ProtoNum++, ActualProtoNum++) {
10721072
/* Compute Average for Actual Proto */
10731073
int temp = 0;
1074-
for (int i = 0; i < ClassTemplate->ProtoLengths[ActualProtoNum]; i++)
1074+
for (uint8_t i = 0; i < ClassTemplate->ProtoLengths[ActualProtoNum]; i++)
10751075
temp += tables.proto_evidence_[ActualProtoNum][i];
10761076

10771077
temp /= ClassTemplate->ProtoLengths[ActualProtoNum];
@@ -1153,7 +1153,7 @@ void ScratchEvidence::UpdateSumOfProtoEvidences(
11531153
((ProtoNum < PROTOS_PER_PROTO_SET) && (ActualProtoNum < NumProtos));
11541154
ProtoNum++, ActualProtoNum++) {
11551155
int temp = 0;
1156-
for (int i = 0; i < ClassTemplate->ProtoLengths[ActualProtoNum]; i++)
1156+
for (uint8_t i = 0; i < ClassTemplate->ProtoLengths[ActualProtoNum]; i++)
11571157
temp += proto_evidence_[ActualProtoNum] [i];
11581158

11591159
ConfigWord = ProtoSet->Protos[ProtoNum].Configs[0];

0 commit comments

Comments
 (0)