Should the default `SEXP` in `r_vector` be a typed vector of length 0 rather than `R_NilValue`?

Abierto

#392 abierto el 24/8/2024 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad: 5/5
Tiempo estimado: Más de una semana
Aptitud para principiantes: 35/100

Tipo de issue: Nueva funcionalidad
Claridad: Bastante claro
Estado de actividad: Estancado
Stack tecnológico: cpp, r
Área: backend

Línea de trabajo

Start at r_vector's default construction and data_ handling, then inspect how default-constructed vectors currently behave when attributes are assigned. Evaluate the proposed typed, length-zero SEXP against the allocation and noexcept trade-offs, and validate that default construction supports attributes without requiring R_NilValue handling.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

i.e. to support things like https://github.com/r-lib/cpp11/issues/315 where you default construct the vector and immediately try and put attributes on it.

Would also simplify internal code that currently has to account for the possibility of R_NilValue as data_ quite a bit.

We'd have to write our own default constructor but I think that's totally possible?

A downside of this is that the default constructor would have to allocate and would no longer be noexcept or as "cheap", but I can't see that mattering much because you eventually allocate anyways?

Lenguaje dominante: C++
Estrellas: 224
Forks: 52
Métricas de merge de PR: Sin PR fusionados en 30 d

Guía de contribución

Abrir la guía de contribución

Primeros pasos

Lee el issue completo y luego la guía de contribución del proyecto.
Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
Haz un fork del repositorio y trabaja en una rama.
Abre un pull request que haga referencia al número del issue.

Más de r-lib/cpp11

Dimension error with `r_vector` Abierto

Dificultad 3/5 1-2 días Aptitud para principiantes 35/100

r-lib/cpp11#462 · 1 comentario · Abierto el 22/7/2025
`as_sexp_strings()` could use a single `unwind_protect()` around the loop Abierto

Dificultad 3/5 1-2 días Aptitud para principiantes 48/100

r-lib/cpp11#460 · Abierto el 14/5/2025
Making things a little bit faster Abierto

Dificultad 5/5 Más de una semana Aptitud para principiantes 20/100

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

Dificultad 3/5 1-2 días Aptitud para principiantes 42/100

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

Dificultad 3/5 1-2 días Aptitud para principiantes 45/100

r-lib/cpp11#453 · 1 comentario · Abierto el 26/4/2025

Todos los issues de r-lib/cpp11

Issues similares

An automatic array with a negative run-time extent has a negative size instead of zero Abierto

Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100

lfortran/lfortran#13475 · Abierto el 24/9/2026
Use ament_cmake_ros_core::ament_ros_defaults instead of setting CMAKE_C_STANDARD or CMAKE_CXX_STANDARD Abierto

good first issue

Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100

ros2/message_filters#338 · Abierto el 24/9/2026
Needs documentation update: Qt6 install or build instructions are not provided Abierto

Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100

subsurface/subsurface#4984 · Abierto el 24/9/2026
[Bug] Desktop: DataChannel onBufferedAmountLow / onBufferedAmountChange never fires — missing C++ OnBufferedAmountChange override Abierto

Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100

flutter-webrtc/flutter-webrtc#2206 · Abierto el 24/9/2026
litertlm-android AAR ships no consumer ProGuard rules → "mid == null" SIGABRT in minified apps Abierto

Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100

google-ai-edge/LiteRT-LM#3739 · Abierto el 24/9/2026

Más issues de C++