Revert macro re-export "fix"

(Doesn't actually help; mis-remembered how rustc handles these things)

Author: Colonial-Dev

CHANGELOG.md

@@ -1,4 +1,4 @@
-### v0.33.0
+### v0.34.0
 - (Routine Maintenance) Updated `rusqlite` from `0.33` to `0.34`.
   - This is a *breaking change,* requiring you to upgrade your `rusqlite` to match - trying to link against two different copies of `libsqlite3` will cause a compile fail.
 - Fixed issue with macros failing if Rusqlite is not linked to the current crate.

exemplar/Cargo.toml

@@ -15,7 +15,7 @@ all-features = true
 rustdoc-args = ["--cfg", "docsrs"]
 
 [dependencies]
-exemplar_proc_macro = { version = "0.34.0" }
+exemplar_proc_macro = { path = "../exemplar_proc_macro" }
 rusqlite = "0.34"
 
 [dev-dependencies]

exemplar/src/lib.rs

@@ -96,11 +96,6 @@ use rusqlite::types::{
     FromSqlResult
 };
 
-// We re-export rusqlite to maximize macro compatibility.
-// (See https://github.com/Colonial-Dev/exemplar/issues/5)
-#[doc(hidden)]
-pub use rusqlite;
-
 pub use crate::macros::*;
 
 /// Type alias for the outcome of converting a value to an SQL-friendly representation.

exemplar/src/macros.rs

@@ -214,17 +214,17 @@ macro_rules! record {
         }
 
         impl $name {
-            fn from_row(row: &::exemplar::rusqlite::Row) -> ::exemplar::rusqlite::Result<Self> {
+            fn from_row(row: &::rusqlite::Row) -> ::rusqlite::Result<Self> {
                 Ok(Self {
                     $($fname : row.get(stringify!($fname))?),*
                 })
             }
         }
 
-        impl<'a> ::std::convert::TryFrom<&'a ::exemplar::rusqlite::Row<'_>> for $name {
-            type Error = ::exemplar::rusqlite::Error;
+        impl<'a> ::std::convert::TryFrom<&'a ::rusqlite::Row<'_>> for $name {
+            type Error = ::rusqlite::Error;
 
-            fn try_from(value: &'a ::exemplar::rusqlite::Row) -> Result<Self, Self::Error> {
+            fn try_from(value: &'a ::rusqlite::Row) -> Result<Self, Self::Error> {
                 Self::from_row(value)
             }
         }
@@ -313,28 +313,28 @@ macro_rules! sql_enum {
         }
 
         #[automatically_derived]
-        impl ::exemplar::rusqlite::ToSql for $name {
-            fn to_sql(&self) -> ::exemplar::rusqlite::Result<::exemplar::rusqlite::types::ToSqlOutput<'_>> {
-                let value = ::exemplar::rusqlite::types::Value::Integer(*self as i64);
-                let value = ::exemplar::rusqlite::types::ToSqlOutput::Owned(value);
+        impl ::rusqlite::ToSql for $name {
+            fn to_sql(&self) -> ::rusqlite::Result<::rusqlite::types::ToSqlOutput<'_>> {
+                let value = ::rusqlite::types::Value::Integer(*self as i64);
+                let value = ::rusqlite::types::ToSqlOutput::Owned(value);
                 Ok(value)
             }
         }
 
         #[automatically_derived]
-        impl ::exemplar::rusqlite::types::FromSql for $name {
-            fn column_result(value: ::exemplar::rusqlite::types::ValueRef<'_>) -> ::exemplar::rusqlite::types::FromSqlResult<Self> {
+        impl ::rusqlite::types::FromSql for $name {
+            fn column_result(value: ::rusqlite::types::ValueRef<'_>) -> ::rusqlite::types::FromSqlResult<Self> {
                 value.as_i64()
                     .map(<$name>::try_from)?
                     .map_err(|err| {
-                        ::exemplar::rusqlite::types::FromSqlError::Other(Box::new(err))
+                        ::rusqlite::types::FromSqlError::Other(Box::new(err))
                     })
             }
         }
 
         #[automatically_derived]
         impl ::std::convert::TryFrom<i64> for $name {
-            type Error = ::exemplar::rusqlite::types::FromSqlError;
+            type Error = ::rusqlite::types::FromSqlError;
 
             fn try_from(value: i64) -> Result<Self, Self::Error> {
                 match value {
@@ -345,7 +345,7 @@ macro_rules! sql_enum {
                             stringify!($name)
                         );
 
-                        Err(::exemplar::rusqlite::types::FromSqlError::Other(
+                        Err(::rusqlite::types::FromSqlError::Other(
                             msg.into()
                         ))
                     }

exemplar_proc_macro/src/codegen.rs

@@ -22,7 +22,7 @@ pub fn from_row(derivee: &Derivee) -> QuoteStream {
 
     quote! {
         #[inline]
-        fn from_row(row: &::exemplar::rusqlite::Row) -> ::exemplar::rusqlite::Result<Self> 
+        fn from_row(row: &::rusqlite::Row) -> ::rusqlite::Result<Self> 
         where
             Self: ::std::marker::Sized,
         {
@@ -64,15 +64,15 @@ pub fn inserts(derivee: &Derivee) -> QuoteStream {
 
     quote! {
         #[inline]
-        fn insert(&self, conn: &::exemplar::rusqlite::Connection) -> ::exemplar::rusqlite::Result<()> {
+        fn insert(&self, conn: &::rusqlite::Connection) -> ::rusqlite::Result<()> {
             self.insert_or(conn, ::exemplar::OnConflict::Abort)
         }
 
         #[inline]
-        fn insert_or(&self, conn: &::exemplar::rusqlite::Connection, strategy: ::exemplar::OnConflict) -> ::exemplar::rusqlite::Result<()> {
+        fn insert_or(&self, conn: &::rusqlite::Connection, strategy: ::exemplar::OnConflict) -> ::rusqlite::Result<()> {
             use ::exemplar::OnConflict::*;
 
-            let exec = |sql: &str| -> ::exemplar::rusqlite::Result<()> {
+            let exec = |sql: &str| -> ::rusqlite::Result<()> {
                 let mut stmt = conn.prepare_cached(sql)?;
 
                 stmt.execute(rusqlite::named_params! {
@@ -92,7 +92,7 @@ pub fn inserts(derivee: &Derivee) -> QuoteStream {
         }
 
         #[inline]
-        fn insert_with(&self, stmt: &mut::exemplar::rusqlite::Statement) -> ::exemplar::rusqlite::Result<()> {
+        fn insert_with(&self, stmt: &mut::rusqlite::Statement) -> ::rusqlite::Result<()> {
             stmt.execute(rusqlite::named_params! {
                 #(#col_names: #field_idents),*
             })?;
@@ -116,17 +116,17 @@ pub fn to_params(derivee: &Derivee) -> QuoteStream {
         .map(|(ident, field)| {
             if let Some(bind) = util::get_bind_path(field) {
                 // If the field has a #[bind] attribute, then we execute it now and box the result.
-                quote! { Boxed(Box::new(#bind(&self.#ident)?) as Box<dyn ::exemplar::rusqlite::ToSql>) }
+                quote! { Boxed(Box::new(#bind(&self.#ident)?) as Box<dyn ::rusqlite::ToSql>) }
             }
             else {
                 // Otherwise, we're good to just borrow directly from self and cast to a dyn ToSql.
-                quote! { Borrowed(&self.#ident as &dyn ::exemplar::rusqlite::ToSql) }
+                quote! { Borrowed(&self.#ident as &dyn ::rusqlite::ToSql) }
             }
         });
 
     quote! {
         #[inline]
-        fn to_params(&self) -> ::exemplar::rusqlite::Result<::exemplar::Parameters> {
+        fn to_params(&self) -> ::rusqlite::Result<::exemplar::Parameters> {
             use ::std::boxed::Box;
             use ::exemplar::Parameter::*;
 
@@ -206,7 +206,7 @@ pub fn check_test(derivee: &Derivee) -> QuoteStream {
         #[test]
         fn #func() {
             use ::std::collections::HashSet;
-            use ::exemplar::rusqlite::Connection;
+            use ::rusqlite::Connection;
 
             let schema = include_str!(#path);
 

exemplar_proc_macro/src/lib.rs

@@ -85,10 +85,10 @@ pub fn derive_model(input: TokenStream) -> TokenStream {
         }
 
         #[automatically_derived]
-        impl<'a> ::std::convert::TryFrom<&'a ::exemplar::rusqlite::Row<'_>> for #name {
-            type Error = ::exemplar::rusqlite::Error;
+        impl<'a> ::std::convert::TryFrom<&'a ::rusqlite::Row<'_>> for #name {
+            type Error = ::rusqlite::Error;
 
-            fn try_from(value: &'a ::exemplar::rusqlite::Row) -> Result<Self, Self::Error> {
+            fn try_from(value: &'a ::rusqlite::Row) -> Result<Self, Self::Error> {
                 Self::from_row(value)
             }
         }