solve missing return code error

stickbreaker · web-flow · commit 123d25713969 · 2017-11-17T15:36:10.000-07:00
i2cFreeQueue() was not explicitly returning a result, this cause -Werror flag to abort compilation. As noted by @PhilColbert
diff --git a/cores/esp32/esp32-hal-i2c.c b/cores/esp32/esp32-hal-i2c.c
@@ -143,13 +143,15 @@ else {//not found
 }
 
 i2c_err_t i2cFreeQueue(i2c_t * i2c){
+i2c_err_t rc=I2C_ERROR_OK;
 if(i2c->dq!=NULL){
 // what about EventHandle?
   free(i2c->dq);
   i2c->dq = NULL;
   }
 i2c->queueCount=0;
 i2c->queuePos=0;
+return rc;
 }
 
 i2c_err_t i2cAddQueueWrite(i2c_t * i2c, uint16_t i2cDeviceAddr, uint8_t *dataPtr, uint16_t dataLen,bool sendStop,EventGroupHandle_t event){
@@ -1667,4 +1669,4 @@ if(i2c->intr_handle){
   i2c->intr_handle=NULL;
   }
 return I2C_ERROR_OK;
-}
+}

Original file line number	Diff line number	Diff line change
`@@ -143,13 +143,15 @@ else {//not found`
`143`	`143`	`}`
`144`	`144`
`145`	`145`	`i2c_err_t i2cFreeQueue(i2c_t * i2c){`
	`146`	`+i2c_err_t rc=I2C_ERROR_OK;`
`146`	`147`	`if(i2c->dq!=NULL){`
`147`	`148`	`// what about EventHandle?`
`148`	`149`	`free(i2c->dq);`
`149`	`150`	`i2c->dq = NULL;`
`150`	`151`	`}`
`151`	`152`	`i2c->queueCount=0;`
`152`	`153`	`i2c->queuePos=0;`
	`154`	`+return rc;`
`153`	`155`	`}`
`154`	`156`
`155`	`157`	`i2c_err_t i2cAddQueueWrite(i2c_t * i2c, uint16_t i2cDeviceAddr, uint8_t *dataPtr, uint16_t dataLen,bool sendStop,EventGroupHandle_t event){`
`@@ -1667,4 +1669,4 @@ if(i2c->intr_handle){`
`1667`	`1669`	`i2c->intr_handle=NULL;`
`1668`	`1670`	`}`
`1669`	`1671`	`return I2C_ERROR_OK;`
`1670`		`-}`
	`1672`	`+}`