working on restructuring and renaming. Change IRremote.h to AIRremote.h

If there are other ideas for a better name then we can use that, A =
Arduino in case it wasnt obvious.

AIRremote.h

@@ -15,8 +15,8 @@
 // Whynter A/C ARC-110WD added by Francesco Meschia
 //******************************************************************************
 
-#ifndef IRremote_h
+#ifndef AIRremote_h
-#define IRremote_h
+#define AIRremote_h
 
 //------------------------------------------------------------------------------
 // The ISR header contains several useful macros the user may wish to use
@@ -79,9 +79,6 @@
 #define DECODE_LEGO_PF       0 // NOT WRITTEN
 #define SEND_LEGO_PF         1
 
-#define DECODE_CDTV          1
-#define SEND_CDTV            1
-
 //------------------------------------------------------------------------------
 // When sending a Pronto code we request to send either the "once" code
 //                                                   or the "repeat" code
@@ -122,7 +119,6 @@ typedef
 		DENON,
 		PRONTO,
 		LEGO_PF,
-		CDTV,
 	}
 decode_type_t;
 
@@ -254,10 +250,6 @@ class IRrecv
 //......................................................................
 #		if DECODE_LEGO_PF
 			bool  decodeLegoPowerFunctions (decode_results *results) ;
-#		endif
-		//......................................................................
-#		if DECODE_CDTV
-			bool  decodeCDTV	   (decode_results *results) ;
 #		endif
 } ;
 
@@ -346,10 +338,6 @@ class IRsend
 //......................................................................
 #		if SEND_LEGO_PF
 			void  sendLegoPowerFunctions (uint16_t data, bool repeat = true) ;
-#		endif
-		//......................................................................
-#		if SEND_CDTV
-			void  sendCDTV		 (unsigned long data, int nbits) ;
 #		endif
 } ;
 

Contributors.md

