Skip to content

Commit ef5aab3

Browse files
BenFordTytheringtonBenFordTytherington
committed
Add Transitive casting and testing code
1 parent 68651a3 commit ef5aab3

File tree

15 files changed

+295
-176
lines changed

15 files changed

+295
-176
lines changed

‎crates/cxx-qt-gen/src/generator/rust/fragment.rs

+9-17
Original file line numberDiff line numberDiff line change
@@ -77,25 +77,17 @@ impl GeneratedRustFragment {
7777
unsafe fn #downcast_fn(base: *const #base_unqualified) -> *const #struct_name_unqualified;
7878
}
7979
}],
80-
cxx_qt_mod_contents: vec![
81-
parse_quote! {
82-
unsafe impl ::cxx_qt::Upcast<#base_qualified> for #struct_name {
83-
unsafe fn upcast_ptr(this: *const Self) -> *const #base_qualified {
84-
#upcast_fn_qualified(this)
85-
}
80+
cxx_qt_mod_contents: vec![parse_quote! {
81+
unsafe impl ::cxx_qt::casting::Upcast<#base_qualified> for #struct_name {
82+
unsafe fn upcast_ptr(this: *const Self) -> *const #base_qualified {
83+
#upcast_fn_qualified(this)
84+
}
8685

87-
unsafe fn from_base_ptr(base: *const #base_qualified) -> *const Self {
88-
#downcast_fn_qualified(base)
89-
}
86+
unsafe fn from_base_ptr(base: *const #base_qualified) -> *const Self {
87+
#downcast_fn_qualified(base)
9088
}
91-
},
92-
// Add back once we figure out the bug with QObject, for automatic transitive casts
93-
// parse_quote! {
94-
// unsafe impl ::cxx_qt::MainCast for #struct_name {
95-
// type Base = #base_qualified;
96-
// }
97-
// }
98-
],
89+
}
90+
}],
9991
})
10092
}
10193

‎crates/cxx-qt-gen/test_outputs/cfgs.rs

