MISRA C fixes
authorRishabh Garg <rishabh@ti.com>
Thu, 14 Feb 2019 15:58:28 +0000 (21:28 +0530)
committerRishabh Garg <rishabh@ti.com>
Thu, 14 Feb 2019 17:11:50 +0000 (22:41 +0530)
- Changes in memset and memcpy APIs

Signed-off-by: Rishabh Garg <rishabh@ti.com>
include/fvid2_utils.h
src/fvid2_utils.c

index 021cda1f5cffdddbb0d064b0ac5a2862312fdc72..6e6c0857bf5a912dccc26ea34bda00496fddb0b5 100755 (executable)
@@ -217,9 +217,9 @@ typedef Fvid2Utils_Handle Fvid2Utils_QHandle;
  *  \param ch               Byte value to fill with.
  *  \param byteCount        Number of bytes to fill.
  *
- *  \return                 The destination memory pointer.
+ *  \return                 None.
  */
-void *Fvid2Utils_memset(void *mem, uint8_t ch, uint32_t byteCount);
+void Fvid2Utils_memset(void *mem, uint8_t ch, uint32_t byteCount);
 
 /**
  *  Fvid2Utils_memsetw
@@ -244,9 +244,9 @@ void *Fvid2Utils_memsetw(void *mem, uint32_t word, uint32_t wordCount);
  *  \param src              Source memory pointer.
  *  \param byteCount        Number of bytes to copy.
  *
- *  \return                 The destination memory pointer.
+ *  \return                 None
  */
-void *Fvid2Utils_memcpy(void *dest, const void *src, uint32_t byteCount);
+void Fvid2Utils_memcpy(void *dest, const void *src, uint32_t byteCount);
 
 /**
  *  Fvid2Utils_memcmp
@@ -284,9 +284,9 @@ int32_t Fvid2Utils_constructLinkList(Fvid2UtilsLinkListObj  *llobj,
  *
  *  \param llobj pointer to object of type Fvid2UtilsLinkListObj
  *
- *  \return                 Returns FVID2_SOK on success else returns error value.
+ *  \return                 None
  */
-int32_t Fvid2Utils_destructLinkList(Fvid2UtilsLinkListObj *llobj);
+void Fvid2Utils_destructLinkList(Fvid2UtilsLinkListObj *llobj);
 
 /**
  *  Fvid2Utils_linkNodePri
@@ -478,9 +478,9 @@ int32_t Fvid2Utils_constructQ(Fvid2UtilsLinkListObj *llobj);
  *
  * \param llobj pointer to object of type Fvid2UtilsLinkListObj
  *
- *  \return                 Returns FVID2_SOK on success else returns error value.
+ *  \return                 None
  */
-int32_t Fvid2Utils_destructQ(Fvid2UtilsLinkListObj *llobj);
+void Fvid2Utils_destructQ(Fvid2UtilsLinkListObj *llobj);
 
 /**
  *  Fvid2Utils_queue
index 07d01ed6cd221fdb17f0245b854fecb32e87cb3e..dca5d91f5aca32ebc8b36478266fe109e3047e2c 100644 (file)
@@ -127,12 +127,12 @@ int32_t Fvid2Utils_deInit(void *arg)
  *
  *  \return                 The destination memory pointer
  */
-void *Fvid2Utils_memset(void *mem, uint8_t ch, uint32_t byteCount)
+void Fvid2Utils_memset(void *mem, uint8_t ch, uint32_t byteCount)
 {
     /* NULL pointer check */
     GT_assert(Fvid2Trace, (NULL != mem));
 
-    return memset(mem, (int32_t) ch, byteCount);
+    memset(mem, (int32_t) ch, byteCount);
 }
 
 /**
@@ -175,13 +175,13 @@ void *Fvid2Utils_memsetw(void *mem, uint32_t word, uint32_t wordCount)
  *
  *  \return                 The destination memory pointer
  */
-void *Fvid2Utils_memcpy(void *dest, const void *src, uint32_t byteCount)
+void Fvid2Utils_memcpy(void *dest, const void *src, uint32_t byteCount)
 {
     /* NULL pointer check */
     GT_assert(Fvid2Trace, (NULL != dest));
     GT_assert(Fvid2Trace, (NULL != src));
 
-    return memcpy(dest, src, byteCount);
+    memcpy(dest, src, byteCount);
 }
 
 /**
@@ -250,9 +250,9 @@ int32_t Fvid2Utils_constructLinkList(Fvid2UtilsLinkListObj  *llobj,
  *
  *  \param llobj pointer to object of type Fvid2UtilsLinkListObj
  *
- *  \return                 Returns FVID2_SOK on success else returns error value.
+ *  \return                None
  */
-int32_t Fvid2Utils_destructLinkList(Fvid2UtilsLinkListObj *llobj)
+void Fvid2Utils_destructLinkList(Fvid2UtilsLinkListObj *llobj)
 {
     /* NULL pointer check */
     GT_assert(Fvid2Trace, (NULL != llobj));
@@ -261,8 +261,6 @@ int32_t Fvid2Utils_destructLinkList(Fvid2UtilsLinkListObj *llobj)
 
     /* Free link list object */
     Fvid2Utils_memset(llobj, 0, sizeof (Fvid2UtilsLinkListObj));
-
-    return (FVID2_SOK);
 }
 
 /**
@@ -713,11 +711,11 @@ int32_t Fvid2Utils_constructQ(Fvid2UtilsLinkListObj *llobj)
  *
  * \param llobj pointer to object of type Fvid2UtilsLinkListObj
  *
- *  \return                 Returns FVID2_SOK on success else returns error value.
+ *  \return                 None.
  */
-int32_t Fvid2Utils_destructQ(Fvid2UtilsLinkListObj *llobj)
+void Fvid2Utils_destructQ(Fvid2UtilsLinkListObj *llobj)
 {
-    return Fvid2Utils_destructLinkList(llobj);
+    Fvid2Utils_destructLinkList(llobj);
 }
 
 /**