Reference to AzureIoTHubClient in SimpleSample code appears not necessary

[azure]

Reference to AzureIoTHubClient in SimpleSample code appears not necessary

[mamokarz]

Hi @GregTerrell ,

Thanks for your question, you are right, it is not necessary anymore. We can safely remove, not only the AzureIoTHubClient definition, but the sslClient as well, from both SimpleSample_MQTT and SimpleSample_HTTP.

// In the next line we decide each client ssl we'll use.
#ifdef ARDUINO_ARCH_ESP8266
static WiFiClientSecure sslClient; // for ESP8266
#elif ARDUINO_SAMD_FEATHER_M0
static Adafruit_WINC1500SSLClient sslClient; // for Adafruit WINC1500
#else
static WiFiSSLClient sslClient;
#endif

static AzureIoTHubClient iotHubClient;

iotHubClient.begin(sslClient);

If you would like to contribute with a pull request, I’ll be glad to accept and merge it.

Thanks,
Marcos

[GregTerrell]

Sorry for the delay... yes I will prepare a pull-request in the next day or two for the removal. Looks like it affects the items in the examples folder for azure-iot-arduino, azure-iot-arduino-protocol-http, and azure-iot-protocol-mqtt.

[tameraw]

PR merged.