Refactor: remove OptionalBound from T

Author: SteveLauC

openraft/src/type_config.rs

@@ -72,7 +72,7 @@ pub trait RaftTypeConfig:
     /// See the [storage chapter of the guide][sto] for details on log compaction / snapshotting.
     ///
     /// [sto]: crate::docs::getting_started#3-implement-raftlogstorage-and-raftstatemachine
-    type SnapshotData: OptionalSend + OptionalSync + 'static;
+    type SnapshotData: OptionalSend + 'static;
 
     /// Asynchronous runtime type.
     type AsyncRuntime: AsyncRuntime;

openraft/src/type_config/async_runtime/impls/tokio_runtime.rs

@@ -77,7 +77,7 @@ impl AsyncRuntime for TokioRuntime {
     type MpscUnbounded = TokioMpscUnbounded;
     type Watch = TokioWatch;
     type Oneshot = TokioOneshot;
-    type Mutex<T: OptionalSend + OptionalSync + 'static> = TokioMutex<T>;
+    type Mutex<T: OptionalSend + 'static> = TokioMutex<T>;
 }
 
 pub struct TokioMpscUnbounded;
@@ -203,7 +203,7 @@ where T: OptionalSend
 type TokioMutex<T> = tokio::sync::Mutex<T>;
 
 impl<T> mutex::Mutex<T> for TokioMutex<T>
-where T: OptionalSend + OptionalSync + 'static
+where T: OptionalSend + 'static
 {
     type Guard<'a> = tokio::sync::MutexGuard<'a, T>;
 

openraft/src/type_config/async_runtime/mod.rs

@@ -104,5 +104,5 @@ pub trait AsyncRuntime: Debug + Default + PartialEq + Eq + OptionalSend + Option
 
     type Oneshot: Oneshot;
 
-    type Mutex<T: OptionalSend + OptionalSync + 'static>: Mutex<T>;
+    type Mutex<T: OptionalSend + 'static>: Mutex<T>;
 }

openraft/src/type_config/async_runtime/mutex.rs

@@ -5,7 +5,7 @@ use crate::OptionalSend;
 use crate::OptionalSync;
 
 /// Represents an implementation of an asynchronous Mutex.
-pub trait Mutex<T: OptionalSend + OptionalSync + 'static>: OptionalSend + OptionalSync {
+pub trait Mutex<T: OptionalSend + 'static>: OptionalSend + OptionalSync {
     /// Handle to an acquired lock, should release it when dropped.
     type Guard<'a>: DerefMut<Target = T> + OptionalSend + OptionalSync
     where Self: 'a;

openraft/src/type_config/util.rs

@@ -97,7 +97,7 @@ pub trait TypeConfigExt: RaftTypeConfig {
     /// This is just a wrapper of
     /// [`AsyncRuntime::Mutex::new()`](`crate::async_runtime::Mutex::new`).
     fn mutex<T>(value: T) -> MutexOf<Self, T>
-    where T: OptionalSend + OptionalSync {
+    where T: OptionalSend {
         MutexOf::<Self, T>::new(value)
     }