PVS-Studio 是否知道前向声明的必要性?
Does PVS-Studio know about forward declaration necessity?
当不同的编译器表现不同时会出现问题。可惜,PVS没有告诉我下面的危险情况。
我有一些重载函数,但我忘记写前向声明。所以 Visual Studio 编译一个程序,其他编译器编译另一个程序。
我的职能:
// For simple types (i.e. int, bool, char) and plain old data
template <typename T>
void serialize(T pod, std::ostream& out) {
std::cout << "pod\n";
out.write(reinterpret_cast<const char*>(&pod), sizeof(T));
}
void serialize(const std::string& str, std::ostream& out) {
std::cout << "string\n";
serialize(str.size(), out);
out.write(str.data(), str.size());
}
template <typename T>
void serialize(const std::vector<T>& data, std::ostream& out) {
std::cout << "vector\n";
serialize(data.size(), out);
for (const T& elem : data) {
serialize(elem, out); // <== runs POD serialization for map :(
}
}
template <typename T1, typename T2>
void serialize(const std::map<T1, T2>& data, std::ostream& out) {
std::cout << "map\n";
serialize(data.size(), out);
for (const auto& p : data) {
serialize(p.first, out);
serialize(p.second, out);
}
}
我用代码测试这个:
std::vector<std::map<int, int>> v;
v.emplace_back();
v[0][1] = 2;
std::stringstream ss;
serialize(v, ss);
在 VS 中测试正常(调试和发布版本):
vector
pod [size of vector]
map
pod [size of map]
pod [key = 1]
pod [value = 2]
但是我的同事突然告诉我根本没有任何效果。
我已经在不同的编译器上进行了测试(感谢 Ideone 站点)。所有行为都是错误的(gcc 4.3.2、6.3;clang 4.0):
vector
pod [size of vector]
pod [map]
当然,在所有函数之前放置前向声明后,在所有编译器中一切都变得正确。
我要求实施关于前向声明的警告。
是的,在PVS-Studio中有这样的不足之处。随着时间的推移,我们会尽力修复它。
当不同的编译器表现不同时会出现问题。可惜,PVS没有告诉我下面的危险情况。
我有一些重载函数,但我忘记写前向声明。所以 Visual Studio 编译一个程序,其他编译器编译另一个程序。
我的职能:
// For simple types (i.e. int, bool, char) and plain old data
template <typename T>
void serialize(T pod, std::ostream& out) {
std::cout << "pod\n";
out.write(reinterpret_cast<const char*>(&pod), sizeof(T));
}
void serialize(const std::string& str, std::ostream& out) {
std::cout << "string\n";
serialize(str.size(), out);
out.write(str.data(), str.size());
}
template <typename T>
void serialize(const std::vector<T>& data, std::ostream& out) {
std::cout << "vector\n";
serialize(data.size(), out);
for (const T& elem : data) {
serialize(elem, out); // <== runs POD serialization for map :(
}
}
template <typename T1, typename T2>
void serialize(const std::map<T1, T2>& data, std::ostream& out) {
std::cout << "map\n";
serialize(data.size(), out);
for (const auto& p : data) {
serialize(p.first, out);
serialize(p.second, out);
}
}
我用代码测试这个:
std::vector<std::map<int, int>> v;
v.emplace_back();
v[0][1] = 2;
std::stringstream ss;
serialize(v, ss);
在 VS 中测试正常(调试和发布版本):
vector
pod [size of vector]
map
pod [size of map]
pod [key = 1]
pod [value = 2]
但是我的同事突然告诉我根本没有任何效果。
我已经在不同的编译器上进行了测试(感谢 Ideone 站点)。所有行为都是错误的(gcc 4.3.2、6.3;clang 4.0):
vector
pod [size of vector]
pod [map]
当然,在所有函数之前放置前向声明后,在所有编译器中一切都变得正确。
我要求实施关于前向声明的警告。
是的,在PVS-Studio中有这样的不足之处。随着时间的推移,我们会尽力修复它。