Use better name for variable

This commit is contained in:
Allen Hill
2026-05-15 22:52:13 +00:00
parent 87f0ddc2d4
commit 83271ef0d2
+11 -11
View File
@@ -107,7 +107,7 @@ int main() {
// Binary-mode REPL includes the full wire preamble (broadcast + 2*addr + // Binary-mode REPL includes the full wire preamble (broadcast + 2*addr +
// control + length), so size for the worst case. // control + length), so size for the worst case.
uint8_t data_tmp[MAXMSGLEN + sizeof(AVCLAN_frame_t)]; uint8_t data_tmp[MAXMSGLEN + sizeof(AVCLAN_frame_t)];
uint8_t seqLen = 0; // current length written to data_tmp uint8_t seqIdx = 0; // current index in data_tmp
uint8_t err = 0; uint8_t err = 0;
@@ -272,7 +272,7 @@ int main() {
case 0x10: // Signals binary sequence incoming case 0x10: // Signals binary sequence incoming
if (!readSeq && !readBinary) { if (!readSeq && !readBinary) {
readSeq = readBinary = 1; readSeq = readBinary = 1;
seqLen = 0; seqIdx = 0;
break; break;
} else } else
goto DEFAULT; // reading binary and this is a real data byte goto DEFAULT; // reading binary and this is a real data byte
@@ -282,7 +282,7 @@ int main() {
lastPrintAllFrames = printAllFrames; lastPrintAllFrames = printAllFrames;
printAllFrames = 0; printAllFrames = 0;
readSeq = 1; readSeq = 1;
seqLen = hexDigit = 0; seqIdx = hexDigit = 0;
hexChars[0] = hexChars[1] = 0; hexChars[0] = hexChars[1] = 0;
seqIsUnicast = true; seqIsUnicast = true;
break; break;
@@ -291,16 +291,16 @@ int main() {
lastPrintAllFrames = printAllFrames; lastPrintAllFrames = printAllFrames;
printAllFrames = 0; printAllFrames = 0;
readSeq = 1; readSeq = 1;
seqLen = hexDigit = 0; seqIdx = hexDigit = 0;
hexChars[0] = hexChars[1] = 0; hexChars[0] = hexChars[1] = 0;
seqIsUnicast = false; seqIsUnicast = false;
break; break;
case '\n': case '\n':
if (readSeq) { if (readSeq) {
if (readBinary) { if (readBinary) {
if (data_tmp[seqLen] == 0x17) { if (data_tmp[seqIdx] == 0x17) {
if (AVCLAN_frame_t *out = (AVCLAN_frame_t *)popQueue(&cache)) { if (AVCLAN_frame_t *out = (AVCLAN_frame_t *)popQueue(&cache)) {
if (!AVCLAN_parseframe(data_tmp, --seqLen, out)) { if (!AVCLAN_parseframe(data_tmp, --seqIdx, out)) {
if (RFrame_t *resp = popQueue(&rcache)) { if (RFrame_t *resp = popQueue(&rcache)) {
*resp = (RFrame_t){.r = r_Handled, .frame = out}; *resp = (RFrame_t){.r = r_Handled, .frame = out};
push_or_return_resp(resp); push_or_return_resp(resp);
@@ -319,8 +319,8 @@ int main() {
out->is_unicast = seqIsUnicast; out->is_unicast = seqIsUnicast;
out->controller_addr = DEVICE_ADDR; out->controller_addr = DEVICE_ADDR;
out->peripheral_addr = seqIsUnicast ? HU_ADDR : 0x1FF; out->peripheral_addr = seqIsUnicast ? HU_ADDR : 0x1FF;
out->length = seqLen; out->length = seqIdx;
memcpy(out->data, data_tmp, seqLen); memcpy(out->data, data_tmp, seqIdx);
*resp = (RFrame_t){.r = r_Handled, .frame = out}; *resp = (RFrame_t){.r = r_Handled, .frame = out};
push_or_return_resp(resp); push_or_return_resp(resp);
} else } else
@@ -334,7 +334,7 @@ int main() {
default: default:
if (readSeq) { if (readSeq) {
if (readBinary) { if (readBinary) {
data_tmp[seqLen++] = readkey; data_tmp[seqIdx++] = readkey;
} else { } else {
hexChars[hexDigit++] = readkey; hexChars[hexDigit++] = readkey;
@@ -346,12 +346,12 @@ int main() {
l = toupper(hexChars[1]); l = toupper(hexChars[1]);
l += (l < ':') ? 0xd0 : 0xc9; l += (l < ':') ? 0xd0 : 0xc9;
data_tmp[seqLen++] = (h << 4) | l; data_tmp[seqIdx++] = (h << 4) | l;
hexDigit = hexChars[0] = hexChars[1] = 0; hexDigit = hexChars[0] = hexChars[1] = 0;
} }
if (echoCharacters) { if (echoCharacters) {
RS232_Print("CURRENT SEQUENCE > "); RS232_Print("CURRENT SEQUENCE > ");
for (uint8_t i = 0; i < seqLen; i++) { for (uint8_t i = 0; i < seqIdx; i++) {
RS232_PrintHex8(data_tmp[i]); RS232_PrintHex8(data_tmp[i]);
RS232_SendByte(' '); RS232_SendByte(' ');
} }