External pointers and attributes
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 50/100
Línea de trabajo
Start with cpp11/external_pointer.hpp and the R reproduction in the issue; compare the vendored cpp11 implementation with the referenced DuckDB commit. Confirm that moving an external pointer preserves its attributes, then rerun the supplied example and verify the third and fourth outputs retain the class attribute.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Invoking the move constructor of external_pointer<> clears attributes on an external pointer object.
I would expect the third and fourth output to be identical to the second output. A possible fix is https://github.com/duckdb/duckdb/pull/6526/commits/12f6f10c12edda4d1153777d7a172279be11191c (this project uses a vendored version of cpp11).
library(cpp11)
cpp_source(
code = '
#include "cpp11/external_pointer.hpp"
#include "cpp11/strings.hpp"
template <class X>
cpp11::external_pointer<int> test_int() {
X a = 0;
auto p = cpp11::external_pointer<X>(&a, false);
Rf_PrintValue(p);
Rf_setAttrib(p, R_ClassSymbol, cpp11::writable::strings({"foo"}));
Rf_PrintValue(p);
return std::move(p);
}
[[cpp11::register]]
SEXP test() {
auto p = cpp11::sexp(test_int<int>());
Rf_PrintValue(p);
return p;
}
'
)
out <- test()
#> <pointer: 0x16f0af42c>
#> <pointer: 0x16f0af42c>
#> attr(,"class")
#> [1] "foo"
#> <pointer: 0x16f0af42c>
out
#> <pointer: 0x16f0af42c>
Created on 2023-03-02 with reprex v2.0.2
- Lenguaje dominante
- C++
- Estrellas
- 225
- Forks
- 52
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer 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
-
Dificultad 3/5 1-2 días Aptitud para principiantes 48/100
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 20/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 42/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
Todos los issues de r-lib/cpp11
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
ws_bridge: stripping format=evr for matchmaker connections can concatenate the path and queryAbiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 73/100
EchoTools/nevr-runtime#116 ·
Los mantenedores suelen responder en 1 día
-
code-quality libc++
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
llvm/llvm-project#229284 ·
Los mantenedores suelen responder en 1 día
-
test-issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
llvm/offload-test-suite#1557 ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día