Update Irremote lib to latest master commit ca474a6 (#24226)

Author: Jason2866

lib/lib_basic/IRremoteESP8266/IRremoteESP8266/.github/workflows/Build.yml

@@ -25,7 +25,7 @@ jobs:
         JSONI=${JSONI//$'\n'}
         echo $JSONI
         # Set output
-        echo "::set-output name=matrix::[${JSONI}]"
+        echo "matrix=[${JSONI}]" >> $GITHUB_OUTPUT
   Build_Example:
     needs: Gen_Matrix
     runs-on: ubuntu-latest
@@ -36,30 +36,33 @@ jobs:
     steps:
     - uses: actions/checkout@v2
     - name: Cache pip
-      uses: actions/cache@v2
+      uses: actions/cache@v4
       with:
         path: ~/.cache/pip
         key: ${{ runner.os }}-pip-${{ hashFiles('**/requirements.txt') }}
         restore-keys: |
           ${{ runner.os }}-pip-
     - name: Cache PlatformIO
-      uses: actions/cache@v2
+      uses: actions/cache@v4
       with:
         path: ~/.platformio
         key: ${{ runner.os }}-${{ hashFiles('**/platformio.ini') }}
     - name: Cache PlatformIO build
-      uses: actions/cache@v2
+      uses: actions/cache@v4
       with:
         path: .pio
         key: pio-${{ runner.os }}-${{ matrix.project }}
         restore-keys: |
           pio-${{ runner.os }}-
           pio-
     - name: Set up Python
-      uses: actions/setup-python@v2
+      uses: actions/setup-python@v5
+      with:
+        python-version: '3.13'
     - name: Install PlatformIO
       run: |
         python -m pip install --upgrade pip
+        pip install --upgrade intelhex
         pip install --upgrade platformio
     - name: Build example
       env:

lib/lib_basic/IRremoteESP8266/IRremoteESP8266/.github/workflows/codeql-analysis.yml

@@ -42,7 +42,7 @@ jobs:
 
     # Initializes the CodeQL tools for scanning.
     - name: Initialize CodeQL
-      uses: github/codeql-action/init@v2
+      uses: github/codeql-action/init@v3
       with:
         languages: ${{ matrix.language }}
         # If you wish to specify custom queries, you can do so here or in a config file.
@@ -56,7 +56,7 @@ jobs:
     # Autobuild attempts to build any compiled languages  (C/C++, C#, or Java).
     # If this step fails, then you should remove it and run the build manually (see below)
     - name: Autobuild
-      uses: github/codeql-action/autobuild@v2
+      uses: github/codeql-action/autobuild@v3
 
     # ℹ️ Command-line programs to run using the OS shell.
     # 📚 See https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_idstepsrun
@@ -69,4 +69,4 @@ jobs:
     #   ./location_of_script_within_repo/buildscript.sh
 
     - name: Perform CodeQL Analysis
-      uses: github/codeql-action/analyze@v2
+      uses: github/codeql-action/analyze@v3

lib/lib_basic/IRremoteESP8266/IRremoteESP8266/CPPLINT.cfg

@@ -1,3 +1,4 @@
 set noparent
 root=src
 linelength=80
+filter=-whitespace/indent_namespace

lib/lib_basic/IRremoteESP8266/IRremoteESP8266/README.md

@@ -77,6 +77,8 @@ If you want to [contribute](.github/CONTRIBUTING.md#how-can-i-contribute) to thi
 - Improve our documentation
 - [Creating issues](.github/CONTRIBUTING.md#reporting-bugs) and [pull requests](.github/CONTRIBUTING.md#pull-requests)
 - Tell other people about this library
+- Updated documentation formatting and clarified installation steps (Hacktoberfest contribution by Prerna Utage)
+
 
 ## Contributors
 Available [here](.github/Contributors.md)
@@ -87,3 +89,6 @@ This library was originally based on Ken Shirriff's work (https://github.com/shi
 [Mark Szabo](https://github.com/crankyoldgit/IRremoteESP8266) has updated the IRsend class to work on ESP8266 and [Sebastien Warin](https://github.com/sebastienwarin/IRremoteESP8266) the receiving & decoding part (IRrecv class).
 
 As of v2.0, the library was almost entirely re-written with the ESP8266's resources in mind.
+
+## About This Project
+This project allows decoding and encoding of IR signals for controlling Air Conditioners and other devices using ESP8266 or ESP32 boards.

lib/lib_basic/IRremoteESP8266/IRremoteESP8266/examples/IRMQTTServer/IRMQTTServer.ino

@@ -593,7 +593,7 @@ bool mountSpiffs(void) {
 bool saveConfig(void) {
   debug("Saving the config.");
   bool success = false;
-  DynamicJsonDocument json(kJsonConfigMaxSize);
+  JsonDocument json;
 #if MQTT_ENABLE
   json[kMqttServerKey] = MqttServer;
   json[kMqttPortKey] = MqttPort;
@@ -611,6 +611,8 @@ bool saveConfig(void) {
     const String key = KEY_TX_GPIO + String(i);
     json[key] = static_cast<int>(txGpioTable[i]);
   }
+  if (json.overflowed())
+    debug("ERROR: no enough memory to store the entire json document");
 
   if (mountSpiffs()) {
     File configFile = FILESYSTEM.open(kConfigFile, "w");
@@ -642,7 +644,7 @@ bool loadConfigFile(void) {
         std::unique_ptr<char[]> buf(new char[size]);
 
         configFile.readBytes(buf.get(), size);
-        DynamicJsonDocument json(kJsonConfigMaxSize);
+        JsonDocument json;
         if (!deserializeJson(json, buf.get(), kJsonConfigMaxSize)) {
           debug("Json config file parsed ok.");
 #if MQTT_ENABLE
@@ -1659,11 +1661,13 @@ bool parseStringAndSendAirCon(IRsend *irsend, const decode_type_t irType,
       stateSize = inputLength / 2;  // Every two hex chars is a byte.
       // Use at least the minimum size.
       stateSize = std::max(stateSize,
-                           (uint16_t) (kFujitsuAcStateLengthShort - 1));
+                           static_cast<uint16_t>(kFujitsuAcStateLengthShort -
+                                                 1));
       // If we think it isn't a "short" message.
       if (stateSize > kFujitsuAcStateLengthShort)
         // Then it has to be at least the smaller version of the "normal" size.
-        stateSize = std::max(stateSize, (uint16_t) (kFujitsuAcStateLength - 1));
+        stateSize = std::max(stateSize,
+                             static_cast<uint16_t>(kFujitsuAcStateLength - 1));
       // Lastly, it should never exceed the maximum "normal" size.
       stateSize = std::min(stateSize, kFujitsuAcStateLength);
       break;
@@ -1675,12 +1679,12 @@ bool parseStringAndSendAirCon(IRsend *irsend, const decode_type_t irType,
       stateSize = inputLength / 2;  // Every two hex chars is a byte.
       // Use at least the minimum size.
       stateSize = std::max(stateSize,
-                           (uint16_t) (kHitachiAc3MinStateLength));
+                           static_cast<uint16_t>(kHitachiAc3MinStateLength));
       // If we think it isn't a "short" message.
       if (stateSize > kHitachiAc3MinStateLength)
         // Then it probably the "normal" size.
         stateSize = std::max(stateSize,
-                             (uint16_t) (kHitachiAc3StateLength));
+                             static_cast<uint16_t>(kHitachiAc3StateLength));
       // Lastly, it should never exceed the maximum "normal" size.
       stateSize = std::min(stateSize, kHitachiAc3StateLength);
       break;
@@ -1691,7 +1695,7 @@ bool parseStringAndSendAirCon(IRsend *irsend, const decode_type_t irType,
       // the correct length/byte size.
       stateSize = inputLength / 2;  // Every two hex chars is a byte.
       // Use at least the minimum size.
-      stateSize = std::max(stateSize, (uint16_t) 3);
+      stateSize = std::max(stateSize, static_cast<uint16_t>(3));
       // Cap the maximum size.
       stateSize = std::min(stateSize, kStateSizeMax);
       break;
@@ -1702,12 +1706,13 @@ bool parseStringAndSendAirCon(IRsend *irsend, const decode_type_t irType,
       // the correct length/byte size.
       stateSize = inputLength / 2;  // Every two hex chars is a byte.
       // Use at least the minimum size.
-      stateSize = std::max(stateSize, (uint16_t) (kSamsungAcStateLength));
+      stateSize = std::max(stateSize,
+                           static_cast<uint16_t>(kSamsungAcStateLength));
       // If we think it isn't a "normal" message.
       if (stateSize > kSamsungAcStateLength)
         // Then it probably the extended size.
-        stateSize = std::max(stateSize,
-                             (uint16_t) (kSamsungAcExtendedStateLength));
+        stateSize = std::max(
+            stateSize, static_cast<uint16_t>(kSamsungAcExtendedStateLength));
       // Lastly, it should never exceed the maximum "extended" size.
       stateSize = std::min(stateSize, kSamsungAcExtendedStateLength);
       break;
@@ -2671,7 +2676,8 @@ void receivingMQTT(String const topic_name, String const callback_str) {
     switch (ircommand[0]) {
       case kPauseChar:
         {  // It's a pause. Everything after the 'P' should be a number.
-          int32_t msecs = std::min((int32_t) strtoul(ircommand + 1, NULL, 10),
+          int32_t msecs = std::min(static_cast<int32_t>(strtoul(ircommand + 1,
+                                                        NULL, 10)),
                                    kMaxPauseMs);
           delay(msecs);
           mqtt_client.publish(MqttAck.c_str(),
@@ -3142,7 +3148,7 @@ bool sendFloat(const String topic, const float_t temp, const bool retain) {
 #if MQTT_CLIMATE_JSON
 void sendJsonState(const stdAc::state_t state, const String topic,
                    const bool retain, const bool ha_mode) {
-  DynamicJsonDocument json(kJsonAcStateMaxSize);
+  JsonDocument json;
   json[KEY_PROTOCOL] = typeToString(state.protocol);
   json[KEY_MODEL] = state.model;
   json[KEY_COMMAND] = IRac::commandToString(state.command);
@@ -3168,23 +3174,25 @@ void sendJsonState(const stdAc::state_t state, const String topic,
   json[KEY_CLEAN] = IRac::boolToString(state.clean);
   json[KEY_BEEP] = IRac::boolToString(state.beep);
   json[KEY_SLEEP] = state.sleep;
+  if (json.overflowed())
+    debug("ERROR: no enough memory to store the entire json document");
 
   String payload = "";
   payload.reserve(200);
   serializeJson(json, payload);
   sendString(topic, payload, retain);
 }
 
-bool validJsonStr(DynamicJsonDocument doc, const char* key) {
+bool validJsonStr(JsonDocument doc, const char* key) {
   return doc.containsKey(key) && doc[key].is<char*>();
 }
 
-bool validJsonInt(DynamicJsonDocument doc, const char* key) {
+bool validJsonInt(JsonDocument doc, const char* key) {
   return doc.containsKey(key) && doc[key].is<signed int>();
 }
 
 stdAc::state_t jsonToState(const stdAc::state_t current, const char *str) {
-  DynamicJsonDocument json(kJsonAcStateMaxSize);
+  JsonDocument json;
   if (deserializeJson(json, str, kJsonAcStateMaxSize)) {
     debug("json MQTT message did not parse. Skipping!");
     return current;

lib/lib_basic/IRremoteESP8266/IRremoteESP8266/examples/IRMQTTServer/platformio.ini

@@ -14,7 +14,7 @@ monitor_speed = 115200
 lib_deps_builtin =
 lib_deps_external =
   PubSubClient@>=2.8.0
-  ArduinoJson@>=6.0
+  ArduinoJson@>=7.0
 # Uncomment the following to enable SHT-3x support.
 #  https://github.com/wemos/WEMOS_SHT3x_Arduino_Library.git
 

lib/lib_basic/IRremoteESP8266/IRremoteESP8266/examples/IRrecvDump/IRrecvDump.ino

@@ -87,7 +87,7 @@ void dump(decode_results *results) {
       Serial.print(results->rawbuf[i] * kRawTick, DEC);
     } else {
       Serial.print(", ");
-      Serial.print((uint32_t) results->rawbuf[i] * kRawTick, DEC);
+      Serial.print(static_cast<uint32_t>(results->rawbuf[i] * kRawTick), DEC);
     }
   }
   Serial.println("};");

lib/lib_basic/IRremoteESP8266/IRremoteESP8266/examples/IRrecvDumpV2/IRrecvDumpV2.ino

@@ -130,6 +130,8 @@ decode_results results;  // Somewhere to store the results
 void setup() {
 #if defined(ESP8266)
   Serial.begin(kBaudRate, SERIAL_8N1, SERIAL_TX_ONLY);
+#elif ARDUINO_USB_CDC_ON_BOOT
+  Serial.begin(kBaudRate);
 #else  // ESP8266
   Serial.begin(kBaudRate, SERIAL_8N1);
 #endif  // ESP8266

lib/lib_basic/IRremoteESP8266/IRremoteESP8266/examples/IRrecvDumpV3/IRrecvDumpV3.ino

@@ -138,6 +138,8 @@ void setup() {
   OTAwifi();  // start default wifi (previously saved on the ESP) for OTA
 #if defined(ESP8266)
   Serial.begin(kBaudRate, SERIAL_8N1, SERIAL_TX_ONLY);
+#elif ARDUINO_USB_CDC_ON_BOOT
+  Serial.begin(kBaudRate);
 #else  // ESP8266
   Serial.begin(kBaudRate, SERIAL_8N1);
 #endif  // ESP8266

lib/lib_basic/IRremoteESP8266/IRremoteESP8266/library.json

@@ -46,5 +46,6 @@
   ],
   "exclude": [".github", "extras", "docs", "assets"],
   "frameworks": "arduino",
+  "libCompatMode": "strict",
   "platforms": ["espressif8266", "espressif32"]
 }