Skip to content

Commit c72af11

Browse files
committed
remove old vtable! macro
Signed-off-by: Joe Isaacs <joe.isaacs@live.co.uk>
1 parent 69afd8d commit c72af11

2 files changed

Lines changed: 0 additions & 29 deletions

File tree

vortex-array/src/array/vtable/mod.rs

Lines changed: 0 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -291,29 +291,3 @@ pub fn patches_child_name(idx: usize) -> &'static str {
291291
}
292292
}
293293

294-
/// vtable! macro — adds a deprecated `to_array()` helper on legacy array types.
295-
///
296-
/// Two forms:
297-
/// - `vtable!(Foo)` — short for `vtable!(Foo, Foo)`
298-
/// - `vtable!(Foo, FooVT)` — where `FooArray` is the inner struct name
299-
#[macro_export]
300-
macro_rules! vtable {
301-
($V:ident) => {
302-
$crate::vtable!($V, $V);
303-
};
304-
// Legacy form: FooArray is the inner struct name, no type alias generated.
305-
($Base:ident, $VT:ident) => {
306-
$crate::aliases::paste::paste! {
307-
impl [<$Base Array>] {
308-
#[deprecated(note = "use `.into_array()` (owned) or `.clone().into_array()` (ref) to make clones explicit")]
309-
pub fn to_array(&self) -> $crate::ArrayRef
310-
where
311-
Self: Clone + $crate::IntoArray,
312-
{
313-
use $crate::IntoArray;
314-
self.clone().into_array()
315-
}
316-
}
317-
}
318-
};
319-
}

vortex-python/src/arrays/py/vtable.rs

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@ use vortex::array::VTable;
1919
use vortex::array::ValidityVTable;
2020
use vortex::array::buffer::BufferHandle;
2121
use vortex::array::validity::Validity;
22-
use vortex::array::vtable;
2322
use vortex::dtype::DType;
2423
use vortex::error::VortexResult;
2524
use vortex::error::vortex_bail;
@@ -30,8 +29,6 @@ use vortex::session::VortexSession;
3029

3130
use crate::arrays::py::PythonArray;
3231

33-
vtable!(Python, PythonVTable);
34-
3532
/// Wrapper struct encapsulating a Python encoding.
3633
#[derive(Debug, Clone)]
3734
pub struct PythonVTable {

0 commit comments

Comments
 (0)