Mirrors/ESP32-BLE-Gamepad
2
0
Fork 0
mirror of https://github.com/lemmingDev/ESP32-BLE-Gamepad.git synced 2026-02-20 01:21:26 +01:00
Code Issues Packages Projects Releases Wiki Activity

Update Fightstick.ino

Browse Source 
Fixed logic bug in button state update. Also converts for loops to memcpy built-in to match memcmp usage
This commit is contained in:
tachikoma0023
2025-10-21 19:33:15 -05:00
committed by GitHub
parent 1a3d3ea06a
commit 8500df4764
1 changed files with 7 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
examples/Fightstick/Fightstick.ino
View File

@@ -124,23 +124,19 @@ void loop()
bleGamepad.setHats(hatValues[0], hatValues[1], hatValues[2], hatValues[3]);
// Update previous states to current states and send report
// Uses memcmp and memcpy to efficiently analyze/copy bytes in chunks
// readable, but with compiler warning:
// if (currentButtonStates != previousButtonStates || currentHatStates != previousHatStates)
if ((memcmp((const void *)currentButtonStates, (const void *)previousButtonStates, sizeof(currentButtonStates)) != 0) && (memcmp((const void *)currentHatStates, (const void *)previousHatStates, sizeof(currentHatStates)) != 0))
// previousButtonStates = currentButtonStates;
// previousHatStates = currentHatStates;
if ((memcmp((const void *)currentButtonStates, (const void *)previousButtonStates, sizeof(currentButtonStates)) != 0) || (memcmp((const void *)currentHatStates, (const void *)previousHatStates, sizeof(currentHatStates)) != 0))
{
for (byte currentIndex = 0; currentIndex < numOfButtons; currentIndex++)
{
previousButtonStates[currentIndex] = currentButtonStates[currentIndex];
}
for (byte currentIndex = 0; currentIndex < numOfHats * 4; currentIndex++)
{
previousHatStates[currentIndex] = currentHatStates[currentIndex];
}
memcpy((void *)previousButtonStates, (void *)currentButtonStates, sizeof(currentButtonStates));
memcpy((void *)previousHatStates, (void *)currentHatStates, sizeof(currentHatStates));
bleGamepad.sendReport(); // Send a report if any of the button states or hat directions have changed
}
delay(10); // Reduce for less latency
}
}