[Cleanup] Use of "byte" as a type. uint8_t or (C++17) std::byte are better

As discussed here: esp8266/Arduino#8089 (comment)

Author: TD-er

src/ESPEasy-Globals.h

@@ -60,9 +60,9 @@ struct pinStatesStruct
 {
   pinStatesStruct() : value(0), plugin(0), index(0), mode(0) {}
   uint16_t value;
-  byte plugin;
-  byte index;
-  byte mode;
+  uint8_t plugin;
+  uint8_t index;
+  uint8_t mode;
 } pinStates[PINSTATE_TABLE_MAX];
 */
 

src/_Plugin_Helper.cpp

@@ -86,12 +86,12 @@ String getPluginCustomArgName(int varNr) {
 // if the regular values should also be displayed.
 // The call to PLUGIN_WEBFORM_SHOW_VALUES should only return success = true when no regular values should be displayed
 // Note that the varNr of the custom values should not conflict with the existing variable numbers (e.g. start at VARS_PER_TASK)
-void pluginWebformShowValue(taskIndex_t taskIndex, byte varNr, const __FlashStringHelper * label, const String& value, bool addTrailingBreak) {
+void pluginWebformShowValue(taskIndex_t taskIndex, uint8_t varNr, const __FlashStringHelper * label, const String& value, bool addTrailingBreak) {
   pluginWebformShowValue(taskIndex, varNr, String(label), value, addTrailingBreak);
 }
 
 void pluginWebformShowValue(taskIndex_t   taskIndex,
-                            byte          varNr,
+                            uint8_t          varNr,
                             const String& label,
                             const String& value,
                             bool          addTrailingBreak) {
@@ -123,7 +123,7 @@ void pluginWebformShowValue(const String& valName, const String& valName_id, con
   }
 }
 
-bool pluginOptionalTaskIndexArgumentMatch(taskIndex_t taskIndex, const String& string, byte paramNr) {
+bool pluginOptionalTaskIndexArgumentMatch(taskIndex_t taskIndex, const String& string, uint8_t paramNr) {
   if (!validTaskIndex(taskIndex)) {
     return false;
   }

src/_Plugin_Helper.h

@@ -114,13 +114,13 @@ String               getPluginCustomArgName(int varNr);
 // The call to PLUGIN_WEBFORM_SHOW_VALUES should only return success = true when no regular values should be displayed
 // Note that the varNr of the custom values should not conflict with the existing variable numbers (e.g. start at VARS_PER_TASK)
 void pluginWebformShowValue(taskIndex_t   taskIndex,
-                            byte          varNr,
+                            uint8_t          varNr,
                             const __FlashStringHelper * label,
                             const String& value,
                             bool          addTrailingBreak = false);
 
 void pluginWebformShowValue(taskIndex_t   taskIndex,
-                            byte          varNr,
+                            uint8_t          varNr,
                             const String& label,
                             const String& value,
                             bool          addTrailingBreak = false);
@@ -141,7 +141,7 @@ void pluginWebformShowValue(const String& valName,
 // Return if parameter at given paramNr matches given taskIndex.
 bool pluginOptionalTaskIndexArgumentMatch(taskIndex_t   taskIndex,
                                           const String& string,
-                                          byte          paramNr);
+                                          uint8_t          paramNr);
 
 bool pluginWebformShowGPIOdescription(taskIndex_t taskIndex, const String& newline);
 

src/src/Commands/Blynk.cpp

@@ -143,7 +143,7 @@ bool Blynk_get(const String& command, controllerIndex_t controllerIndex, float *
       if (data && line.startsWith("["))
       {
         String strValue = line;
-        byte   pos      = strValue.indexOf('"', 2);
+        uint8_t   pos      = strValue.indexOf('"', 2);
         strValue = strValue.substring(2, pos);
         strValue.trim();
         *data   = 0.0f;

src/src/Commands/Common.cpp

@@ -62,7 +62,7 @@ const __FlashStringHelper * return_see_serial(struct EventStruct *event)
 String Command_GetORSetIP(struct EventStruct *event,
                           const String      & targetDescription,
                           const char         *Line,
-                          byte               *IP,
+                          uint8_t               *IP,
                           const IPAddress   & dhcpIP,
                           int                 arg)
 {

src/src/Commands/Common.h

@@ -22,7 +22,7 @@ bool   IsNumeric(const char *source);
 String Command_GetORSetIP(struct EventStruct *event,
                           const String      & targetDescription,
                           const char         *Line,
-                          byte               *IP,
+                          uint8_t               *IP,
                           const IPAddress&    dhcpIP,
                           int                 arg);
 

src/src/Commands/Diagnostic.cpp

@@ -159,7 +159,7 @@ const __FlashStringHelper * Command_Debug(struct EventStruct *event, const char
 
 const __FlashStringHelper * Command_logentry(struct EventStruct *event, const char *Line)
 {
-  byte level = LOG_LEVEL_INFO;
+  uint8_t level = LOG_LEVEL_INFO;
   // An extra optional parameter to set log level.
   if (event->Par2 > LOG_LEVEL_NONE && event->Par2 <= LOG_LEVEL_DEBUG_MORE) { level = event->Par2; }
   addLog(level, tolerantParseStringKeepCase(Line, 2));