Improved ir_DistanceProtocol

Author: ArminJo

README.md

@@ -123,10 +123,13 @@ For applications only requiring NEC protocol, there is a receiver which has very
 
 # Handling unknown Protocols
 ## Disclaimer
-This library was never designed to handle long codes like the ones used by air conditioners.
-See [Recording long Infrared Remote control signals with Arduino](https://www.analysir.com/blog/2014/03/19/air-conditioners-problems-recording-long-infrared-remote-control-signals-arduino).<br/>
+**This library was never designed to handle long codes like the ones used by air conditioners.**<br/>
+For air condioners [see this fork](https://github.com/crankyoldgit/IRremoteESP8266) which supports an impressive set of protocols and a lot of air conditioners and the blog entry: ["Recording long Infrared Remote control signals with Arduino"](https://www.analysir.com/blog/2014/03/19/air-conditioners-problems-recording-long-infrared-remote-control-signals-arduino).<br/>
 The main reason is, that it was designed to fit inside MCUs with relatively low levels of resources and was intended to work as a library together with other applications which also require some resources of the MCU to operate.
 
+## Protocol=PULSE_DISTANCE
+If you get something like this: `PULSE_DISTANCE: HeaderMarkMicros=8900 HeaderSpaceMicros=4450 MarkMicros=550 OneSpaceMicros=1700 ZeroSpaceMicros=600  NumberOfBits=56 0x43D8613C 0x3BC3BC`, then you have a code consisting of **56 bits**, which is probably from an air condioner remote. You can send it with calling sendPulseDistanceWidthData() twice, once for the first 32 bit and next for the remaining 24 bits.
+
 ## Protocol=UNKNOWN
 If you see something like `Protocol=UNKNOWN Hash=0x13BD886C 35 bits received` as output of e.g. the ReceiveDemo example, you either have a problem with decoding a protocol, or an unsupported protocol.
 

changelog.md

@@ -3,6 +3,7 @@ The latest version may not be released!
 See also the commit log at github: https://github.com/Arduino-IRremote/Arduino-IRremote/commits/master
 
 ## 3.4.1
+- Improved ir_DistanceProtocol.
 
 ## 3.4.0
 - Added LG2 protocol.

src/ir_DistanceProtocol.cpp

@@ -218,7 +218,7 @@ bool IRrecv::decodeDistance() {
                 return false;
             }
             if (i == 0) {
-                // Print protocol timing only once
+                // Print protocol timing and length only once
                 INFO_PRINTLN();
                 INFO_PRINT(F("PULSE_WIDTH:"));
                 INFO_PRINT(F(" HeaderMarkMicros="));
@@ -230,16 +230,18 @@ bool IRrecv::decodeDistance() {
                 INFO_PRINT(F(" ZeroMarkMicros="));
                 INFO_PRINT(tMarkTicksShort * MICROS_PER_TICK);
                 INFO_PRINT(F(" SpaceMicros="));
-                INFO_PRINTLN(tSpaceTicksShort * MICROS_PER_TICK);
-            }
-            if (tNumberOfAdditionalLong > 0) {
-                // print only if we have more than 32 bits for decode
-                INFO_PRINT(F(" 0x"));
-                INFO_PRINT(decodedIRData.decodedRawData, HEX);
-                tStartIndex += 64;
-                tNumberOfBits -= 32;
+                INFO_PRINT(tSpaceTicksShort * MICROS_PER_TICK);
+                INFO_PRINT(F(" NumberOfBits="));
+                INFO_PRINT(decodedIRData.numberOfBits);
+                INFO_PRINT(F(" DecodedRawData:"));
+
             }
+            INFO_PRINT(F(" 0x"));
+            INFO_PRINT(decodedIRData.decodedRawData, HEX);
+            tStartIndex += 64;
+            tNumberOfBits -= 32;
         }
+        INFO_PRINTLN();
 
         // Store ticks used for decoding in extra
         decodedIRData.extra = (tMarkTicksShort << 8) | tMarkTicksLong;
@@ -252,6 +254,9 @@ bool IRrecv::decodeDistance() {
 //            tNumberOfBits++;
 //        }
 
+        /*
+         * Decode in 32 bit chunks
+         */
         for (uint8_t i = 0; i <= tNumberOfAdditionalLong; ++i) {
             uint8_t tNumberOfBitsForOneDecode = tNumberOfBits;
             if (tNumberOfBitsForOneDecode > 32) {
@@ -276,16 +281,17 @@ bool IRrecv::decodeDistance() {
                     INFO_PRINT(F(" OneSpaceMicros="));
                     INFO_PRINT(tSpaceTicksLong * MICROS_PER_TICK);
                     INFO_PRINT(F(" ZeroSpaceMicros="));
-                    INFO_PRINTLN(tSpaceTicksShort * MICROS_PER_TICK);
-                }
-                if (tNumberOfAdditionalLong > 0) {
-                    // print only if we have more than 32 bits for decode
-                    INFO_PRINT(F(" 0x"));
-                    INFO_PRINT(decodedIRData.decodedRawData, HEX);
-                    tStartIndex += 64;
-                    tNumberOfBits -= 32;
+                    INFO_PRINT(tSpaceTicksShort * MICROS_PER_TICK);
+                    INFO_PRINT(F(" NumberOfBits="));
+                    INFO_PRINT(decodedIRData.numberOfBits);
+                    INFO_PRINT(F(" DecodedRawData:"));
                 }
+                INFO_PRINT(F(" 0x"));
+                INFO_PRINT(decodedIRData.decodedRawData, HEX);
+                tStartIndex += 64;
+                tNumberOfBits -= 32;
             }
+            INFO_PRINTLN();
         }
 
         // Store ticks used for decoding in extra