Fix smithay-clipboard integration

Fixes: alacritty#2574

Author: kchibisov

Diff for: CHANGELOG.md

@@ -39,6 +39,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Selecting trailing tab with semantic expansion
 - URL parser incorrectly handling Markdown URLs and angled brackets
 - Intermediate bytes of CSI sequences not checked
+- Wayland clipboard integration
 
 ## 0.3.3
 

Diff for: Cargo.lock

@@ -17,10 +17,7 @@ use std::ffi::c_void;
 
 use copypasta::nop_clipboard::NopClipboardContext;
 #[cfg(not(any(target_os = "macos", target_os = "windows")))]
-use copypasta::wayland_clipboard::{
-    Clipboard as WaylandClipboardClipboard, Primary as WaylandPrimaryClipboard,
-    WaylandClipboardContext,
-};
+use copypasta::wayland_clipboard;
 #[cfg(not(any(target_os = "macos", target_os = "windows")))]
 use copypasta::x11_clipboard::{Primary as X11SelectionClipboard, X11ClipboardContext};
 use copypasta::{ClipboardContext, ClipboardProvider};
@@ -39,22 +36,9 @@ impl Clipboard {
     #[cfg(not(any(target_os = "macos", target_os = "windows")))]
     pub fn new(display: Option<*mut c_void>) -> Self {
         if let Some(display) = display {
-            return Self {
-                clipboard: unsafe {
-                    Box::new(
-                        WaylandClipboardContext::<WaylandClipboardClipboard>::new_from_external(
-                            display,
-                        ),
-                    )
-                },
-                selection: unsafe {
-                    Some(Box::new(
-                        WaylandClipboardContext::<WaylandPrimaryClipboard>::new_from_external(
-                            display,
-                        ),
-                    ))
-                },
-            };
+            let (selection, clipboard) =
+                unsafe { wayland_clipboard::create_clipboards_from_external(display) };
+            return Self { clipboard: Box::new(clipboard), selection: Some(Box::new(selection)) };
         }
 
         Self {

Diff for: alacritty_terminal/src/clipboard.rs

@@ -17,7 +17,7 @@ objc-foundation = "0.1"
 
 [target.'cfg(all(unix, not(any(target_os="macos", target_os="android", target_os="emscripten"))))'.dependencies]
 x11-clipboard = "0.3"
-smithay-clipboard = "0.3.2"
+smithay-clipboard = "0.3.4"
 wayland-client = { version = "0.23.3", features = ["dlopen"] }
 
 [target.'cfg(all(unix, not(any(target_os="macos", target_os="android", target_os="emscripten"))))'.dev-dependencies]

Diff for: copypasta/Cargo.toml

@@ -14,7 +14,7 @@ mod wayland {
     extern crate copypasta;
     extern crate smithay_client_toolkit as sctk;
 
-    use wayland::copypasta::wayland_clipboard::{Clipboard, WaylandClipboardContext};
+    use wayland::copypasta::wayland_clipboard::create_clipboards;
     use wayland::copypasta::ClipboardProvider;
 
     use std::io::{Read, Seek, SeekFrom, Write};
@@ -37,7 +37,7 @@ mod wayland {
             Display::connect_to_env().expect("Failed to connect to the wayland server.");
         let env = Environment::from_display(&*display, &mut event_queue).unwrap();
 
-        let mut ctx = WaylandClipboardContext::<Clipboard>::new(&display);
+        let (mut ctx, _) = create_clipboards(&display);
         let cb_contents = Arc::new(Mutex::new(String::new()));
 
         let seat = env.manager.instantiate_range(2, 6, NewProxy::implement_dummy).unwrap();

Diff for: copypasta/examples/wayland.rs

@@ -14,57 +14,56 @@
 
 use std::error::Error;
 use std::ffi::c_void;
-use std::marker::PhantomData;
+use std::sync::{Arc, Mutex};
 
 use smithay_clipboard::WaylandClipboard;
+
 use wayland_client::sys::client::wl_display;
 use wayland_client::Display;
 
 use common::ClipboardProvider;
 
-pub trait ClipboardType: Send {}
+pub struct Clipboard {
+    context: Arc<Mutex<WaylandClipboard>>,
+}
 
-pub struct Clipboard;
-impl ClipboardType for Clipboard {}
+pub struct Primary {
+    context: Arc<Mutex<WaylandClipboard>>,
+}
 
-pub struct Primary;
-impl ClipboardType for Primary {}
+pub fn create_clipboards(display: &Display) -> (Primary, Clipboard) {
+    let context = Arc::new(Mutex::new(WaylandClipboard::new(display)));
 
-pub struct WaylandClipboardContext<T: ClipboardType>(WaylandClipboard, PhantomData<T>);
+    (Primary { context: context.clone() }, Clipboard { context } )
+}
 
-impl<T: ClipboardType> WaylandClipboardContext<T> {
-    /// Create a new clipboard context.
-    pub fn new(display: &Display) -> Self {
-        WaylandClipboardContext(WaylandClipboard::new(display), PhantomData)
-    }
+pub unsafe fn create_clipboards_from_external(display: *mut c_void) -> (Primary, Clipboard) {
+    let context =
+        Arc::new(Mutex::new(WaylandClipboard::new_from_external(display as *mut wl_display)));
 
-    /// Create a new clipboard context from an external pointer.
-    pub unsafe fn new_from_external(display: *mut c_void) -> Self {
-        WaylandClipboardContext(
-            WaylandClipboard::new_from_external(display as *mut wl_display),
-            PhantomData,
-        )
-    }
+    (Primary { context: context.clone() }, Clipboard { context} )
 }
 
-impl ClipboardProvider for WaylandClipboardContext<Clipboard> {
+impl ClipboardProvider for Clipboard {
     fn get_contents(&mut self) -> Result<String, Box<dyn Error>> {
-        Ok(self.0.load(None))
+        Ok(self.context.lock().unwrap().load(None))
     }
 
     fn set_contents(&mut self, data: String) -> Result<(), Box<dyn Error>> {
-        self.0.store(None, data);
+        self.context.lock().unwrap().store(None, data);
+
         Ok(())
     }
 }
 
-impl ClipboardProvider for WaylandClipboardContext<Primary> {
+impl ClipboardProvider for Primary {
     fn get_contents(&mut self) -> Result<String, Box<dyn Error>> {
-        Ok(self.0.load_primary(None))
+        Ok(self.context.lock().unwrap().load_primary(None))
     }
 
     fn set_contents(&mut self, data: String) -> Result<(), Box<dyn Error>> {
-        self.0.store_primary(None, data);
+        self.context.lock().unwrap().store_primary(None, data);
+
         Ok(())
     }
 }