simplify three false conditions for r_vector string

オープン

#431 2025年1月1日に作成コメント 0 件リアクション 0 件担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度: 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 にコメントします — 二人が同じ作業をするのを防げます。
リポジトリをフォークし、ブランチを切って変更します。
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 1週間以上初心者へのやさしさ 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 をもっと見る