Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Add prot argument to external_pointer() constructor #420

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

krlmlr
Copy link
Member

@krlmlr krlmlr commented Dec 7, 2024

From duckdb, see #401.

@krlmlr krlmlr requested a review from DavisVaughan December 7, 2024 19:43
krlmlr added a commit to duckdb/duckdb-r that referenced this pull request Dec 7, 2024
r-lib/cpp11#420

This reverts commit abcae40e2aff8b1664098294959b8df743605737.
krlmlr added a commit to duckdb/duckdb-r that referenced this pull request Dec 7, 2024
* Revert "ifndef"

This reverts commit 9afb676.

* Revert "Reapply 782630b minus the whitespace changes"

This reverts commit f05d56c.

* Reapply "Revert a011d42"

This reverts commit fe615c4.

* Reapply "Revert fb18a2d"

This reverts commit 6f4979d.

* Vendor cpp11 0.5.1

* Add `prot` argument to `external_pointer()` constructor

r-lib/cpp11#420

This reverts commit abcae40e2aff8b1664098294959b8df743605737.

* END_CPP11_EX()

r-lib/cpp11#421

* Protection

r-lib/cpp11#422

This reverts commit 1e181e5e5c2cf0a21906b49dafe80364c8ca3f2b.

* External pointer premature release

r-lib/cpp11#423

This reverts commit 1b698e533ea7c7003cb610a5f5f7e5a47966c59d.
krlmlr added a commit to duckdb/duckdb-r that referenced this pull request Dec 8, 2024
* Revert "ifndef"

This reverts commit 9afb676.

* Revert "Reapply 782630b minus the whitespace changes"

This reverts commit f05d56c.

* Reapply "Revert a011d42"

This reverts commit fe615c4.

* Reapply "Revert fb18a2d"

This reverts commit 6f4979d.

* Vendor cpp11 0.5.1

* Add `prot` argument to `external_pointer()` constructor

r-lib/cpp11#420

This reverts commit abcae40e2aff8b1664098294959b8df743605737.

* END_CPP11_EX()

r-lib/cpp11#421

* Protection

r-lib/cpp11#422

This reverts commit 1e181e5e5c2cf0a21906b49dafe80364c8ca3f2b.

* External pointer premature release

r-lib/cpp11#423

This reverts commit 1b698e533ea7c7003cb610a5f5f7e5a47966c59d.
krlmlr added a commit to duckdb/duckdb-r that referenced this pull request Dec 8, 2024
* Revert "ifndef"

This reverts commit 9afb676.

* Revert "Reapply 782630b minus the whitespace changes"

This reverts commit f05d56c.

* Reapply "Revert a011d42"

This reverts commit fe615c4.

* Reapply "Revert fb18a2d"

This reverts commit 6f4979d.

* Vendor cpp11 0.5.1

* Add `prot` argument to `external_pointer()` constructor

r-lib/cpp11#420

This reverts commit abcae40e2aff8b1664098294959b8df743605737.

* END_CPP11_EX()

r-lib/cpp11#421

* Protection

r-lib/cpp11#422

This reverts commit 1e181e5e5c2cf0a21906b49dafe80364c8ca3f2b.

* External pointer premature release

r-lib/cpp11#423

This reverts commit 1b698e533ea7c7003cb610a5f5f7e5a47966c59d.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant