[update] fence implementation.

Author: vmarcella

crates/lambda-platform/src/gfx/fence.rs

@@ -44,6 +44,20 @@ impl<RenderBackend: gfx_hal::Backend> RenderSemaphore<RenderBackend> {
   }
 }
 
+impl<RenderBackend: gfx_hal::Backend> RenderSemaphore<RenderBackend> {
+  /// Retrieve a reference to the internal semaphore.
+  pub(super) fn internal_semaphore(&self) -> &RenderBackend::Semaphore {
+    return &self.semaphore;
+  }
+
+  /// Retrieve a mutable reference to the internal semaphore.
+  pub(super) fn internal_semaphore_mut(
+    &mut self,
+  ) -> &mut RenderBackend::Semaphore {
+    return &mut self.semaphore;
+  }
+}
+
 pub struct RenderSubmissionFenceBuilder {
   default_render_timeout: u64,
 }
@@ -118,24 +132,14 @@ impl<RenderBackend: gfx_hal::Backend> RenderSubmissionFence<RenderBackend> {
   }
 }
 
-pub(crate) mod internal {
-  /// Retrieve the underlying submission fence.
-  pub fn mutable_fence_for<RenderBackend: gfx_hal::Backend>(
-    fence: &mut super::RenderSubmissionFence<RenderBackend>,
-  ) -> &mut RenderBackend::Fence {
-    return &mut fence.fence;
-  }
-
-  /// Retrieve the underlying semaphore.
-  pub fn mutable_semaphore_for<RenderBackend: gfx_hal::Backend>(
-    semaphore: &mut super::RenderSemaphore<RenderBackend>,
-  ) -> &mut RenderBackend::Semaphore {
-    return &mut semaphore.semaphore;
+impl<RenderBackend: gfx_hal::Backend> RenderSubmissionFence<RenderBackend> {
+  /// Retrieve the underlying fence.
+  pub fn internal_fence(&self) -> &RenderBackend::Fence {
+    return &self.fence;
   }
 
-  pub fn semaphore_for<RenderBackend: gfx_hal::Backend>(
-    semaphore: &super::RenderSemaphore<RenderBackend>,
-  ) -> &RenderBackend::Semaphore {
-    return &semaphore.semaphore;
+  /// Retrieve a mutable reference to the underlying fence.
+  pub fn internal_fence_mut(&mut self) -> &mut RenderBackend::Fence {
+    return &mut self.fence;
   }
 }

crates/lambda-platform/src/gfx/gpu.rs

@@ -145,9 +145,9 @@ impl<RenderBackend: gfx_hal::Backend> Gpu<RenderBackend> {
           // TODO(vmarcella): This was needed to allow the push constants to
           // properly render to the screen. Look into a better way to do this.
           signal_semaphores.into_iter().map(|semaphore| {
-            return super::fence::internal::semaphore_for(semaphore);
+            return semaphore.internal_semaphore();
           }),
-          Some(super::fence::internal::mutable_fence_for(fence)),
+          Some(fence.internal_fence_mut()),
         );
     }
   }
@@ -165,7 +165,7 @@ impl<RenderBackend: gfx_hal::Backend> Gpu<RenderBackend> {
       self.queue_group.queues[0].present(
         render_surface,
         render_image,
-        Some(super::fence::internal::mutable_semaphore_for(semaphore)),
+        Some(semaphore.internal_semaphore_mut()),
       )
     };