diff mbox series

[01/15] eal: add macro to mark macros as deprecated

Message ID	20200911190701.29171-2-stephen@networkplumber.org (mailing list archive)
State	Superseded, archived
Headers	From: Stephen Hemminger <stephen@networkplumber.org> To: dev@dpdk.org Cc: Stephen Hemminger <stephen@networkplumber.org> Date: Fri, 11 Sep 2020 12:06:47 -0700 Message-Id: <20200911190701.29171-2-stephen@networkplumber.org> In-Reply-To: <20200911190701.29171-1-stephen@networkplumber.org> References: <20200911190701.29171-1-stephen@networkplumber.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [dpdk-dev] [PATCH 01/15] eal: add macro to mark macros as deprecated Precedence: list Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org>
Series	Replace terms master/slave lcore with main/worker lcore \| [00/15] Replace terms master/slave lcore with main/worker lcore [01/15] eal: add macro to mark macros as deprecated [02/15] eal: rename lcore word choices [03/15] rte_power: replace rte_master_lcore with rte_main_lcore [04/15] drivers: replace master lcore with main lcore [05/15] doc: replace master lcore with main lcore [06/15] app/test: replace uses of master/slave [07/15] app/test-pmd: replace master lcore with main lcore [08/15] app/test-eventdev: replace use of master lcore with main lcore [09/15] app: replace references to master/slave [10/15] examples: replace use of master with main [11/15] examples/multi_process: replace references to master/slave [12/15] examples/performance-thread: replace reference to master lcore [13/15] examples/qos: replace references to master [14/15] examples/ipsec-secgw: replace master lcore with main lcore [15/15] examples: replace master lcore with main lcore

Checks

Context	Check	Description
ci/checkpatch	success	coding style OK

Commit Message

Stephen Hemminger Sept. 11, 2020, 7:06 p.m. UTC

  Add a macro that causes GCC and CLANG to emit a warning when
a deprecated macro is used.

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 lib/librte_eal/include/rte_common.h | 11 +++++++++++
 1 file changed, 11 insertions(+)

Comments

Bruce Richardson Sept. 14, 2020, 8:43 a.m. UTC | #1

On Fri, Sep 11, 2020 at 12:06:47PM -0700, Stephen Hemminger wrote:
> Add a macro that causes GCC and CLANG to emit a warning when
> a deprecated macro is used.
> 
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> ---
>  lib/librte_eal/include/rte_common.h | 11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> diff --git a/lib/librte_eal/include/rte_common.h b/lib/librte_eal/include/rte_common.h
> index 8f487a563dfc..1ade136b64e4 100644
> --- a/lib/librte_eal/include/rte_common.h
> +++ b/lib/librte_eal/include/rte_common.h
> @@ -86,6 +86,17 @@ typedef uint16_t unaligned_uint16_t;
>  /******* Macro to mark functions and fields scheduled for removal *****/
>  #define __rte_deprecated	__attribute__((__deprecated__))
>  
> +/**
> + *  Macro to mark macros and defines scheduled for removal
> + */
> +#if defined(RTE_CC_GCC) || defined(RTE_CC_CLANG)
> +#define RTE_PRAGMA(x)  _Pragma(#x)
> +#define RTE_PRAGMA_WARNING(w) RTE_PRAGMA(GCC warning #w)
> +#define RTE_DEPRECATED(o, n)  RTE_PRAGMA_WARNING(#o is deprecated use #n)
> +#else
> +#define RTE_DEPRECATED(o, n)
> +#endif
> +
>  /**
>   * Mark a function or variable to a weak reference.
>   */

Great to see this, as this is something I was looking for myself recently
too!

Acked-by: Bruce Richardson <bruce.richardson@intel.com>

diff mbox series

Patch

diff --git a/lib/librte_eal/include/rte_common.h b/lib/librte_eal/include/rte_common.h
index 8f487a563dfc..1ade136b64e4 100644
--- a/lib/librte_eal/include/rte_common.h
+++ b/lib/librte_eal/include/rte_common.h
@@ -86,6 +86,17 @@  typedef uint16_t unaligned_uint16_t;
 /******* Macro to mark functions and fields scheduled for removal *****/
 #define __rte_deprecated	__attribute__((__deprecated__))
 
+/**
+ *  Macro to mark macros and defines scheduled for removal
+ */
+#if defined(RTE_CC_GCC) || defined(RTE_CC_CLANG)
+#define RTE_PRAGMA(x)  _Pragma(#x)
+#define RTE_PRAGMA_WARNING(w) RTE_PRAGMA(GCC warning #w)
+#define RTE_DEPRECATED(o, n)  RTE_PRAGMA_WARNING(#o is deprecated use #n)
+#else
+#define RTE_DEPRECATED(o, n)
+#endif
+
 /**
  * Mark a function or variable to a weak reference.
  */