Fix unsorted AF logic

Include current frequency if AF sort is disabled, but keep other conditions separate, to avoid repeated AF list with 0.0 values
This commit is contained in:
Adam Wisher
2023-11-19 20:53:52 +00:00
committed by GitHub
parent 7811cf3c08
commit 95bb3d3e03

View File

@@ -719,7 +719,7 @@ void TEF6686::readRDS(byte showrdserrors)
bool isValuePresent = false;
for (int i = 0; i < 50; i++) { // Check if already in list
if (rds.sortaf && ((buffer0 == currentfreq) || buffer0 == 0 || af[i].frequency == buffer0)) {
if ((rds.sortaf && (buffer0 == currentfreq)) || buffer0 == 0 || af[i].frequency == buffer0) {
isValuePresent = true;
break;
}
@@ -733,7 +733,7 @@ void TEF6686::readRDS(byte showrdserrors)
isValuePresent = false;
for (int i = 0; i < 50; i++) { // Check if already in list
if (rds.sortaf && ((buffer1 == currentfreq) || buffer1 == 0 || af[i].frequency == buffer1)) {
if ((rds.sortaf && (buffer1 == currentfreq)) || buffer1 == 0 || af[i].frequency == buffer1) {
isValuePresent = true;
break;
}
@@ -1835,4 +1835,4 @@ void TEF6686::RDScharConverter(const char* input, wchar_t* output, size_t size,
}
}
output[size - 1] = L'\0';
}
}