3

gcc 4.7.2 c89

你好，

我正在进行一些维护工作，并且我有以下类似函数的宏：

#define GET_ERROR() ((errno == 0) ? "None" : strerror(errno))

#define LOG_ERR(fmt, ...)                                               \
    fprintf(stderr, "[ERROR] %s:%d: error [%s] " fmt "\n", __func__, __LINE__, GET_ERROR(), ##__VA_ARGS__)

#define LOG_ASSERT(ARG, fmt, ...) do {                          \
        if(!(ARG)) {                                            \
            char arg_fmt[512];                                  \
            snprintf(arg_fmt, sizeof arg_fmt, "%s, ", #ARG);    \
            strcat(arg_fmt, fmt);                               \
            LOG_ERR(arg_fmt, ##__VA_ARGS__);                    \
            errno = 0;                                          \
            abort();                                            \
        }                                                       \
    } while(0)

LOG_ASSERT 将模拟一个断言函数。但是会包含一些来自 strerror errno 的额外信息。

我使用的是这样的：

LOG_ASSERT(msg_id != -1, "Failed to connect to the message queue [ %d ]", msg_id);

所以它应该像这样显示：

"msg_id != -1, Failed to connect to the message queue [ -1 ]"

我使用 strcat 在 fmt 的开头添加了“msg_id！= -1”的前缀。

但是，我得到了这些似乎无法修复的编译错误。我认为这很容易，如果我遗漏了一些简单的东西，我只想获得另一个想法。

error: expected ‘)’ before ‘arg_fmt’
warning: format ‘%s’ expects a matching ‘char *’ argument [-Wformat]
warning: format ‘%d’ expects a matching ‘int’ argument [-Wformat]
warning: format ‘%s’ expects a matching ‘char *’ argument [-Wformat]

我知道有些人可能不同意这个宏，但这是我必须维护的。

非常感谢您的任何建议

4

1 回答 1

4

据我所知，您只能连接常量字符串文字，而不是 char 数组，这是您的问题：

fprintf(stderr, "[ERROR] %s:%d: error [%s] " fmt "\n", __func__, ....

fmt是一个字符数组，您将需要另一个缓冲区来复制格式，或者您可以拆分printf

#define LOG_ERR(fmt, ...)                                               \
    fprintf(stderr, "[ERROR] %s:%d: error [%s] ", __func__, __LINE__, GET_ERROR());\
    fprintf(stderr, fmt, ##__VA_ARGS__)
于 2012-12-24T11:15:46.297 回答