simplify three false conditions for r_vector string

Đang mở

#431 mở ngày 1/1/2025 0 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ó: 2/5
Thời gian dự kiến: 1-3 giờ
Mức phù hợp với người mới: 42/100

Loại issue: Tái cấu trúc
Độ rõ ràng: Khá rõ ràng
Mức độ hoạt động: Đình trệ
Công nghệ: cpp, r
Lĩnh vực: backend

Hướng nghiên cứu

Start in inst/include/cpp11/r_vector.hpp around line 1395 and inspect the three conditions discussed in the issue. Compare them with the proposed constructibility and std::string checks, then run the existing test suite to verify equivalent behavior. Done means the condition is simpler without changing its type-selection behavior.

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

Mô tả

Is there a better condition we could use, e.g. assert something true rather than three things false? https://github.com/r-lib/cpp11/blob/main/inst/include/cpp11/r_vector.hpp#L1395

Still using 3 conditions, I think it is simpler to ensure that C is not constructible from SEXP, and neither C nor T is a std::string.

Ngôn ngữ chính: C++
Star: 224
Fork: 52
Chỉ số merge pull request: Không có pull request nào được merge trong 30 ngày

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
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.
Fork repository và làm thay đổi trên một nhánh.
Mở pull request có tham chiếu số hiệu của issue.

Issue khác của r-lib/cpp11

Dimension error with `r_vector` Đang mở

Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100

r-lib/cpp11#462 · 1 bình luận · Mở ngày 22/7/2025
`as_sexp_strings()` could use a single `unwind_protect()` around the loop Đang mở

Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 48/100

r-lib/cpp11#460 · Mở ngày 14/5/2025
Making things a little bit faster Đang mở

Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 20/100

r-lib/cpp11#459 · Mở ngày 10/5/2025
cpp_source fails to compile indented void-returning function Đang mở

Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 42/100

r-lib/cpp11#455 · 1 bình luận · Mở ngày 6/5/2025
Values added to a vector with `push_back` cannot be used immediately Đang mở

Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 45/100

r-lib/cpp11#453 · 1 bình luận · Mở ngày 26/4/2025

Tất cả issue của r-lib/cpp11

Issue tương tự

[Bug] Desktop: DataChannel onBufferedAmountLow / onBufferedAmountChange never fires — missing C++ OnBufferedAmountChange override Đang mở

Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100

flutter-webrtc/flutter-webrtc#2206 · Mở ngày 24/9/2026
litertlm-android AAR ships no consumer ProGuard rules → "mid == null" SIGABRT in minified apps Đang mở

Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100

google-ai-edge/LiteRT-LM#3739 · Mở ngày 24/9/2026
[GLib] Rename run-end encoded array values accessor to physical_values Đang mở

Component: GLib

Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100

apache/arrow#51479 · Mở ngày 24/9/2026
[Windows] Renderer crash (STATUS_BREAKPOINT) with font-palette override-colors on COLR web fonts: new[]/delete mismatch in Skia DWriteFontTypeface Đang mở

Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100

brave/brave-browser#59300 · Mở ngày 24/9/2026
Mute ydb/tests/functional/dstool/test_canonical_requests.py.Test.test_group_take_snapshot in main Đang mở

ai_reviewed

Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100

ydb-platform/ydb#53974 · 3 bình luận · Mở ngày 24/9/2026

Thêm issue về C++