simplify three false conditions for r_vector string

未关闭

#431 创建于 2025年1月1日 0 条评论 0 个 reaction 已指派 0 人在 GitHub 查看

还没有人认领这个 Issue。

评估

难度: 2/5
预计耗时: 1-3 小时
新手友好度: 42/100

Issue 类型: 重构
描述清晰度: 基本清楚
活跃度: 停滞
技术栈: cpp, r
领域: backend

调研方向

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.

由索引模型根据 Issue 内容生成。

描述

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.

主要语言: C++
星标: 224
派生: 52
PR 合并指标: 30 天内没有已合并 PR

贡献指南

打开贡献指南

从这里开始

先读完整个 Issue，再读项目的贡献指南。
在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
Fork 仓库，在一个分支上完成修改。
提交 Pull Request，并在描述里引用这个 Issue 编号。

r-lib/cpp11 的其他 Issue

Dimension error with `r_vector` 未关闭

难度 3/5 1-2 天新手友好度 35/100

r-lib/cpp11#462 · 1 条评论 · 2025年7月22日创建
`as_sexp_strings()` could use a single `unwind_protect()` around the loop 未关闭

难度 3/5 1-2 天新手友好度 48/100

r-lib/cpp11#460 · 2025年5月14日创建
Making things a little bit faster 未关闭

难度 5/5 一周以上新手友好度 20/100

r-lib/cpp11#459 · 2025年5月10日创建
cpp_source fails to compile indented void-returning function 未关闭

难度 3/5 1-2 天新手友好度 42/100

r-lib/cpp11#455 · 1 条评论 · 2025年5月6日创建
Values added to a vector with `push_back` cannot be used immediately 未关闭

难度 3/5 1-2 天新手友好度 45/100

r-lib/cpp11#453 · 1 条评论 · 2025年4月26日创建

查看 r-lib/cpp11 的全部 Issue

相似的 Issue

Use 4:4:4 for multichannel AVIF and HEIF gain maps 未关闭

难度 2/5 1-3 小时新手友好度 70/100

google/libultrahdr#485 · 2026年9月24日创建
Missing tile warnings in TileMap dock display with wrong texture filter 未关闭

难度 2/5 1-3 小时新手友好度 75/100

godotengine/godot#123776 · 2026年9月23日创建
if(cond,a,b) treats any nonzero cond as true in every bngsim backend (ExprTk interpreter, C codegen, SSA), which contradicts docs/reference/expressions.md ('true when > 0.5') and BNG's run_network. A non-boolean condition in (-inf,0) or... 未关闭

bug

难度 1/5 1 小时以内新手友好度 60/100

lanl/bngsim#774 · 2026年9月23日创建
Delete ament_export_include_directories() and ament_export_libraries() calls 未关闭

good first issue

难度 1/5 1 小时以内新手友好度 90/100

ros2/rosbag2#2509 · 2026年9月23日创建
Delete ament_export_include_directories() and ament_export_libraries() calls 未关闭

good first issue

难度 2/5 1-3 小时新手友好度 75/100

ros2/common_interfaces#344 · 2026年9月23日创建

更多 C++ Issue