YAML::Nodes alias rather than moving when using std::vector::erase, std::iter_swap and similar methods.

Đang mở
  • Có gì không ổn?
#1,209 3 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
35/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Đình trệ
Công nghệ
cpp
Lĩnh vực
backend

Hướng nghiên cứu

Start by reproducing the failure with the vector erase test shown in the issue, then inspect the YAML::Node move constructor and move assignment behavior. The issue also mentions std::iter_swap and similar methods; verify whether those operations produce the same aliasing behavior. Done means erased and swapped vectors contain the expected distinct node values.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

The following unit test fails

#include "gtest/gtest.h"
#include <yaml-cpp/yaml.h>
#include <vector>

TEST(VectorErase, ProducesVectorContainingCorrectElements) {
    YAML::Node a;
    a["name"] = "a";
    YAML::Node b;
    b["name"] = "b";
    YAML::Node c;
    c["name"] = "c";
    std::vector<YAML::Node> vec{a, b, c};
    vec.erase(vec.begin());
    EXPECT_EQ(vec[0]["name"].as<std::string>(), "b");
}

When vec.erase(vec.begin()) is called, instead of vec containing names {"b", "c"}, it instead contains names {"c", "c"}.

My guess is that this is something to do with aliasing rather than copying when the vector elements are moved. This is likely an issue with the move constructor or move assignment of the Node.

I am using libcpp-yaml0.6, as I am currently limited to Ubuntu 20.04.

Ngôn ngữ chính
C++
Star
6.1k
Fork
2.2k
Merge trung bình
22 giờ 2 phút
Pull request đã merge (30 ngày)
1

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của jbeder/yaml-cpp

Tất cả issue của jbeder/yaml-cpp

Issue tương tự

Thêm issue về C++

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.