`as_sexp_strings()` could use a single `unwind_protect()` around the loop

Aperta

#460 aperta il 14/5/2025 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà: 3/5
Tempo stimato: 1-2 giorni
Idoneità per principianti: 48/100

Tipo di issue: Refactoring
Chiarezza: Abbastanza chiara
Stato di attività: Ferma
Stack tecnologico: cpp, r
Ambito: tooling

Direzione di ricerca

Start in inst/include/cpp11/as.hpp at as_sexp_strings(), then read the unwind_protect example in vignettes/FAQ.Rmd. Compare the loop and the documented manual-call pitfalls; done means the conversion uses one protection around the loop without changing its behavior or safety.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

To convert from std::vector<std::string> to cpp11::strings you might do cpp11::strings(as_sexp(x)). That goes through as_sexp_strings() which looks like it could be a little more efficient. Rather than calling safe[Rf_mkCharCE] on each iteration, I think we could wrap the whole loop in a single unwind_protect().

https://github.com/r-lib/cpp11/blob/05c888b0c6f49e7b252b79b028e4719e6d5b299d/inst/include/cpp11/as.hpp#L287-L306

Something like
https://github.com/r-lib/cpp11/blob/05c888b0c6f49e7b252b79b028e4719e6d5b299d/vignettes/FAQ.Rmd#L499-L503

Be careful to avoid all the pitfalls of calling unwind_protect() manually that are mentioned in that vignette

Lingua principale: C++
Stelle: 224
Fork: 52
Metriche di merge delle PR: Nessuna PR unita negli ultimi 30g

Guida per i contributori

Apri la guida per i contributori

Come iniziare

Leggi tutta la issue e poi la guida ai contributi del progetto.
Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
Fai un fork del repository e lavora su un branch.
Apri una pull request che faccia riferimento al numero della issue.

Altre issue di r-lib/cpp11

Dimension error with `r_vector` Aperta

Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100

r-lib/cpp11#462 · 1 commento · Aperta il 22/7/2025
Making things a little bit faster Aperta

Difficoltà 5/5 Più di una settimana Idoneità per principianti 20/100

r-lib/cpp11#459 · Aperta il 10/5/2025
cpp_source fails to compile indented void-returning function Aperta

Difficoltà 3/5 1-2 giorni Idoneità per principianti 42/100

r-lib/cpp11#455 · 1 commento · Aperta il 6/5/2025
Values added to a vector with `push_back` cannot be used immediately Aperta

Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100

r-lib/cpp11#453 · 1 commento · Aperta il 26/4/2025
Global symbol visibility / deriving from a C++ class defined in a different shared library Aperta

Difficoltà 5/5 Più di una settimana Idoneità per principianti 30/100

r-lib/cpp11#452 · 1 commento · Aperta il 18/4/2025

Tutte le issue di r-lib/cpp11

Issue simili

[Bug] NULL pointer dereference in llm_running_callback (NGZA-AX-LLM #3) Aperta

Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100

AXERA-TECH/ax-llm#77 · Aperta il 22/9/2026
resume() sends a second HTTP response (400) after a successful resume Aperta

Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100

games-on-whales/wolf#509 · Aperta il 21/9/2026
z3.py omits additional domains of multidimensional arrays during printing Aperta

Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100

Z3Prover/z3#10889 · Aperta il 21/9/2026
pn7160: reader goes permanently deaf — post-read IRQ-low check fails reads that succeeded Aperta

Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100

esphome/esphome#19452 · Aperta il 21/9/2026
Eval bug: Ministral-3 tool-calls during reasoning are not processed Aperta

bug-unconfirmed

Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100

ggml-org/llama.cpp#29248 · Aperta il 21/9/2026

Altre issue su C++