@@ -15,9 +15,8 @@ These are the active contributors of this project that you may contact if there
 - [csBlueChip](https://github.com/csbluechip) : Active contributor, who contributed major and vital changes to the code base.
 - [Sebazzz](https://github.com/sebazz): Contributor
 - [lumbric](https://github.com/lumbric): Contributor
-- [ElectricRCAircraftGuy](https://github.com/electricrcaircraftguy): Contributor
+- [ElectricRCAircraftGuy](https://github.com/electricrcaircraftguy): Active Contributor
 - [henkel](https://github.com/henkel): Contributor
 - [MCUdude](https://github.com/MCUdude): Contributor
-- [salocinx](https://github.com/salocinx): Contributor
 
 Note: This list is being updated constantly so please let [z3t0](https://github.com/z3t0) know if you have been missed.

IRremote.cpp

@@ -22,7 +22,7 @@
 
 // Defining IR_GLOBAL here allows us to declare the instantiation of global variables
 #define IR_GLOBAL
-#	include "IRremote.h"
+#	include "AIRremote.h"
 #	include "IRremoteInt.h"
 #undef IR_GLOBAL
 

IRremoteInt.h

@@ -108,6 +108,6 @@ EXTERN  volatile irparams_t  irparams;
 #define SPACE  1
 
 // All board specific stuff has been moved to its own file, included here.
-#include "IRremoteBoardDefs.h"
+#include "boarddefs.h"
 
 #endif

ISSUE_TEMPLATE.md

@@ -5,7 +5,7 @@
 **Code Block:**
 ```c
 
-#include <IRremote.h> 
+#include <AIRremote.h> 
 
 .....
 

boarddefs.h

@@ -17,8 +17,8 @@
 // Whynter A/C ARC-110WD added by Francesco Meschia
 //******************************************************************************
 
-#ifndef IRremoteBoardDefs_h
+#ifndef boarddefs_h
-#define IRremoteBoardDefs_h
+#define boarddefs_h
 
 //------------------------------------------------------------------------------
 // Defines for blinking the LED
@@ -545,4 +545,4 @@
 #	error "Internal code configuration error, no known IR_USE_TIMER# defined\n"
 #endif
 
-#endif // ! IRremoteBoardDefs_h
+#endif // ! boarddefs_h

changelog.md

@@ -1,10 +1,3 @@
-## 2.3.0 - 2017/02/06
-- Added Commodore Amiga CDTV Support [ISSUE #407](https://github.com/z3t0/Arduino-IRremote/issues/407) [PR #413](https://github.com/z3t0/Arduino-IRremote/pull/413)
-
-## 2.2.2 - 2017/01/20
-- Renamed "boarddefs.h" [ISSUE #375](https://github.com/z3t0/Arduino-IRremote/issues/375)
-- Fixed naming bug [PR #398](https://github.com/z3t0/Arduino-IRremote/pull/398)
-
 ## 2.2.1 - 2016/07/27
 - Added tests for Lego Power Functions Protocol [PR #336](https://github.com/z3t0/Arduino-IRremote/pull/336)
 

examples/AiwaRCT501SendDemo/AiwaRCT501SendDemo.ino

@@ -6,7 +6,7 @@
  * http://arcfn.com
  */
 
-#include "IRremote.h"
+#include "AIRremote.h"
 
 #define POWER 0x7F80
 #define AIWA_RC_T501

examples/IRrecord/IRrecord.ino

@@ -15,7 +15,7 @@
  * http://arcfn.com
  */
 
-#include <IRremote.h>
+#include <AIRremote.h>
 
 int RECV_PIN = 11;
 int BUTTON_PIN = 12;
@@ -126,7 +126,7 @@ void sendCode(int repeat) {
     Serial.println(codeValue, HEX);
   }
   else if (codeType == JVC) {
-    irsend.sendJVC(codeValue, codeLen, false);
+    irsend.sendPanasonic(codeValue, codeLen);
     Serial.print("Sent JVC");
     Serial.println(codeValue, HEX);
   }

examples/IRrecvDemo/IRrecvDemo.ino

@@ -6,7 +6,7 @@
  * http://arcfn.com
  */
 
-#include <IRremote.h>
+#include <AIRremote.h>
 
 int RECV_PIN = 11;
 

examples/IRrecvDump/IRrecvDump.ino

@@ -8,7 +8,7 @@
  * LG added by Darryl Smith (based on the JVC protocol)
  */
 
-#include <IRremote.h>
+#include <AIRremote.h>
 
 /* 
 *  Default is Arduino pin D11. 

examples/IRrecvDumpV2/IRrecvDumpV2.ino

@@ -1,7 +1,7 @@
 //------------------------------------------------------------------------------
 // Include the IRremote library header
 //
-#include <IRremote.h>
+#include <AIRremote.h>
 
 //------------------------------------------------------------------------------
 // Tell IRremote which Arduino pin is connected to the IR Receiver (TSOP4838)
@@ -66,7 +66,7 @@ void  dumpInfo (decode_results *results)
 {
   // Check if the buffer overflowed
   if (results->overflow) {
-    Serial.println("IR code too long. Edit IRremoteInt.h and increase RAWBUF");
+    Serial.println("IR code too long. Edit IRremoteInt.h and increase RAWLEN");
     return;
   }
 

examples/IRrelay/IRrelay.ino

@@ -6,7 +6,7 @@
  * http://arcfn.com
  */
 
-#include <IRremote.h>
+#include <AIRremote.h>
 
 int RECV_PIN = 11;
 int RELAY_PIN = 4;

examples/IRremoteInfo/IRremoteInfo.ino

@@ -11,7 +11,7 @@
  */
 
 
-#include <IRremote.h>
+#include <AIRremote.h>
 
 void setup()
 {
@@ -224,7 +224,7 @@ void dumpFooter() {
   Serial.println(F("Notes: "));
   Serial.println(F("     - Most of the seetings above can be configured in the following files included as part of the library"));
   Serial.println(F("     - IRremteInt.h"));
-  Serial.println(F("     - IRremote.h"));
+  Serial.println(F("     - AIRremote.h"));
   Serial.println(F("     - You can save SRAM by disabling the Decode or Send features for any protocol (Near the top of IRremoteInt.h)"));
   Serial.println(F("     - Some Timer conflicts, with other libraries, can be easily resolved by configuring a differnt Timer for your platform"));
 }

examples/IRsendDemo/IRsendDemo.ino

@@ -7,7 +7,7 @@
  */
 
 
-#include <IRremote.h>
+#include <AIRremote.h>
 
 IRsend irsend;
 

examples/IRsendRawDemo/IRsendRawDemo.ino

@@ -18,7 +18,7 @@
  */
 
 
-#include <IRremote.h>
+#include <AIRremote.h>
 
 IRsend irsend;
 

examples/IRtest/IRtest.ino

@@ -4,12 +4,12 @@
  * Copyright 2009 Ken Shirriff
  * http://arcfn.com
  *
- * Note: to run these tests, edit IRremote/IRremote.h to add "#define TEST"
+ * Note: to run these tests, edit IRremote/AIRremote.h to add "#define TEST"
  * You must then recompile the library by removing IRremote.o and restarting
  * the arduino IDE.
  */
 
-#include <IRremote.h>
+#include <AIRremote.h>
 #include <IRremoteInt.h>
 
 // Dumps out the decode_results structure.

examples/IRtest2/IRtest2.ino

@@ -25,7 +25,7 @@
  * http://arcfn.com
  */
 
-#include <IRremote.h>
+#include <AIRremote.h>
 
 int RECV_PIN = 11;
 int LED_PIN = 3;

examples/JVCPanasonicSendDemo/JVCPanasonicSendDemo.ino

@@ -6,7 +6,7 @@
  * http://arcfn.com
  * JVC and Panasonic protocol added by Kristian Lauszus (Thanks to zenwheel and other people at the original blog post)
  */
-#include <IRremote.h>
+#include <AIRremote.h>
  
 #define PanasonicAddress      0x4004     // Panasonic address (Pre data) 
 #define PanasonicPower        0x100BCBD  // Panasonic Power button

examples/LGACSendDemo/LGACSendDemo.ino

@@ -1,4 +1,4 @@
-#include <IRremote.h>
+#include <AIRremote.h>
 #include <Wire.h>
 
 

examples/LegoPowerFunctionsSendDemo/LegoPowerFunctionsSendDemo.ino

@@ -3,7 +3,7 @@
  * Copyright (c) 2016 Philipp Henkel
  */
 
-#include <IRremote.h>
+#include <AIRremote.h>
 #include <IRremoteInt.h>
 
 IRsend irsend;

irPronto.cpp

@@ -17,7 +17,7 @@
 	void  mark        (int t)     { printf("+%d," , t); }
 	void  space       (int t)     { printf("-%d, ", t); }
 #else
-#	include "IRremote.h"
+#	include "AIRremote.h"
 #endif // TEST
 
 //+=============================================================================

irRecv.cpp

@@ -1,4 +1,4 @@
-#include "IRremote.h"
+#include "AIRremote.h"
 #include "IRremoteInt.h"
 
 //+=============================================================================
@@ -85,11 +85,6 @@ int  IRrecv::decode (decode_results *results)
 	if (decodeLegoPowerFunctions(results))  return true ;
 #endif
 
-#if DECODE_CDTV
-	DBG_PRINTLN("Attempting Commodore Amiga CDTV decode");
-	if (decodeCDTV(results))  return true;
-#endif
-
 	// decodeHash returns a hash on any input.
 	// Thus, it needs to be last in the list.
 	// If you add any decodes, add them before this.

irSend.cpp

@@ -1,4 +1,4 @@
-#include "IRremote.h"
+#include "AIRremote.h"
 #include "IRremoteInt.h"
 
 //+=============================================================================

ir_Aiwa.cpp

@@ -1,4 +1,4 @@
-#include "IRremote.h"
+#include "AIRremote.h"
 #include "IRremoteInt.h"
 
 //==============================================================================

ir_CDTV.cpp

@@ -1,148 +0,0 @@
-#include "IRremote.h"
-#include "IRremoteInt.h"
-
-//==============================================================================
-//                            COMMODORE AMIGA CD-TV
-//==============================================================================
-
-//==============================================================================
-//                           CCCC  DDDD   TTTTT V       V
-//                          C      D   D    T    V     V
-//                          C      D   D    T     V   V
-//                          C      D   D    T      V V
-//                           CCCC  DDDD     T       V
-//==============================================================================
-
-//==============================================================================
-//                                 MEASUREMENTS
-//==============================================================================
-
-/*Encoding: UNKNOWN
-	Code : 72A03D6B(32 bits)
-	Timing[51] :
-	+8900, -4450 + 400, -1200 + 350, -400 + 400, -400
-	+ 400, -400 + 400, -1200 + 400, -400 + 350, -450
-	+ 350, -400 + 400, -400 + 400, -350 + 450, -400
-	+ 350, -450 + 350, -400 + 400, -1200 + 400, -1200
-	+ 350, -1200 + 400, -450 + 350, -1200 + 400, -1200
-	+ 350, -1200 + 400, -1200 + 350, -1250 + 350, -1200
-	+ 400, -1200 + 350
-	unsigned int rawData[51] = { 8900,4450, 400,1200, 350,400, 400,400, 400,400, 400,1200, 400,400, 350,450, 350,400, 400,400, 400,350, 450,400, 350,450, 350,400, 400,1200, 400,1200, 350,1200, 400,450, 350,1200, 400,1200, 350,1200, 400,1200, 350,1250, 350,1200, 400,1200, 350 }; // UNKNOWN 72A03D6B
-*/
-
-// NOTE: The array dump above begins at index = 1; therefore offset = 1 in the code below
-
-//==============================================================================
-
-#define CDTV_BITS          24
-// timing intervals in usec
-#define CDTV_HDR_MARK    8850			// start burst
-#define CDTV_HDR_SPACE   4450			// pause after start
-#define CDTV_BIT_MARK     350			// pulse
-#define CDTV_ONE_SPACE   1250			// receive a '1'
-#define CDTV_ZERO_SPACE   450			// receive a '0'
-#define CDTV_RPT_SPACE   2250			// repeat signal
-// message sizes measured in raw pulses
-#define CDTV_RAW_REPEAT_LENGTH	   4    // CDTV_HDR_MARK + CDTV_HDR_SPACE + CDTV_BIT_MARK
-#define CDTV_RAW_SIGNAL_LENGTH	   52   // CDTV_HDR_MARK + CDTV_HDR_SPACE + CDTV_BITS * (CDTV_BIT_MARK + CDTV_ZERO_SPACE | CDTV_ONE_SPACE)
-
-//+=============================================================================
-#if SEND_CDTV
-void  IRsend::sendCDTV(unsigned long data, int nbits) {
-
-	// set IR carrier frequency
-	enableIROut(40);
-
-	// send header
-	mark(CDTV_HDR_MARK);
-	space(CDTV_HDR_SPACE);
-
-	// send data
-	for (unsigned long mask = 1UL << (nbits - 1); mask; mask >>= 1) {
-		if (data & mask) {
-			mark(CDTV_BIT_MARK);
-			space(CDTV_ONE_SPACE);
-		} else {
-			mark(CDTV_BIT_MARK);
-			space(CDTV_ZERO_SPACE);
-		}
-	}
-
-	// send footer
-	mark(CDTV_BIT_MARK);
-	// always end with the LED off
-	space(0);
-
-}
-#endif
-
-//+=============================================================================
-// CDTV have a repeat signal that is 4-bits long [#FFFFFF]
-//
-#if DECODE_CDTV
-bool  IRrecv::decodeCDTV(decode_results *results) {
-
-	unsigned long  data   = 0;    // we decode into this field; start with empty placeholder
-	int            offset = 1;	  // skip first entry since it has not a lot of meaning
-
-	// check if header mark is within range
-	if (!MATCH_MARK(results->rawbuf[offset], CDTV_HDR_MARK)) {
-		return false;
-	}
-	offset++;	// -> offset[2]
-
-	// check for 4-bit repeat signal
-	if ((irparams.rawlen == CDTV_RAW_REPEAT_LENGTH)
-		&& MATCH_SPACE(results->rawbuf[offset], CDTV_RPT_SPACE)
-		&& MATCH_MARK(results->rawbuf[offset+1], CDTV_BIT_MARK)) {
-		results->bits = 4;
-		results->value = 0xFFFFFF;
-		results->decode_type = CDTV;
-		return true;
-	}
-
-	// check on minimal raw length for decoding the 24-bit signal correctly
-	if (irparams.rawlen < CDTV_RAW_SIGNAL_LENGTH) {
-		return false;
-	}
-
-	// check header space
-	if (!MATCH_SPACE(results->rawbuf[offset], CDTV_HDR_SPACE)) {
-		return false;
-	}
-	offset++;	// -> offset[3]
-
-	// build the data
-	for (int i = 0; i < CDTV_BITS; i++) {
-		// check if data mark is available; otherwise break
-		if (!MATCH_MARK(results->rawbuf[offset], CDTV_BIT_MARK)) {
-			return false;
-		}
-		offset++;    // -> offset[4->28]
-		// append this bit
-		if (MATCH_SPACE(results->rawbuf[offset], CDTV_ONE_SPACE)) {
-			data = (data << 1) | 1;
-		} else if (MATCH_SPACE(results->rawbuf[offset], CDTV_ZERO_SPACE)) {
-			data = (data << 1) | 0;
-		} else {
-			return false;
-		}
-		offset++;
-	}
-
-	// validate checksum by comparing lower 12-bits with inverted higher 12-bits
-	unsigned long lo = data & 0xFFF;		// extract lower 12-bit
-	unsigned long hi = data >> 12;			// extract higher 12-bit
-
-	// success if (low XOR high == 0xFFF)
-	if (lo^hi == 0xFFF) {
-		results->bits = CDTV_BITS;
-		results->value = data;
-		results->decode_type = CDTV;
-		return true;
-	}
-
-	return false;
-
-}
-#endif

ir_Denon.cpp

@@ -1,4 +1,4 @@
-#include "IRremote.h"
+#include "AIRremote.h"
 #include "IRremoteInt.h"
 
 // Reverse Engineered by looking at RAW dumps generated by IRremote

ir_Dish.cpp

@@ -1,4 +1,4 @@
-#include "IRremote.h"
+#include "AIRremote.h"
 #include "IRremoteInt.h"
 
 //==============================================================================

ir_JVC.cpp

@@ -1,4 +1,4 @@
-#include "IRremote.h"
+#include "AIRremote.h"
 #include "IRremoteInt.h"
 
 //==============================================================================

ir_LG.cpp

@@ -1,4 +1,4 @@
-#include "IRremote.h"
+#include "AIRremote.h"
 #include "IRremoteInt.h"
 
 //==============================================================================

ir_Lego_PF.cpp

@@ -1,4 +1,4 @@
-#include "IRremote.h"
+#include "AIRremote.h"
 #include "IRremoteInt.h"
 #include "ir_Lego_PF_BitStreamEncoder.h"
 

ir_Mitsubishi.cpp

@@ -1,4 +1,4 @@
-#include "IRremote.h"
+#include "AIRremote.h"
 #include "IRremoteInt.h"
 
 //==============================================================================

ir_NEC.cpp

@@ -1,4 +1,4 @@
-#include "IRremote.h"
+#include "AIRremote.h"
 #include "IRremoteInt.h"
 
 //==============================================================================

ir_Panasonic.cpp

@@ -1,4 +1,4 @@
-#include "IRremote.h"
+#include "AIRremote.h"
 #include "IRremoteInt.h"
 
 //==============================================================================

ir_RC5_RC6.cpp

@@ -1,4 +1,4 @@
-#include "IRremote.h"
+#include "AIRremote.h"
 #include "IRremoteInt.h"
 
 //+=============================================================================

ir_Samsung.cpp

@@ -1,4 +1,4 @@
-#include "IRremote.h"
+#include "AIRremote.h"
 #include "IRremoteInt.h"
 
 //==============================================================================

ir_Sanyo.cpp

@@ -1,4 +1,4 @@
-#include "IRremote.h"
+#include "AIRremote.h"
 #include "IRremoteInt.h"
 
 //==============================================================================

ir_Sharp.cpp

@@ -1,4 +1,4 @@
-#include "IRremote.h"
+#include "AIRremote.h"
 #include "IRremoteInt.h"
 
 //==============================================================================

ir_Sony.cpp

@@ -1,4 +1,4 @@
-#include "IRremote.h"
+#include "AIRremote.h"
 #include "IRremoteInt.h"
 
 //==============================================================================

ir_Template.cpp

@@ -31,7 +31,7 @@ You have written the code to support your new protocol!
 
 Now you must do a few things to add it to the IRremote system:
 
-1. Open IRremote.h and make the following changes:
+1. Open AIRremote.h and make the following changes:
    REMEMEBER to change occurences of "SHUZU" with the name of your protocol
 
    A. At the top, in the section "Supported Protocols", add:
@@ -91,7 +91,7 @@ Regards,
   BlueChip
 */
 
-#include "IRremote.h"
+#include "AIRremote.h"
 #include "IRremoteInt.h"
 
 //==============================================================================

ir_Whynter.cpp

@@ -1,4 +1,4 @@
-#include "IRremote.h"
+#include "AIRremote.h"
 #include "IRremoteInt.h"
 
 //==============================================================================