Register function with namespace

Aperta

#262 aperta il 4/2/2022 1 commento 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: 54/100

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

Direzione di ricerca

Start in R/register.R at the wrapper-generation code linked in the issue, then trace how decor records namespaced functions and how R-side functions are generated. Add or extend tests for registering foo::bar and verify that the generated extern "C" wrapper and R-side function use valid names. The existing comparison branch may provide useful context.

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

Descrizione

When registering a function in a namespace such as

[[cpp11::register]] int foo::bar(int a) {
  return a;
}

decor recognizes the function name as foo::bar and all would be fine (as far as I can tell), if the generated extern "C" wrapper

https://github.com/r-lib/cpp11/blob/c22dc9b611fe7d56ddfc57add2e01b1424869c0f/R/register.R#L171-L176

would not blindly forward the ::. Some further minor adjustments are required for the generation of the R-side functions. For the R-side function name, we could for example strip away anything that comes before the last scope resolution operator.

I started putting something together under https://github.com/r-lib/cpp11/compare/main...nbenn:main and I'm happy to clean this up a bit, add some tests and submit a PR.

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
`as_sexp_strings()` could use a single `unwind_protect()` around the loop Aperta

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

r-lib/cpp11#460 · Aperta il 14/5/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

Tutte le issue di r-lib/cpp11

Issue simili

Better wasm http timeout error message Aperta

enhancement

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

QuantStack/git2cpp#187 · Aperta il 22/9/2026
scheduled_tasks.hidden returns the task's Enabled state instead of Hidden (Windows) Aperta

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

osquery/osquery#9114 · Aperta il 22/9/2026
Total score collapses to ~0 when only a subset of benchmarks is run Aperta

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

mlcommons/mobile_app_open#1182 · Aperta il 22/9/2026
Exclude generated *.lock.yml workflow files from spelling checks Aperta

Needs-Triage

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

microsoft/winget-cli#6547 · Aperta il 22/9/2026
[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

Altre issue su C++