+4-4
Original file line numberDiff line numberDiff line change
@@ -668,7 +668,7 @@ cxx_qt::static_assertions::assert_eq_size!(
668668
cxx_qt::signalhandler::CxxQtSignalHandler<QObjectEnabledCxxQtSignalClosuresignal_enabled>,
669669
[usize; 2]
670670
);
671-
unsafe impl ::cxx_qt::Upcast<::cxx_qt::QObject> for ffi::QObjectEnabled {
671+
unsafe impl ::cxx_qt::casting::Upcast<::cxx_qt::QObject> for ffi::QObjectEnabled {
672672
unsafe fn upcast_ptr(this: *const Self) -> *const ::cxx_qt::QObject {
673673
ffi::cxx_qt_ffi_QObjectEnabled_upcastPtr(this)
674674
}
@@ -847,7 +847,7 @@ cxx_qt::static_assertions::assert_eq_size!(
847847
cxx_qt::signalhandler::CxxQtSignalHandler<QObjectDisabledCxxQtSignalClosuresignal_enabled>,
848848
[usize; 2]
849849
);
850-
unsafe impl ::cxx_qt::Upcast<::cxx_qt::QObject> for ffi::QObjectDisabled {
850+
unsafe impl ::cxx_qt::casting::Upcast<::cxx_qt::QObject> for ffi::QObjectDisabled {
851851
unsafe fn upcast_ptr(this: *const Self) -> *const ::cxx_qt::QObject {
852852
ffi::cxx_qt_ffi_QObjectDisabled_upcastPtr(this)
853853
}
@@ -878,7 +878,7 @@ impl ::cxx_qt::CxxQtType for ffi::QObjectDisabled {
878878
ffi::cxx_qt_ffi_QObjectDisabled_unsafeRustMut(self)
879879
}
880880
}
881-
unsafe impl ::cxx_qt::Upcast<::cxx_qt::QObject> for ffi::QObjectExternEnabled {
881+
unsafe impl ::cxx_qt::casting::Upcast<::cxx_qt::QObject> for ffi::QObjectExternEnabled {
882882
unsafe fn upcast_ptr(this: *const Self) -> *const ::cxx_qt::QObject {
883883
ffi::cxx_qt_ffi_QObjectExternEnabled_upcastPtr(this)
884884
}
@@ -1048,7 +1048,7 @@ cxx_qt::static_assertions::assert_eq_size!(
10481048
>,
10491049
[usize; 2]
10501050
);
1051-
unsafe impl ::cxx_qt::Upcast<::cxx_qt::QObject> for ffi::QObjectExternDisabled {
1051+
unsafe impl ::cxx_qt::casting::Upcast<::cxx_qt::QObject> for ffi::QObjectExternDisabled {
10521052
unsafe fn upcast_ptr(this: *const Self) -> *const ::cxx_qt::QObject {
10531053
ffi::cxx_qt_ffi_QObjectExternDisabled_upcastPtr(this)
10541054
}

‎crates/cxx-qt-gen/test_outputs/inheritance.rs

+3-3
Original file line numberDiff line numberDiff line change
@@ -130,7 +130,7 @@ mod inheritance {
130130
type QObject = cxx_qt::QObject;
131131
}
132132
}
133-
unsafe impl ::cxx_qt::Upcast<inheritance::QAbstractItemModel> for inheritance::MyObject {
133+
unsafe impl ::cxx_qt::casting::Upcast<inheritance::QAbstractItemModel> for inheritance::MyObject {
134134
unsafe fn upcast_ptr(this: *const Self) -> *const inheritance::QAbstractItemModel {
135135
inheritance::cxx_qt_ffi_MyObject_upcastPtr(this)
136136
}
@@ -158,15 +158,15 @@ impl ::cxx_qt::CxxQtType for inheritance::MyObject {
158158
inheritance::cxx_qt_ffi_MyObject_unsafeRustMut(self)
159159
}
160160
}
161-
unsafe impl ::cxx_qt::Upcast<::cxx_qt::QObject> for inheritance::QPushButton {
161+
unsafe impl ::cxx_qt::casting::Upcast<::cxx_qt::QObject> for inheritance::QPushButton {
162162
unsafe fn upcast_ptr(this: *const Self) -> *const ::cxx_qt::QObject {
163163
inheritance::cxx_qt_ffi_QPushButton_upcastPtr(this)
164164
}
165165
unsafe fn from_base_ptr(base: *const ::cxx_qt::QObject) -> *const Self {
166166
inheritance::cxx_qt_ffi_QPushButton_downcastPtr(base)
167167
}
168168
}
169-
unsafe impl ::cxx_qt::Upcast<inheritance::QPushButton> for inheritance::QPushButtonChild {
169+
unsafe impl ::cxx_qt::casting::Upcast<inheritance::QPushButton> for inheritance::QPushButtonChild {
170170
unsafe fn upcast_ptr(this: *const Self) -> *const inheritance::QPushButton {
171171
inheritance::cxx_qt_ffi_QPushButtonChild_upcastPtr(this)
172172
}

‎crates/cxx-qt-gen/test_outputs/invokables.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -326,7 +326,7 @@ impl cxx_qt::Threading for ffi::MyObject {
326326
pub struct MyObjectCxxQtThreadQueuedFn {
327327
inner: std::boxed::Box<dyn FnOnce(core::pin::Pin<&mut ffi::MyObject>) + Send>,
328328
}
329-
unsafe impl ::cxx_qt::Upcast<::cxx_qt::QObject> for ffi::MyObject {
329+
unsafe impl ::cxx_qt::casting::Upcast<::cxx_qt::QObject> for ffi::MyObject {
330330
unsafe fn upcast_ptr(this: *const Self) -> *const ::cxx_qt::QObject {
331331
ffi::cxx_qt_ffi_MyObject_upcastPtr(this)
332332
}

‎crates/cxx-qt-gen/test_outputs/passthrough_and_naming.rs

+5-5
Original file line numberDiff line numberDiff line change
@@ -648,7 +648,7 @@ cxx_qt::static_assertions::assert_eq_size!(
648648
cxx_qt::signalhandler::CxxQtSignalHandler<MyObjectCxxQtSignalClosureready>,
649649
[usize; 2]
650650
);
651-
unsafe impl ::cxx_qt::Upcast<ffi::QStringListModel> for ffi::MyObject {
651+
unsafe impl ::cxx_qt::casting::Upcast<ffi::QStringListModel> for ffi::MyObject {
652652
unsafe fn upcast_ptr(this: *const Self) -> *const ffi::QStringListModel {
653653
ffi::cxx_qt_ffi_MyObject_upcastPtr(this)
654654
}
@@ -822,7 +822,7 @@ cxx_qt::static_assertions::assert_eq_size!(
822822
cxx_qt::signalhandler::CxxQtSignalHandler<SecondObjectCxxQtSignalClosureready>,
823823
[usize; 2]
824824
);
825-
unsafe impl ::cxx_qt::Upcast<::cxx_qt::QObject> for ffi::SecondObject {
825+
unsafe impl ::cxx_qt::casting::Upcast<::cxx_qt::QObject> for ffi::SecondObject {
826826
unsafe fn upcast_ptr(this: *const Self) -> *const ::cxx_qt::QObject {
827827
ffi::cxx_qt_ffi_SecondObject_upcastPtr(this)
828828
}
@@ -850,7 +850,7 @@ impl ::cxx_qt::CxxQtType for ffi::SecondObject {
850850
ffi::cxx_qt_ffi_SecondObject_unsafeRustMut(self)
851851
}
852852
}
853-
unsafe impl ::cxx_qt::Upcast<::cxx_qt::QObject> for ffi::MyRustName {
853+
unsafe impl ::cxx_qt::casting::Upcast<::cxx_qt::QObject> for ffi::MyRustName {
854854
unsafe fn upcast_ptr(this: *const Self) -> *const ::cxx_qt::QObject {
855855
ffi::cxx_qt_ffi_MyCxxName_upcastPtr(this)
856856
}
@@ -878,15 +878,15 @@ impl ::cxx_qt::CxxQtType for ffi::MyRustName {
878878
ffi::cxx_qt_ffi_MyCxxName_unsafeRustMut(self)
879879
}
880880
}
881-
unsafe impl ::cxx_qt::Upcast<::cxx_qt::QObject> for ffi::QPushButton {
881+
unsafe impl ::cxx_qt::casting::Upcast<::cxx_qt::QObject> for ffi::QPushButton {
882882
unsafe fn upcast_ptr(this: *const Self) -> *const ::cxx_qt::QObject {
883883
ffi::cxx_qt_ffi_QPushButton_upcastPtr(this)
884884
}
885885
unsafe fn from_base_ptr(base: *const ::cxx_qt::QObject) -> *const Self {
886886
ffi::cxx_qt_ffi_QPushButton_downcastPtr(base)
887887
}
888888
}
889-
unsafe impl ::cxx_qt::Upcast<::cxx_qt::QObject> for ffi::ExternObject {
889+
unsafe impl ::cxx_qt::casting::Upcast<::cxx_qt::QObject> for ffi::ExternObject {
890890
unsafe fn upcast_ptr(this: *const Self) -> *const ::cxx_qt::QObject {
891891
ffi::cxx_qt_ffi_ExternObjectCpp_upcastPtr(this)
892892
}

‎crates/cxx-qt-gen/test_outputs/properties.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -1052,7 +1052,7 @@ cxx_qt::static_assertions::assert_eq_size!(
10521052
cxx_qt::signalhandler::CxxQtSignalHandler<MyObjectCxxQtSignalClosuremy_on_changed>,
10531053
[usize; 2]
10541054
);
1055-
unsafe impl ::cxx_qt::Upcast<::cxx_qt::QObject> for ffi::MyObject {
1055+
unsafe impl ::cxx_qt::casting::Upcast<::cxx_qt::QObject> for ffi::MyObject {
10561056
unsafe fn upcast_ptr(this: *const Self) -> *const ::cxx_qt::QObject {
10571057
ffi::cxx_qt_ffi_MyObject_upcastPtr(this)
10581058
}

‎crates/cxx-qt-gen/test_outputs/qenum.rs

+2-2
Original file line numberDiff line numberDiff line change
@@ -169,7 +169,7 @@ mod ffi {
169169
type QObject = cxx_qt::QObject;
170170
}
171171
}
172-
unsafe impl ::cxx_qt::Upcast<::cxx_qt::QObject> for ffi::MyObject {
172+
unsafe impl ::cxx_qt::casting::Upcast<::cxx_qt::QObject> for ffi::MyObject {
173173
unsafe fn upcast_ptr(this: *const Self) -> *const ::cxx_qt::QObject {
174174
ffi::cxx_qt_ffi_MyObject_upcastPtr(this)
175175
}
@@ -197,7 +197,7 @@ impl ::cxx_qt::CxxQtType for ffi::MyObject {
197197
ffi::cxx_qt_ffi_MyObject_unsafeRustMut(self)
198198
}
199199
}
200-
unsafe impl ::cxx_qt::Upcast<::cxx_qt::QObject> for ffi::MyRenamedObject {
200+
unsafe impl ::cxx_qt::casting::Upcast<::cxx_qt::QObject> for ffi::MyRenamedObject {
201201
unsafe fn upcast_ptr(this: *const Self) -> *const ::cxx_qt::QObject {
202202
ffi::cxx_qt_ffi_CxxName_upcastPtr(this)
203203
}

‎crates/cxx-qt-gen/test_outputs/signals.rs

+2-2
Original file line numberDiff line numberDiff line change
@@ -455,7 +455,7 @@ cxx_qt::static_assertions::assert_eq_size!(
455455
cxx_qt::signalhandler::CxxQtSignalHandler<MyObjectCxxQtSignalClosurenewData>,
456456
[usize; 2]
457457
);
458-
unsafe impl ::cxx_qt::Upcast<::cxx_qt::QObject> for ffi::MyObject {
458+
unsafe impl ::cxx_qt::casting::Upcast<::cxx_qt::QObject> for ffi::MyObject {
459459
unsafe fn upcast_ptr(this: *const Self) -> *const ::cxx_qt::QObject {
460460
ffi::cxx_qt_ffi_MyObject_upcastPtr(this)
461461
}
@@ -483,7 +483,7 @@ impl ::cxx_qt::CxxQtType for ffi::MyObject {
483483
ffi::cxx_qt_ffi_MyObject_unsafeRustMut(self)
484484
}
485485
}
486-
unsafe impl ::cxx_qt::Upcast<::cxx_qt::QObject> for ffi::QTimer {
486+
unsafe impl ::cxx_qt::casting::Upcast<::cxx_qt::QObject> for ffi::QTimer {
487487
unsafe fn upcast_ptr(this: *const Self) -> *const ::cxx_qt::QObject {
488488
ffi::cxx_qt_ffi_QTimer_upcastPtr(this)
489489
}

‎crates/cxx-qt-lib/src/core/qstringlist.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ use crate::core::qstringlist::ffi::QList_QString;
66
use crate::{QList, QString};
77
use core::mem::MaybeUninit;
88
use cxx::{type_id, ExternType};
9-
use cxx_qt::Upcast;
9+
use cxx_qt::casting::Upcast;
1010
use std::fmt;
1111
use std::ops::{Deref, DerefMut};
1212

0 commit comments

Comments
 (0)