Macros now exported under gdnative::macros, not in crate root

Author: Bromeon

gdnative-derive/src/lib.rs

@@ -50,7 +50,7 @@ mod variant;
 /// impl gdnative::nativescript::NativeClassMethods for Foo {
 ///     fn register(builder: &ClassBuilder<Self>) {
 ///         use gdnative::nativescript::export::*;
-///         builder.build_method("foo", gdnative::godot_wrap_method!(Foo, fn foo(&self, _owner: &Reference, bar: i64) -> i64))
+///         builder.build_method("foo", gdnative::macros::godot_wrap_method!(Foo, fn foo(&self, _owner: &Reference, bar: i64) -> i64))
 ///             .with_rpc_mode(RpcMode::Disabled)
 ///             .done_stateless();
 ///     }

gdnative-derive/src/methods.rs

@@ -128,7 +128,7 @@ pub(crate) fn derive_methods(item_impl: ItemImpl) -> TokenStream2 {
 
             quote_spanned!( sig_span=>
                 {
-                    let method = ::gdnative::godot_wrap_method!(
+                    let method = ::gdnative::macros::godot_wrap_method!(
                         #class_name,
                         fn #name ( #( #args )* ) -> #ret_ty
                     );

gdnative/src/lib.rs

@@ -60,16 +60,18 @@
 #[doc(inline)]
 pub use gdnative_core::{core_types, nativescript, object};
 
-// Make macros available inside crate on top-level, for convenience and to make other macros look nicer
-// For the user, they are exported either in prelude (convenience) or fully qualified.
-#[deprecated]
-pub use gdnative_core::{
-    godot_dbg, godot_error, godot_gdnative_init, godot_gdnative_terminate, godot_init,
-    godot_nativescript_init, godot_print, godot_warn, godot_wrap_method,
-};
+/// Collection of declarative `godot_*` macros, mostly for GDNative registration and output.
+pub mod macros {
+    pub use gdnative_core::{
+        godot_dbg, godot_error, godot_gdnative_init, godot_gdnative_terminate, godot_init,
+        godot_nativescript_init, godot_print, godot_warn, godot_wrap_method,
+    };
+}
 
+// Implementation details (e.g. used by macros).
+// However, do not re-export macros (on crate level), thus no wildcard
 #[doc(hidden)]
-pub use gdnative_core::*;
+pub use gdnative_core::{libc, sys};
 
 /// Derive macros and macro attributes.
 #[doc(inline)]

test/src/lib.rs

@@ -86,7 +86,7 @@ fn test_underscore_method_binding() -> bool {
     .is_ok();
 
     if !ok {
-        gdnative::godot_error!("   !! Test test_underscore_method_binding failed");
+        godot_error!("   !! Test test_underscore_method_binding failed");
     }
 
     ok
@@ -165,7 +165,7 @@ fn test_rust_class_construction() -> bool {
     .is_ok();
 
     if !ok {
-        gdnative::godot_error!("   !! Test test_rust_class_construction failed");
+        godot_error!("   !! Test test_rust_class_construction failed");
     }
 
     ok
@@ -247,7 +247,7 @@ fn test_from_instance_id() -> bool {
     .is_ok();
 
     if !ok {
-        gdnative::godot_error!("   !! Test test_from_instance_id failed");
+        godot_error!("   !! Test test_from_instance_id failed");
     }
 
     ok
@@ -268,4 +268,4 @@ fn init(handle: InitHandle) {
     test_vararray_return::register(handle);
 }
 
-gdnative::godot_init!(init);
+godot_init!(init);

test/src/test_constructor.rs

@@ -62,7 +62,7 @@ fn test_from_class_name() -> bool {
     .is_ok();
 
     if !ok {
-        gdnative::godot_error!("   !! Test test_from_class_name failed");
+        godot_error!("   !! Test test_from_class_name failed");
     }
 
     ok

test/src/test_derive.rs

@@ -129,7 +129,7 @@ fn test_derive_to_variant() -> bool {
     .is_ok();
 
     if !ok {
-        gdnative::godot_error!("   !! Test test_derive_to_variant failed");
+        godot_error!("   !! Test test_derive_to_variant failed");
     }
 
     ok
@@ -167,7 +167,7 @@ fn test_derive_owned_to_variant() -> bool {
     .is_ok();
 
     if !ok {
-        gdnative::godot_error!("   !! Test test_derive_owned_to_variant failed");
+        godot_error!("   !! Test test_derive_owned_to_variant failed");
     }
 
     ok
@@ -278,7 +278,7 @@ fn test_derive_nativeclass_with_property_hooks() -> bool {
     .is_ok();
 
     if !ok {
-        gdnative::godot_error!("   !! Test test_derive_owned_to_variant failed");
+        godot_error!("   !! Test test_derive_owned_to_variant failed");
     }
 
     ok
@@ -315,7 +315,7 @@ fn test_derive_nativeclass_without_constructor() -> bool {
     .is_ok();
 
     if !ok {
-        gdnative::godot_error!("   !! Test test_derive_nativeclass_without_constructor failed");
+        godot_error!("   !! Test test_derive_nativeclass_without_constructor failed");
     }
 
     ok

test/src/test_free_ub.rs

@@ -75,7 +75,7 @@ fn test_owner_free_ub() -> bool {
     .is_ok();
 
     if !ok {
-        gdnative::godot_error!("   !! Test test_owner_free_ub failed");
+        godot_error!("   !! Test test_owner_free_ub failed");
     }
 
     ok

test/src/test_map_owned.rs

@@ -62,7 +62,7 @@ fn test_map_owned() -> bool {
     .is_ok();
 
     if !ok {
-        gdnative::godot_error!("   !! Test test_map_owned failed");
+        godot_error!("   !! Test test_map_owned failed");
     }
 
     ok

test/src/test_register.rs

@@ -109,7 +109,7 @@ fn test_register_property() -> bool {
     .is_ok();
 
     if !ok {
-        gdnative::godot_error!("   !! Test test_register_property failed");
+        godot_error!("   !! Test test_register_property failed");
     }
 
     ok
@@ -236,7 +236,7 @@ fn test_advanced_methods() -> bool {
     .is_ok();
 
     if !ok {
-        gdnative::godot_error!("   !! Test test_advanced_methods failed");
+        godot_error!("   !! Test test_advanced_methods failed");
     }
 
     ok

test/src/test_return_leak.rs

@@ -83,7 +83,7 @@ fn test_return_leak() -> bool {
     .is_ok();
 
     if !ok {
-        gdnative::godot_error!("   !! Test test_return_leak failed");
+        godot_error!("   !! Test test_return_leak failed");
     }
 
     ok