Skip to content

Commit

Permalink
compressdev: fix missing space in log macro
Browse files Browse the repository at this point in the history
[ upstream commit dfb90fb ]

Building with clang on FreeBSD with chkincs enabled, we get the
following error about a missing space:

lib/compressdev/rte_compressdev_internal.h:25:58: error:
invalid suffix on literal;
C++11 requires a space between literal and identifier [-Wreserved-user-defined-literal]
        rte_log(RTE_LOG_ ## level, compressdev_logtype, "%s(): "fmt "\n", \

Adding in a space between the '"' and 'fmt' removes the error.

Fixes: ed7dd94 ("compressdev: add basic device management")

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
  • Loading branch information
bruce-richardson authored and bluca committed Mar 15, 2022
1 parent eca0c20 commit b16e422
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/librte_compressdev/rte_compressdev_internal.h
Expand Up @@ -22,7 +22,7 @@ extern "C" {
/* Logging Macros */
extern int compressdev_logtype;
#define COMPRESSDEV_LOG(level, fmt, args...) \
rte_log(RTE_LOG_ ## level, compressdev_logtype, "%s(): "fmt "\n", \
rte_log(RTE_LOG_ ## level, compressdev_logtype, "%s(): " fmt "\n", \
__func__, ##args)

/**
Expand Down

0 comments on commit b16e422

Please sign in to comment.