不同编译器的C++ & varargs

Question

我正在尝试重新编译一个用GCC编写的C++项目，它给我带来了varargs方法的一些问题。

NB：该项目还应该使用非c++-11兼容的编译器进行编译，因此我不能在这里使用C++11语法。

GCC不加警告地接受以下代码(1)：

void Set(TreeIter &it, ...) {
    va_list va;
    va_start(va, it);
    gtk_list_store_set_valist(*this, const_cast<TreeIter *>(&it), va);
    va_end(va);
}

和(2)：

void AddTail(...) {
    TreeIter it;
    gtk_list_store_append(*this, &it);
    va_list va;
    va_start(va, this);
    gtk_list_store_set_valist(*this, &it, va);
    va_end(va);
}

当CLANG对两者发出警告时：

(1)：

 ./ootree.h:444:30: warning: 'va_start' has undefined behavior with reference types [-Wvarargs]

(2)：

 ./ootree.h:476:30: warning: second parameter of 'va_start' not last named argument [-Wvarargs]

我可以轻松地用一个副本替换TreeIter引用(标准似乎要求我这么做)，但我找不到一种简单的方法来修复(2)，下面是一个显示行为的最小独立示例，这个程序不仅适用于gcc，而且也适用于clang，但是这些警告非常可怕.

#include <stdio.h>
#include <stdarg.h>

struct T {
    void print(...) {
        va_list va;
        va_start(va, this);
        vprintf("%d %d %d\n", va);
        va_end(va);
    }
};

int main() {
    T().print(2, 4, 6);
}

Answer 1

(2)的一个解决方案是显式地接受前2个参数，可能会提供一个1参数重载：

void AddTail(gint column, GValue *value, ...) {
    TreeIter it;
    gtk_list_store_append(*this, &it);
    gtk_list_store_set_value(*this, &it, column, value);
    va_list va;
    va_start(va, value);
    gtk_list_store_set_valist(*this, &it, va);
    va_end(va);
}

void AddTail(gint column) {
    TreeIter it;
    gtk_list_store_append(*this, &it);
    assert(column == -1);
}

Answer 2

你可以将(2)改为

static void AddTail(T* that, ...) {
    TreeIter it;
    gtk_list_store_append(*that, &it);
    va_list va;
    va_start(va, that);
    gtk_list_store_set_valist(*that, &it, va);
    va_end(va);
}

并将调用从T().print(2, 4, 6);更改为T::print(T(), 2, 4, 6);。

甚至在第一个参数中添加一个虚拟标记：

struct variadic_tag {};

void AddTail(variadic_tag, ...) {
    TreeIter it;
    gtk_list_store_append(*that, &it);
    va_list va;
    va_start(va, that);
    gtk_list_store_set_valist(*that, &it, va);
    va_end(va);
}

并将T().print(2, 4, 6);更改为T().print(variadic_tag{}, 2, 4, 6);。