[opensm] cl_vector_copy16/32/64 should be static functions

Commit Message

Honggang LI Sept. 28, 2018, 10:23 a.m. UTC

From: Honggang Li <[e-mail]>

Signed-off-by: Honggang Li <[e-mail]>
---
 complib/cl_vector.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

Comments

Hal Rosenstock Sept. 28, 2018, 2:22 p.m. UTC | #1

On 9/28/2018 6:23 AM, Honggang LI wrote:
> From: Honggang Li <[e-mail]>
> 
> Signed-off-by: Honggang Li <[e-mail]>

Thanks. Applied with some formatting changes as well as corresponding
change to complib/libosmcomp.map

-- Hal

Patch

diff --git a/complib/cl_vector.c b/complib/cl_vector.c
index f309a098..374dc2e2 100644
--- a/complib/cl_vector.c
+++ b/complib/cl_vector.c
@@ -118,7 +118,7 @@  static void cl_vector_copy8(OUT void *const p_dest,
  *	None
  *
  */
-void cl_vector_copy16(OUT void *const p_dest,
+static void cl_vector_copy16(OUT void *const p_dest,
 		      IN const void *const p_src, IN const size_t size)
 {
 	CL_ASSERT(size == sizeof(uint16_t));
@@ -143,7 +143,7 @@  void cl_vector_copy16(OUT void *const p_dest,
  *	None
  *
  */
-void cl_vector_copy32(OUT void *const p_dest,
+static void cl_vector_copy32(OUT void *const p_dest,
 		      IN const void *const p_src, IN const size_t size)
 {
 	CL_ASSERT(size == sizeof(uint32_t));
@@ -168,7 +168,7 @@  void cl_vector_copy32(OUT void *const p_dest,
  *	None
  *
  */
-void cl_vector_copy64(OUT void *const p_dest,
+static void cl_vector_copy64(OUT void *const p_dest,
 		      IN const void *const p_src, IN const size_t size)
 {
 	CL_ASSERT(size == sizeof(uint64_t));