Remove __AVR__ compile options to improve code clarity

Author: pennam

src/ArduinoIoTCloudTCP.cpp

@@ -141,16 +141,7 @@ int ArduinoIoTCloudTCP::begin(bool const enable_watchdog, String brokerAddress,
   _brokerAddress = brokerAddress;
   _brokerPort = brokerPort;
 
-#if defined(__AVR__)
-  String const nina_fw_version = WiFi.firmwareVersion();
-  if (nina_fw_version < "1.4.2")
-  {
-    DEBUG_ERROR("ArduinoIoTCloudTCP::%s NINA firmware needs to be >= 1.4.2 to support cloud on Uno WiFi Rev. 2, current %s", __FUNCTION__, nina_fw_version.c_str());
-    return 0;
-  }
-#endif /* AVR */
-
-#if OTA_ENABLED && !defined(__AVR__)
+#if OTA_ENABLED
 #if defined(ARDUINO_PORTENTA_H7_M7) || defined(ARDUINO_NICLA_VISION)
   /* The length of the application can be retrieved the same way it was
    * communicated to the bootloader, that is by writing to the non-volatile
@@ -547,18 +538,14 @@ ArduinoIoTCloudTCP::State ArduinoIoTCloudTCP::handle_SubscribeThingTopics()
   if (!_mqttClient.subscribe(_dataTopicIn))
   {
     DEBUG_ERROR("ArduinoIoTCloudTCP::%s could not subscribe to %s", __FUNCTION__, _dataTopicIn.c_str());
-#if !defined(__AVR__)
     DEBUG_ERROR("Check your thing configuration, and press the reset button on your board.");
-#endif
     return State::SubscribeThingTopics;
   }
 
   if (!_mqttClient.subscribe(_shadowTopicIn))
   {
     DEBUG_ERROR("ArduinoIoTCloudTCP::%s could not subscribe to %s", __FUNCTION__, _shadowTopicIn.c_str());
-#if !defined(__AVR__)
     DEBUG_ERROR("Check your thing configuration, and press the reset button on your board.");
-#endif
     return State::SubscribeThingTopics;
   }
 

src/cbor/CBORDecoder.h

@@ -24,10 +24,6 @@
 
 #include <Arduino.h>
 
-#ifdef __AVR__
-# include <Arduino_AVRSTL.h>
-#endif
-
 #undef max
 #undef min
 #include <list>

src/cbor/CBOREncoder.h

@@ -24,10 +24,6 @@
 
 #include <Arduino.h>
 
-#ifdef __AVR__
-# include <Arduino_AVRSTL.h>
-#endif
-
 #undef max
 #undef min
 #include <list>

src/cbor/lib/tinycbor/src/cbor.h

@@ -25,11 +25,6 @@
 #ifndef CBOR_H
 #define CBOR_H
 
-#ifdef __AVR__
-#define FP_NAN          NAN
-#define FP_INFINITE     INFINITY
-#endif
-
 #ifndef assert
 #include <assert.h>
 #endif

src/cbor/lib/tinycbor/src/cborpretty.c

@@ -28,8 +28,6 @@
 #  define __STDC_LIMIT_MACROS 1
 #endif
 
-#ifndef __AVR__
-
 #include "cbor.h"
 #include "cborinternal_p.h"
 #include "compilersupport_p.h"
@@ -582,6 +580,4 @@ CborError cbor_value_to_pretty_stream(CborStreamFunction streamFunction, void *t
 
 #pragma GCC diagnostic pop
 
-#endif // __AVR__
-
 /** @} */

src/cbor/lib/tinycbor/src/cbortojson.c

@@ -30,8 +30,6 @@
 #  define __STDC_LIMIT_MACROS 1
 #endif
 
-#ifndef __AVR__
-
 #include "cbor.h"
 #include "cborjson.h"
 #include "cborinternal_p.h"
@@ -703,6 +701,4 @@ CborError cbor_value_to_json_advance(FILE *out, CborValue *value, int flags)
 
 #pragma GCC diagnostic pop
 
-#endif // __AVR__
-
 /** @} */

src/cbor/lib/tinycbor/src/open_memstream.c

@@ -34,10 +34,6 @@
 
 #include <unistd.h>
 
-#ifdef __AVR__
-typedef size_t ssize_t;
-#endif
-
 typedef ssize_t RetType;
 typedef size_t LenType;
 

src/nonstd/nonstd.h

@@ -222,11 +222,7 @@ CborError Property::appendAttributeReal(String value, String attributeName, Cbor
   }, encoder);
 }
 
-#ifdef __AVR__
-CborError Property::appendAttributeName(String attributeName, nonstd::function<CborError (CborEncoder& mapEncoder)>appendValue, CborEncoder *encoder)
-#else
 CborError Property::appendAttributeName(String attributeName, std::function<CborError (CborEncoder& mapEncoder)>appendValue, CborEncoder *encoder)
-#endif
 {
   if (attributeName != "") {
     // when the attribute name string is not empty, the attribute identifier is incremented in order to be encoded in the message if the _lightPayload flag is set
@@ -315,11 +311,7 @@ void Property::setAttributeReal(String& value, String attributeName) {
   });
 }
 
-#ifdef __AVR__
-void Property::setAttributeReal(String attributeName, nonstd::function<void (CborMapData & md)>setValue)
-#else
 void Property::setAttributeReal(String attributeName, std::function<void (CborMapData & md)>setValue)
-#endif
 {
   if (attributeName != "") {
     _attributeIdentifier++;