Merge pull request #9 from dmadison/autosend

Autosend

Author: dmadison

examples/Blink/Blink.ino

@@ -36,10 +36,8 @@ void setup() {
 
 void loop() {
 	XInput.press(BUTTON_A);
-	XInput.send();
 	delay(1000);
 
 	XInput.release(BUTTON_A);
-	XInput.send();
 	delay(1000);
 }

examples/GamepadPins/GamepadPins.ino

@@ -116,6 +116,8 @@ void setup() {
 
 	// Set joystick range to the ADC max
 	XInput.setJoystickRange(0, ADC_Max);
+	
+	XInput.setAutoSend(false);  // Wait for all controls before sending
 }
 
 void loop() {

examples/ReceiveCallback/ReceiveCallback.ino

@@ -55,8 +55,6 @@ void loop() {
 	else {
 		XInput.release(TRIGGER_RIGHT);
 	}
-
-	XInput.send();
 }
 
 void rumbleCallback(uint8_t packetType) {

examples/SimulateAll/SimulateAll.ino

@@ -74,6 +74,7 @@ double angle = 0.0;
 
 void setup() {
 	pinMode(SafetyPin, INPUT_PULLUP);
+	XInput.setAutoSend(false);  // Wait for all controls before sending
 }
 
 void loop() {

examples/WiiClassicController/WiiClassicController.ino

@@ -41,6 +41,8 @@ void setup() {
 	XInput.setRange(JOY_LEFT, 8, 56);
 	XInput.setRange(JOY_RIGHT, 4, 28);
 
+	XInput.setAutoSend(false);  // Wait for all controls before sending
+
 	while (!classic.connect()) {
 		delay(1000);  // Controller not connected
 	}

keywords.txt

@@ -32,6 +32,8 @@ setJoystick	KEYWORD2
 
 releaseAll	KEYWORD2
 
+setAutoSend	KEYWORD2
+
 # Read Control Data
 getButton	KEYWORD2
 getDpad	KEYWORD2

src/XInput.cpp

@@ -219,6 +219,7 @@ void XInputGamepad::setButton(XInputControl button, boolean state) {
 		if (state) { tx[buttonData->index] |= buttonData->mask; }  // Press
 		else { tx[buttonData->index] &= ~(buttonData->mask); }  // Release
 		newData = true;
+		autosend();
 	}
 	else {
 		Range * triggerRange = getRangeFromEnum(button);
@@ -236,10 +237,16 @@ void XInputGamepad::setDpad(boolean up, boolean down, boolean left, boolean righ
 	if (up && down) { down = false; }  // Up + Down = Up
 	if (left && right) { left = false; right = false; }  // Left + Right = Neutral
 
+	const boolean autoSendTemp = autoSendOption;  // Save autosend state
+	autoSendOption = false;  // Disable temporarily
+
 	setDpad(DPAD_UP, up);
 	setDpad(DPAD_DOWN, down);
 	setDpad(DPAD_LEFT, left);
 	setDpad(DPAD_RIGHT, right);
+
+	autoSendOption = autoSendTemp;  // Re-enable from option
+	autosend();
 }
 
 void XInputGamepad::setTrigger(XInputControl trigger, int32_t val) {
@@ -251,6 +258,7 @@ void XInputGamepad::setTrigger(XInputControl trigger, int32_t val) {
 
 	tx[triggerData->index] = val;
 	newData = true;
+	autosend();
 }
 
 void XInputGamepad::setJoystick(XInputControl joy, int32_t x, int32_t y) {
@@ -269,12 +277,18 @@ void XInputGamepad::setJoystick(XInputControl joy, int32_t x, int32_t y) {
 	tx[joyData->y_high] = highByte(y);
 
 	newData = true;
+	autosend();
 }
 
 void XInputGamepad::releaseAll() {
 	const uint8_t offset = 2;  // Skip message type and packet size
 	memset(tx + offset, 0x00, sizeof(tx) - offset);  // Clear TX array
 	newData = true;  // Data changed and is unsent
+	autosend();
+}
+
+void XInputGamepad::setAutoSend(boolean a) {
+	autoSendOption = a;
 }
 
 boolean XInputGamepad::getButton(XInputControl button) const {
@@ -467,8 +481,9 @@ void XInputGamepad::reset() {
 	setTriggerRange(XInputMap_Trigger::range.min, XInputMap_Trigger::range.max);
 	setJoystickRange(XInputMap_Joystick::range.min, XInputMap_Joystick::range.max);
 
-	// Clear user-set receive callback
+	// Clear user-set options
 	recvCallback = nullptr;
+	autoSendOption = true;
 }
 
 void XInputGamepad::printDebug(Print &output) const {

src/XInput.h

@@ -92,6 +92,9 @@ class XInputGamepad {
 
 	void releaseAll();
 
+	// Auto-Send Data
+	void setAutoSend(boolean a);
+
 	// Read Control Surfaces
 	boolean getButton(XInputControl button) const;
 	boolean getDpad(XInputControl dpad) const;
@@ -134,6 +137,11 @@ class XInputGamepad {
 	// Sent Data
 	uint8_t tx[20];  // USB transmit data
 	boolean newData;  // Flag for tx data changed
+	boolean autoSendOption;  // Flag for automatically sending data
+
+	void inline autosend() {
+		if (autoSendOption) { send(); }
+	}
 
 	// Received Data
 	volatile uint8_t player;  // Gamepad player #, buffered