G++-11销毁顺序由G++9更改
我们有以下代码(它更复杂,我试着做一个最小的例子)。
#include <iostream>
#include <vector>
#include <string>
#include <memory>
template<typename T>
struct use_type
{
use_type(T& v) : value(v) {}
T& value;
};
template<typename T>
use_type<T> use(T& value) { return use_type<T>(value); }
template<typename T>
use_type<T> use(const T& value) { return use_type<T>(const_cast<T&>(value)); }
template<typename T>
struct printer_helper
{
void use(const use_type<T>& use) { uses.push_back(use); }
void final_action()
{
for (const auto& use : uses)
{
std::cout << use.value.size() << std::endl;
for (const auto& v : use.value)
{
std::cout << v << ",";
}
std::cout << std::endl;
}
}
std::vector<use_type<T>> uses;
};
template<typename T>
struct printer
{
printer() { helper = new printer_helper<T>(); }
printer<T>& operator , (const use_type<T>& t)
{
helper->use(t);
return *this;
}
~printer()
{
helper->final_action();
delete helper;
}
printer(const printer&) = delete;
printer& operator =(const printer&) = delete;
printer(printer&&) = default;
printer& operator =(printer&&) = default;
printer_helper<T>* helper;
};
template<typename T>
struct printer_creator
{
printer<T> operator << (const char*) { return {}; }
};
int main()
{
using vec = std::vector<std::string>;
{
/*vec v1{"abc", "bcd"};
vec v2{"new", "old", "real"};*/
printer_creator<vec> p;
p << "", use(vec{"abc", "bcd"}), use(vec{"new", "old", "real"});
//p << p, use(v1), use(v2);
}
}
这段代码在 g++-11 之前的 g++ 上工作得很好。使用 g++-11,它在输出中要么是段错误要么是垃圾。是UB吗?如果是的话,也许您可以告诉我,哪些 gcc 更改使其失败并在可能的情况下引用标准?
现场示例:
克++-10
克++-11
回答
临时向量在创建它们的行的末尾被破坏。p在其作用域结束时(即在向量之后)被破坏。由于p使用对现在悬空的向量的引用的析构函数,您的代码具有未定义的行为。
如果它曾经工作过,那纯粹是“运气”,向量析构函数使向量处于可用状态。
回答
在
p << "", use(vec{"abc", "bcd"}), use(vec{"new", "old", "real"});
你创建了 3 个临时工
printer<vec>来自p << "")vec{"abc", "bcd"}vec{"new", "old", "real"}
它们之间没有顺序(见下面的注释),所以它们可以按任何顺序创建。
只有从左到右的破坏是正确的(所以从右到左的构造),其他顺序将使用悬空引用并且是 UB。而且由于您没有订单保证......
注:
我们通过重载失去短路operator &&/ operator ||,
我们失去了通过重载左到右的评估顺序operator,