From mboxrd@z Thu Jan 1 00:00:00 1970 From: Adrien Mazarguil Subject: [PATCH 10/10] lib: hide static functions that are never defined Date: Tue, 5 Apr 2016 16:08:10 +0200 Message-ID: <1459865290-10248-11-git-send-email-adrien.mazarguil@6wind.com> References: <1459865290-10248-1-git-send-email-adrien.mazarguil@6wind.com> Cc: Bruce Richardson , Thomas Monjalon To: dev@dpdk.org Return-path: Received: from mail-wm0-f50.google.com (mail-wm0-f50.google.com [74.125.82.50]) by dpdk.org (Postfix) with ESMTP id 9BBD62C60 for ; Tue, 5 Apr 2016 16:08:46 +0200 (CEST) Received: by mail-wm0-f50.google.com with SMTP id 191so25939900wmq.0 for ; Tue, 05 Apr 2016 07:08:46 -0700 (PDT) In-Reply-To: <1459865290-10248-1-git-send-email-adrien.mazarguil@6wind.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Arch-specific functions not defined for all architectures (missing on x86 in this case) and not used anywhere should not expose a prototype. This commit prevents the following errors: error: `rte_mov48' declared `static' but never defined error: `rte_memcpy_func' declared `static' but never defined Signed-off-by: Adrien Mazarguil --- lib/librte_eal/common/include/generic/rte_memcpy.h | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/lib/librte_eal/common/include/generic/rte_memcpy.h b/lib/librte_eal/common/include/generic/rte_memcpy.h index 03e8477..9f70d24 100644 --- a/lib/librte_eal/common/include/generic/rte_memcpy.h +++ b/lib/librte_eal/common/include/generic/rte_memcpy.h @@ -64,6 +64,8 @@ rte_mov16(uint8_t *dst, const uint8_t *src); static inline void rte_mov32(uint8_t *dst, const uint8_t *src); +#ifdef __DOXYGEN__ + /** * Copy 48 bytes from one location to another using optimised * instructions. The locations should not overlap. @@ -76,6 +78,8 @@ rte_mov32(uint8_t *dst, const uint8_t *src); static inline void rte_mov48(uint8_t *dst, const uint8_t *src); +#endif /* __DOXYGEN__ */ + /** * Copy 64 bytes from one location to another using optimised * instructions. The locations should not overlap. @@ -132,13 +136,12 @@ rte_mov256(uint8_t *dst, const uint8_t *src); static void * rte_memcpy(void *dst, const void *src, size_t n); -#endif /* __DOXYGEN__ */ - /* * memcpy() function used by rte_memcpy macro */ static inline void * rte_memcpy_func(void *dst, const void *src, size_t n) __attribute__((always_inline)); +#endif /* __DOXYGEN__ */ #endif /* _RTE_MEMCPY_H_ */ -- 2.1.4