Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

Fixed unsoundness of #derive(Clone) for FFI structs #882

Merged
merged 1 commit into from
Mar 4, 2022
Merged

Fixed unsoundness of #derive(Clone) for FFI structs #882

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
6 changes: 3 additions & 3 deletions src/ffi/generated.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

/// ABI-compatible struct for [`ArrowSchema`](https://arrow.apache.org/docs/format/CDataInterface.html#structure-definitions)
#[repr(C)]
#[derive(Debug, Clone)]
#[derive(Debug)]
pub struct ArrowSchema {
pub(super) format: *const ::std::os::raw::c_char,
pub(super) name: *const ::std::os::raw::c_char,
Expand All @@ -17,7 +17,7 @@ pub struct ArrowSchema {

/// ABI-compatible struct for [`ArrowArray`](https://arrow.apache.org/docs/format/CDataInterface.html#structure-definitions)
#[repr(C)]
#[derive(Debug, Clone)]
#[derive(Debug)]
pub struct ArrowArray {
pub(super) length: i64,
pub(super) null_count: i64,
Expand All @@ -33,7 +33,7 @@ pub struct ArrowArray {

/// ABI-compatible struct for [`ArrowArrayStream`](https://arrow.apache.org/docs/format/CStreamInterface.html).
#[repr(C)]
#[derive(Debug, Clone)]
#[derive(Debug)]
pub struct ArrowArrayStream {
pub(super) get_schema: ::std::option::Option<
unsafe extern "C" fn(
Expand Down