From f193bfcfeeccbeb0a0671fb8f2f774ef1fe1b43f Mon Sep 17 00:00:00 2001 From: Schneems Date: Thu, 12 Dec 2024 11:03:05 -0600 Subject: [PATCH] Add missing semicolons after macro calls Otherwise I was getting this error: ``` $ RUSTFLAGS="-Zmacro-backtrace" cargo build Compiling heroku-ruby-buildpack v0.0.0 (/Users/rschneeman/Documents/projects/work/buildpacks-ruby/buildpacks/ruby) error: macros that expand to items must be delimited with braces or followed by a semicolon --> /Users/rschneeman/.cargo/registry/src/index.crates.io-6f17d22bba15001f/magic_migrate-0.2.0/src/lib.rs:437:34 | 419 | macro_rules! try_migrate_link { | ----------------------------- in this expansion of `$crate::try_migrate_link!` (#3) ... 437 | $crate::try_migrate_link!($b, $($rest),*) | ^^^^^^^^^^^^^^^^ ... 714 | macro_rules! try_migrate_deserializer_chain { | ------------------------------------------- | | | in this expansion of `try_migrate_deserializer_chain!` (#1) | in this expansion of `$crate::try_migrate_deserializer_chain!` (#2) ... 737 | $crate::try_migrate_link!($a, $($rest),+); | ----------------------------------------- in this macro invocation (#3) ... 764 | $crate::try_migrate_deserializer_chain!(error: $err, deserializer: $deser, chain: [$a, $($rest),+]); | --------------------------------------------------------------------------------------------------- in this macro invocation (#2) | ::: buildpacks/ruby/src/layers/bundle_install_layer.rs:120:1 | 120 | / try_migrate_deserializer_chain!( 121 | | chain: [MetadataV1, MetadataV2, MetadataV3], 122 | | error: MetadataMigrateError, 123 | | deserializer: toml::Deserializer::new, 124 | | ); | |_- in this macro invocation (#1) ``` From this code: ``` try_migrate_deserializer_chain!( chain: [MetadataV1, MetadataV2, MetadataV3], error: MetadataMigrateError, deserializer: toml::Deserializer::new, ); ``` Presumably because I was using 3 values instead of just 2. --- src/lib.rs | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/src/lib.rs b/src/lib.rs index 36d73b8..a2e7bba 100644 --- a/src/lib.rs +++ b/src/lib.rs @@ -203,7 +203,7 @@ pub trait Migrate: From + Any + DeserializeOwned + Debug { /// // support `From` /// impl From for PersonMigrationError { /// fn from(_value: Infallible) -> Self { -/// unreachable!() +/// unreachable!(); /// } /// } /// @@ -335,8 +335,8 @@ macro_rules! migrate_link { $crate::migrate_link!($a, $b); // Link B => C, and the rest - $crate::migrate_link!($b, $($rest),*) - ) + $crate::migrate_link!($b, $($rest),*); + ); } /// Links each struct passed in to each other to build a [`Migrate`] link chain. @@ -361,7 +361,7 @@ macro_rules! migrate_link { /// # struct UserV2; /// # impl From for UserV2 { /// # fn from(value: UserV1) -> Self { -/// # unimplemented!() +/// # unimplemented!(); /// # } /// # } /// # @@ -369,7 +369,7 @@ macro_rules! migrate_link { /// # struct UserV3; /// # impl From for UserV3 { /// # fn from(value: UserV2) -> Self { -/// # unimplemented!() +/// # unimplemented!(); /// # } /// # } /// @@ -397,7 +397,7 @@ macro_rules! migrate_toml_chain { deserializer: toml::Deserializer::new, chain: [$a, $($rest),+] ); - ) + ); } /// Macro for linking structs together in an infallible [`TryMigrate`] migration chain @@ -434,8 +434,8 @@ macro_rules! try_migrate_link { $crate::try_migrate_link!($a, $b); // Link B => C, and the rest - $crate::try_migrate_link!($b, $($rest),*) - ) + $crate::try_migrate_link!($b, $($rest),*); + ); } /// A macro to help define [`TryMigrate`] based migrations @@ -521,7 +521,7 @@ macro_rules! try_migrate_link { macro_rules! try_migrate_toml_chain { // Base case (error: $err:ident, chain: [$a:ident] $(,)?) => { - $crate::try_migrate_deserializer_chain!(error: $err, deserializer: toml::Deserializer::new, chain: [$a]) + $crate::try_migrate_deserializer_chain!(error: $err, deserializer: toml::Deserializer::new, chain: [$a]); }; // Position variant (chain: [$a:ident], error: $err:ident $(,)?) => { @@ -537,7 +537,7 @@ macro_rules! try_migrate_toml_chain { ); // Position variant (chain: [$a:ident, $($rest:ident),+], error: $err:ident $(,)?) => ( - $crate::try_migrate_toml_chain!(error: $err, chain: [$a, $($rest),+]) + $crate::try_migrate_toml_chain!(error: $err, chain: [$a, $($rest),+]); ); } @@ -724,7 +724,7 @@ macro_rules! try_migrate_deserializer_chain { } impl From for $err { fn from(_value: Infallible) -> Self { - unreachable!() + unreachable!(); } } };