[dpdk-dev,2/2] cryptodev: fix C11 extensions in exported header
Checks
Commit Message
Fix issues reported by check-includes.sh:
rte_crypto_sym.h:586:3: error: anonymous structs are a C11 extension
[-Werror,-Wc11-extensions]
rte_crypto_sym.h:510:2: error: anonymous unions are a C11 extension
[-Werror,-Wc11-extensions]
Fixes: b59502a5e3d0 ("cryptodev: add AEAD parameters in crypto operation")
Cc: Pablo de Lara <pablo.de.lara.guarch@intel.com>
Signed-off-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
---
lib/librte_cryptodev/rte_crypto_sym.h | 1 +
1 file changed, 1 insertion(+)
Comments
> -----Original Message-----
> From: Adrien Mazarguil [mailto:adrien.mazarguil@6wind.com]
> Sent: Friday, August 4, 2017 12:56 PM
> To: Thomas Monjalon <thomas@monjalon.net>
> Cc: dev@dpdk.org; De Lara Guarch, Pablo
> <pablo.de.lara.guarch@intel.com>
> Subject: [PATCH 2/2] cryptodev: fix C11 extensions in exported header
>
> Fix issues reported by check-includes.sh:
>
> rte_crypto_sym.h:586:3: error: anonymous structs are a C11 extension
> [-Werror,-Wc11-extensions]
> rte_crypto_sym.h:510:2: error: anonymous unions are a C11 extension
> [-Werror,-Wc11-extensions]
>
> Fixes: b59502a5e3d0 ("cryptodev: add AEAD parameters in crypto
> operation")
> Cc: Pablo de Lara <pablo.de.lara.guarch@intel.com>
>
> Signed-off-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
Patch looks good, although check-git-log.sh complains about it:
Wrong tag:
Fix issues reported by check-includes.sh:
Apart from this.
Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
@@ -507,6 +507,7 @@ struct rte_crypto_sym_op {
/**< Session-less API crypto operation parameters */
};
+ RTE_STD_C11
union {
struct {
struct {