Merge pull request #35 from DCSFlightpanels/InvertableLeds

Add reverse parameter to LED (from No1sonuk)

Author: talbotmcinnis

examples/OneOfEverything/OneOfEverything.ino

@@ -77,14 +77,22 @@ const int ANALOG_OUT_PIN_A = 16;
 // A single LED
 DcsBios::Dimmer defaultDimmerExample(0x1012, OUT_PIN_A);
 DcsBios::LED masterCaution(0x1012, 0x0800, OUT_PIN_A);
+
+DcsBios::LED invertedLedExample(0x1012, 0x0800, OUT_PIN_A, true);
+
 // An analog output with a value that comes from a DCS address
 DcsBios::Dimmer dimmerExample(0x1012, ANALOG_OUT_PIN_A);
+
+// An inverted demo that also demonstrates a custom range.  By the min (200) being greater than max (0), the inverted behaviour is created.
 DcsBios::Dimmer invertedDimmerExample(0x1012, ANALOG_OUT_PIN_A, 200, 0);
+
+// A dimmer where the user provided a mapping function.  In this case a simple modulus stepper, but could include non-linear functions, conditions, etc.
 unsigned int myValueMapper(unsigned int dcsValue)
 {
   return dcsValue % 10;
 }
 DcsBios::Dimmer mappedDimmerExample(0x1012, ANALOG_OUT_PIN_A, myValueMapper);
+
 // A servo motor controlleed from DCS, i.e. a guage.
 DcsBios::ServoOutput servoExample(0x1012, ANALOG_OUT_PIN_A, 544, 2400);
 

releasenotes.md

@@ -3,6 +3,7 @@
 - Fix Switch2Pos state reset logic, and improve it's internal debounce logic for a few edge cases.
 - Add DualModeButton for first multi-seat support.  Originally developed for use in Apache cockpits where one switch toggles the function of a physical button between front and back seat DCS control.
 - Included AerialElectron's Concentric Rotary Encoder Emulator.  Useful for simpits with a rotary that has a pushbutton, to emulate controlling two different dcs cockpit controls, especially if the dcs controls are dual-ring concentric rotaries.  Thanks for the contribution AerialElectron!
+- Integrated No1sonuk's inverted LED support.  LEDs now take an optional parameter which can be used to have an SimPit LED operate opposite to a DcsPit LED.
 
 ## v0.3.7
 - Integrate 2 and 3 position Matrix switches, thanks to Dehuman for the starting point!

src/internal/Leds.h

@@ -10,17 +10,17 @@ namespace DcsBios {
 		private:
 			unsigned int mask;
 			unsigned char pin;
+			bool reverse;
 		public:
-			LED(unsigned int address, unsigned int mask, char pin) : Int16Buffer(address), mask(mask), pin(pin) {
+			LED(unsigned int address, unsigned int mask, char pin, bool reverse = false) : Int16Buffer(address), mask(mask), pin(pin), reverse(reverse) {
 				pinMode(pin, OUTPUT);
 			}
 			virtual void loop() {
 				if (hasUpdatedData()) {
-					if (getData() & mask) {
-						digitalWrite(pin, 1);
-					} else {
-						digitalWrite(pin, 0);
-					}
+					bool state = getData() & mask;
+					if (reverse) state = !state;
+					
+					digitalWrite(pin, state);					
 				}
 			}
 	};