Merge pull request #301 from andreagilardoni/const-parameters

Putting const modifier on function parameters

Author: pennam

src/utility/spi_drv.cpp

@@ -469,7 +469,7 @@ int SpiDrv::waitResponse(uint8_t cmd, uint8_t* numParamRead, uint8_t** params, u
     return 1;
 }
 
-void SpiDrv::sendParamNoLen(uint8_t* param, size_t param_len, uint8_t lastParam)
+void SpiDrv::sendParamNoLen(const uint8_t* param, size_t param_len, uint8_t lastParam)
 {
     size_t i = 0;
     // Send SPI paramLen
@@ -486,7 +486,7 @@ void SpiDrv::sendParamNoLen(uint8_t* param, size_t param_len, uint8_t lastParam)
         spiTransfer(END_CMD);
 }
 
-void SpiDrv::sendParam(uint8_t* param, uint8_t param_len, uint8_t lastParam)
+void SpiDrv::sendParam(const uint8_t* param, uint8_t param_len, uint8_t lastParam)
 {
     int i = 0;
     // Send SPI paramLen
@@ -537,7 +537,7 @@ uint16_t SpiDrv::readParamLen16(uint16_t* param_len)
 }
 
 
-void SpiDrv::sendBuffer(uint8_t* param, uint16_t param_len, uint8_t lastParam)
+void SpiDrv::sendBuffer(const uint8_t* param, uint16_t param_len, uint8_t lastParam)
 {
     uint16_t i = 0;
 

src/utility/spi_drv.h

@@ -83,19 +83,19 @@ class SpiDrv
 */
     static int waitResponse(uint8_t cmd, uint8_t* numParamRead, uint8_t** params, uint8_t maxNumParams);
 
-    static void sendParam(uint8_t* param, uint8_t param_len, uint8_t lastParam = NO_LAST_PARAM);
+    static void sendParam(const uint8_t* param, uint8_t param_len, uint8_t lastParam = NO_LAST_PARAM);
 
-    static void sendParamNoLen(uint8_t* param, size_t param_len, uint8_t lastParam = NO_LAST_PARAM);
+    static void sendParamNoLen(const uint8_t* param, size_t param_len, uint8_t lastParam = NO_LAST_PARAM);
 
-    static void sendParamLen8(uint8_t param_len);
+    static void sendParamLen8(const uint8_t param_len);
 
-    static void sendParamLen16(uint16_t param_len);
+    static void sendParamLen16(const uint16_t param_len);
 
     static uint8_t readParamLen8(uint8_t* param_len = NULL);
 
     static uint16_t readParamLen16(uint16_t* param_len = NULL);
 
-    static void sendBuffer(uint8_t* param, uint16_t param_len, uint8_t lastParam = NO_LAST_PARAM);
+    static void sendBuffer(const uint8_t* param, uint16_t param_len, uint8_t lastParam = NO_LAST_PARAM);
 
     static void sendParam(uint16_t param, uint8_t lastParam = NO_LAST_PARAM);
 

src/utility/wifi_drv.cpp

@@ -1543,7 +1543,7 @@ size_t WiFiDrv::prefStat() {
     return result;
 }
 
-size_t WiFiDrv::prefPut(const char * key, PreferenceType type, uint8_t value[], size_t len) {
+size_t WiFiDrv::prefPut(const char * key, PreferenceType type, const uint8_t value[], size_t len) {
     WAIT_FOR_SLAVE_SELECT();
 
     int commandSize = 4;
@@ -1555,7 +1555,7 @@ size_t WiFiDrv::prefPut(const char * key, PreferenceType type, uint8_t value[],
     SpiDrv::sendParam((uint8_t*)&type, 1);
     commandSize += 2;
 
-    SpiDrv::sendBuffer((uint8_t*)value, len, LAST_PARAM);
+    SpiDrv::sendBuffer(value, len, LAST_PARAM);
     commandSize += len + 1;
 
     // pad to multiple of 4

src/utility/wifi_drv.h

@@ -332,7 +332,7 @@ class WiFiDrv
     static bool prefRemove(const char * key);
     static size_t prefLen(const char * key);
     static size_t prefStat();
-    static size_t prefPut(const char * key, PreferenceType type, uint8_t value[], size_t len);
+    static size_t prefPut(const char * key, PreferenceType type, const uint8_t value[], size_t len);
     static size_t prefGet(const char * key, PreferenceType type, uint8_t value[], size_t len);
     static PreferenceType prefGetType(const char * key);