Revert "Change to register_table_function_with_options"

This reverts commit e3d2c66.

Author: mlafeldt

crates/duckdb/src/vtab/function.rs

@@ -51,11 +51,8 @@ impl BindInfo {
     /// Sets the user-provided bind data in the bind object. This object can be retrieved again during execution.
     ///
     /// # Arguments
-    ///  * `data`: The bind data object.
-    ///  * `free_function`: The callback that will be called to destroy the bind data (if any)
-    ///
-    /// # Safety
-    /// `data` must be a valid pointer, and `free_function` must properly free it.
+    ///  * `extra_data`: The bind data object.
+    ///  * `destroy`: The callback that will be called to destroy the bind data (if any)
     pub unsafe fn set_bind_data(&self, data: *mut c_void, free_function: Option<unsafe extern "C" fn(*mut c_void)>) {
         duckdb_bind_set_bind_data(self.ptr, data, free_function);
     }

crates/duckdb/src/vtab/mod.rs

@@ -23,20 +23,6 @@ mod excel;
 pub use function::{BindInfo, InitInfo, TableFunction, TableFunctionInfo};
 pub use value::Value;
 
-/// Options for registering a table function.
-pub struct TableFunctionOptions<E> {
-    /// Extra info passed to the function at runtime.
-    /// Accessible via `BindInfo::get_extra_info`, `InitInfo::get_extra_info`,
-    /// or `TableFunctionInfo::get_extra_info`.
-    pub extra_info: Option<E>,
-}
-
-impl<E> Default for TableFunctionOptions<E> {
-    fn default() -> Self {
-        Self { extra_info: None }
-    }
-}
-
 use crate::core::{DataChunkHandle, LogicalTypeHandle};
 use ffi::{duckdb_bind_info, duckdb_data_chunk, duckdb_function_info, duckdb_init_info};
 
@@ -165,26 +151,23 @@ impl Connection {
         self.db.borrow_mut().register_table_function(table_function)
     }
 
-    /// Register the given TableFunction with options.
+    /// Register the given TableFunction with custom extra info.
+    ///
+    /// This allows you to pass extra info that can be accessed during bind, init, and execution
+    /// via `BindInfo::get_extra_info`, `InitInfo::get_extra_info`, or `TableFunctionInfo::get_extra_info`.
     #[inline]
-    pub fn register_table_function_with_options<T: VTab, E>(
-        &self,
-        name: &str,
-        options: TableFunctionOptions<E>,
-    ) -> Result<()>
+    pub fn register_table_function_with_extra_info<T: VTab, E>(&self, name: &str, extra_info: &E) -> Result<()>
     where
-        E: Send + Sync + 'static,
+        E: Clone + Send + Sync + 'static,
     {
         let table_function = TableFunction::default();
         table_function
             .set_name(name)
             .supports_pushdown(T::supports_pushdown())
             .set_bind(Some(bind::<T>))
             .set_init(Some(init::<T>))
-            .set_function(Some(func::<T>));
-        if let Some(extra_info) = options.extra_info {
-            table_function.with_extra_info(extra_info);
-        }
+            .set_function(Some(func::<T>))
+            .with_extra_info(extra_info.clone());
         for ty in T::parameters().unwrap_or_default() {
             table_function.add_parameter(&ty);
         }
@@ -371,14 +354,9 @@ mod test {
     }
 
     #[test]
-    fn test_table_function_with_options() -> Result<(), Box<dyn Error>> {
+    fn test_table_function_with_extra_info() -> Result<(), Box<dyn Error>> {
         let conn = Connection::open_in_memory()?;
-        conn.register_table_function_with_options::<PrefixVTab, _>(
-            "greet",
-            TableFunctionOptions {
-                extra_info: Some("Howdy".to_string()),
-            },
-        )?;
+        conn.register_table_function_with_extra_info::<PrefixVTab, _>("greet", &"Howdy".to_string())?;
 
         let val = conn.query_row("select * from greet('partner')", [], |row| <(String,)>::try_from(row))?;
         assert_eq!(val, ("Howdy partner".to_string(),));