Merge #815

815: Rename symbols for increased clarity r=Bromeon a=Bromeon

Addressing the first few of the pending renames in #773.
Changes are separated by commits.

bors try


Co-authored-by: Jan Haller <bromeon@gmail.com>

Author: hesuteia

bindings_generator/src/documentation.rs

@@ -129,8 +129,8 @@ This class is used to interact with Godot's editor."#
 All types in the Godot API have _interior mutability_ in Rust parlance.
 To enforce that the official [thread-safety guidelines][thread-safety] are
 followed, the typestate pattern is used in the `Ref` and `TRef` smart pointers,
-and the `Instance` API. The typestate `Access` in these types tracks whether the
-access is unique, shared, or exclusive to the current thread. For more information,
+and the `Instance` API. The typestate `Ownership` in these types tracks whether
+ownership is unique, shared, or exclusive to the current thread. For more information,
 see the type-level documentation on `Ref`.
 
 [thread-safety]: https://docs.godotengine.org/en/stable/tutorials/threads/thread_safe_apis.html"#;

bindings_generator/src/special_methods.rs

@@ -52,7 +52,7 @@ pub fn generate_godot_object_impl(class: &GodotClass) -> TokenStream {
         impl gdnative_core::private::godot_object::Sealed for #class_name {}
 
         unsafe impl GodotObject for #class_name {
-            type RefKind = #memory;
+            type Memory = #memory;
 
             #[inline]
             fn class_name() -> &'static str {

examples/dodge_the_creeps/src/main_scene.rs

@@ -140,7 +140,7 @@ impl Main {
 ///   scene as the root. For instance Spatial is used for this example.
 fn instance_scene<Root>(scene: &Ref<PackedScene, Shared>) -> Ref<Root, Unique>
 where
-    Root: gdnative::object::GodotObject<RefKind = ManuallyManaged> + SubClass<Node>,
+    Root: gdnative::object::GodotObject<Memory = ManuallyManaged> + SubClass<Node>,
 {
     let scene = unsafe { scene.assume_safe() };
 

examples/native_plugin/src/lib.rs

@@ -58,7 +58,7 @@ impl MyButton {
 
 unsafe fn load<T>(path: &str, hint: &str) -> Option<Ref<T, Shared>>
 where
-    T: GodotObject<RefKind = RefCounted> + SubClass<Resource>,
+    T: GodotObject<Memory = RefCounted> + SubClass<Resource>,
 {
     let resource = ResourceLoader::godot_singleton().load(path, hint, false)?;
     let resource = resource.assume_safe().claim();

examples/scene_create/src/lib.rs

@@ -116,7 +116,7 @@ pub fn load_scene(path: &str) -> Option<Ref<PackedScene, ThreadLocal>> {
 ///   scene as the root. For instance Spatial is used for this example.
 fn instance_scene<Root>(scene: &PackedScene) -> Result<Ref<Root, Unique>, ManageErrs>
 where
-    Root: gdnative::object::GodotObject<RefKind = ManuallyManaged> + SubClass<Node>,
+    Root: gdnative::object::GodotObject<Memory = ManuallyManaged> + SubClass<Node>,
 {
     let instance = scene
         .instance(PackedScene::GEN_EDIT_STATE_DISABLED)

gdnative-async/src/method.rs

@@ -8,7 +8,7 @@ use gdnative_core::core_types::{ToVariant, Variant};
 use gdnative_core::export::{Method, NativeClass, Varargs};
 use gdnative_core::log::{self, Site};
 use gdnative_core::object::ownership::Shared;
-use gdnative_core::object::RefInstance;
+use gdnative_core::object::TInstance;
 
 use crate::rt::Context;
 
@@ -41,7 +41,7 @@ pub trait AsyncMethod<C: NativeClass>: Send + Sync + 'static {
 pub struct Spawner<'a, C: NativeClass> {
     sp: &'static dyn LocalSpawn,
     ctx: Context,
-    this: RefInstance<'a, C, Shared>,
+    this: TInstance<'a, C, Shared>,
     args: Varargs<'a>,
     result: &'a mut Option<Result<(), SpawnError>>,
     /// Remove Send and Sync
@@ -54,7 +54,7 @@ impl<'a, C: NativeClass> Spawner<'a, C> {
     /// future types.
     pub fn spawn<F, R>(self, f: F)
     where
-        F: FnOnce(Arc<Context>, RefInstance<'_, C, Shared>, Varargs<'_>) -> R,
+        F: FnOnce(Arc<Context>, TInstance<'_, C, Shared>, Varargs<'_>) -> R,
         R: Future<Output = Variant> + 'static,
     {
         let ctx = Arc::new(self.ctx);
@@ -84,7 +84,7 @@ impl<F> Async<F> {
 }
 
 impl<C: NativeClass, F: AsyncMethod<C>> Method<C> for Async<F> {
-    fn call(&self, this: RefInstance<'_, C, Shared>, args: Varargs<'_>) -> Variant {
+    fn call(&self, this: TInstance<'_, C, Shared>, args: Varargs<'_>) -> Variant {
         if let Some(sp) = crate::executor::local_spawn() {
             let ctx = Context::new();
             let func_state = ctx.func_state();

gdnative-async/src/rt.rs

@@ -5,7 +5,7 @@ use gdnative_bindings::Object;
 use gdnative_core::core_types::{GodotError, Variant};
 use gdnative_core::init::InitHandle;
 use gdnative_core::object::ownership::Shared;
-use gdnative_core::object::{Instance, RefInstance, SubClass, TRef};
+use gdnative_core::object::{Instance, SubClass, TInstance, TRef};
 
 use crate::future;
 
@@ -31,7 +31,7 @@ impl Context {
         self.func_state.clone()
     }
 
-    fn safe_func_state(&self) -> RefInstance<'_, FuncState, Shared> {
+    fn safe_func_state(&self) -> TInstance<'_, FuncState, Shared> {
         // SAFETY: FuncState objects are bound to their origin threads in Rust, and
         // Context is !Send, so this is safe to call within this type.
         // Non-Rust code is expected to be following the official guidelines as per

gdnative-async/src/rt/bridge.rs

@@ -9,7 +9,7 @@ use gdnative_core::export::user_data::{ArcData, Map};
 use gdnative_core::export::{ClassBuilder, Method, NativeClass, NativeClassMethods, Varargs};
 use gdnative_core::godot_site;
 use gdnative_core::object::ownership::Shared;
-use gdnative_core::object::{Instance, RefInstance, TRef};
+use gdnative_core::object::{Instance, TInstance, TRef};
 
 use crate::future::Resume;
 
@@ -97,7 +97,7 @@ impl SignalBridge {
 struct OnSignalFn;
 
 impl Method<SignalBridge> for OnSignalFn {
-    fn call(&self, this: RefInstance<'_, SignalBridge, Shared>, args: Varargs<'_>) -> Variant {
+    fn call(&self, this: TInstance<'_, SignalBridge, Shared>, args: Varargs<'_>) -> Variant {
         let args = args.cloned().collect();
 
         let this_persist = this.clone().claim();

gdnative-async/src/rt/func_state.rs

@@ -7,7 +7,7 @@ use gdnative_core::export::{
 };
 use gdnative_core::godot_site;
 use gdnative_core::object::ownership::{Shared, Unique};
-use gdnative_core::object::{Instance, RefInstance};
+use gdnative_core::object::{Instance, TInstance};
 use gdnative_derive::FromVarargs;
 
 use crate::future::Resume;
@@ -57,7 +57,7 @@ impl FuncState {
     }
 }
 
-pub(super) fn resolve(this: RefInstance<'_, FuncState, Shared>, value: Variant) {
+pub(super) fn resolve(this: TInstance<'_, FuncState, Shared>, value: Variant) {
     this.script()
         .map_mut(|s| {
             match s.kind {
@@ -80,7 +80,7 @@ pub(super) fn resolve(this: RefInstance<'_, FuncState, Shared>, value: Variant)
     this.base().emit_signal("completed", &[value]);
 }
 
-pub(super) fn make_resumable(this: RefInstance<'_, FuncState, Shared>, resume: Resume<Variant>) {
+pub(super) fn make_resumable(this: TInstance<'_, FuncState, Shared>, resume: Resume<Variant>) {
     let kind = this
         .script()
         .map_mut(|s| std::mem::replace(&mut s.kind, Kind::Resumable(resume)))
@@ -113,7 +113,7 @@ struct IsValidArgs {
 
 impl StaticArgsMethod<FuncState> for IsValidFn {
     type Args = IsValidArgs;
-    fn call(&self, this: RefInstance<'_, FuncState, Shared>, args: Self::Args) -> Variant {
+    fn call(&self, this: TInstance<'_, FuncState, Shared>, args: Self::Args) -> Variant {
         if args.extended_check.is_some() {
             gdnative_core::log::warn(
                 Self::site().unwrap(),
@@ -145,7 +145,7 @@ struct ResumeArgs {
 
 impl StaticArgsMethod<FuncState> for ResumeFn {
     type Args = ResumeArgs;
-    fn call(&self, this: RefInstance<'_, FuncState, Shared>, args: Self::Args) -> Variant {
+    fn call(&self, this: TInstance<'_, FuncState, Shared>, args: Self::Args) -> Variant {
         this.map_mut(
             |s, owner| match std::mem::replace(&mut s.kind, Kind::Pending) {
                 Kind::Resumable(resume) => {

gdnative-bindings/src/utils.rs

@@ -2,7 +2,7 @@
 
 use gdnative_core::export::NativeClass;
 use gdnative_core::object::ownership::Shared;
-use gdnative_core::object::{RefInstance, SubClass, TRef};
+use gdnative_core::object::{SubClass, TInstance, TRef};
 
 use super::generated::{Engine, Node, SceneTree};
 
@@ -58,7 +58,7 @@ pub trait NodeExt {
     /// invariants must be observed for the resulting node during `'a`, if any.
     ///
     /// [thread-safety]: https://docs.godotengine.org/en/stable/tutorials/threads/thread_safe_apis.html
-    unsafe fn get_node_as_instance<'a, T>(&self, path: &str) -> Option<RefInstance<'a, T, Shared>>
+    unsafe fn get_node_as_instance<'a, T>(&self, path: &str) -> Option<TInstance<'a, T, Shared>>
     where
         T: NativeClass,
         T::Base: SubClass<Node>,