From 3eb1ae70d606c348f9a0d79433c55266e6b75099 Mon Sep 17 00:00:00 2001
From: XLP <xlp@xlp.cx>
Date: Sun, 19 Jan 2025 16:21:23 +0100
Subject: [PATCH] feat: add `default_sync` and `is_async` feature gate

Purpose of these features is to enable depdendants to provide a feature
to enable async support instead of a feature to disable it.
Creates (which would want to be sync by default) could then:
* Always enable the `default_sync` feature.
* Enable async support with the `is_async` feature when needed.
---
 Cargo.toml |  2 ++
 README.md  |  2 ++
 src/lib.rs | 16 +++++++++-------
 3 files changed, 13 insertions(+), 7 deletions(-)

diff --git a/Cargo.toml b/Cargo.toml
index c9ec242..32aea69 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -43,3 +43,5 @@ async-trait = "0.1"
 [features]
 default = [ ]
 is_sync = [ ]
+default_sync = [ ]
+is_async = [ ]
diff --git a/README.md b/README.md
index 1289a73..331d6ab 100644
--- a/README.md
+++ b/README.md
@@ -17,6 +17,8 @@ macro**.
 those `async` and `await` when you need a blocking code.
 - Switch between sync and async by toggling `is_sync` feature gate in
   `Cargo.toml`.
+- Alternatively activating the `default_sync` feature will switch to sync by
+  default and can be switched to async via `is_async`.
 - use `must_be_async` and `must_be_sync` to keep code in specified version
 - use `async_impl` and `sync_impl` to only compile code block on specified
   version
diff --git a/src/lib.rs b/src/lib.rs
index 7df2b3f..9137b68 100644
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -15,6 +15,8 @@
 //! those `async` and `await` when you need a blocking code.
 //! - Switch between sync and async by toggling `is_sync` feature gate in
 //!   `Cargo.toml`.
+//! - Alternatively activating the `default_sync` feature will switch to sync by
+//!   default and can be switched to async via `is_async`.
 //! - use `must_be_async` and `must_be_sync` to keep code in specified version
 //! - use `async_impl` and `sync_impl` to only compile code block on specified
 //!   version
@@ -396,7 +398,7 @@ pub fn maybe_async(args: TokenStream, input: TokenStream) -> TokenStream {
     };
     let mut item = parse_macro_input!(input as Item);
 
-    let token = if cfg!(feature = "is_sync") {
+    let token = if is_sync() {
         convert_sync(&mut item)
     } else {
         convert_async(&mut item, mode)
@@ -429,11 +431,7 @@ pub fn must_be_sync(_args: TokenStream, input: TokenStream) -> TokenStream {
 #[proc_macro_attribute]
 pub fn sync_impl(_args: TokenStream, input: TokenStream) -> TokenStream {
     let input = TokenStream2::from(input);
-    let token = if cfg!(feature = "is_sync") {
-        quote!(#input)
-    } else {
-        quote!()
-    };
+    let token = if is_sync() { quote!(#input) } else { quote!() };
     token.into()
 }
 
@@ -447,7 +445,7 @@ pub fn async_impl(args: TokenStream, _input: TokenStream) -> TokenStream {
         Ok(m) => m,
         Err(e) => return e.to_compile_error().into(),
     };
-    let token = if cfg!(feature = "is_sync") {
+    let token = if is_sync() {
         quote!()
     } else {
         let mut item = parse_macro_input!(_input as Item);
@@ -466,6 +464,10 @@ fn parse_nested_meta_or_str(input: ParseStream) -> Result<TokenStream2> {
     }
 }
 
+fn is_sync() -> bool {
+    cfg!(feature = "is_sync") || (cfg!(feature = "default_sync") && !cfg!(feature = "is_async"))
+}
+
 /// Handy macro to unify test code of sync and async code
 ///
 /// Since the API of both sync and async code are the same,