Clippy fixes

anvil/src/window_map.rs

@@ -56,21 +56,18 @@ impl Kind {
 
     /// Activate/Deactivate this window
     pub fn set_activated(&self, active: bool) {
-        match *self {
+        if let Kind::Xdg(ref t) = self {
-            Kind::Xdg(ref t) => {
+            let changed = t.with_pending_state(|state| {
-                let changed = t.with_pending_state(|state| {
+                if active {
-                    if active {
+                    state.states.set(xdg_toplevel::State::Activated)
-                        state.states.set(xdg_toplevel::State::Activated)
+                } else {
-                    } else {
+                    state.states.unset(xdg_toplevel::State::Activated)
-                        state.states.unset(xdg_toplevel::State::Activated)
-                    }
-                });
-                if let Ok(true) = changed {
-                    t.send_configure();
                 }
+            });
+            if let Ok(true) = changed {
+                t.send_configure();
             }
-            _ => {}
+        }
-        };
     }
 }
 

src/backend/drm/surface/atomic.rs

@@ -752,6 +752,7 @@ impl<A: AsRawFd + 'static> AtomicDrmSurface<A> {
     }
 
     // If a mode is set a matching blob needs to be set (the inverse is not true)
+    #[allow(clippy::too_many_arguments)]
     pub fn build_request<'a>(
         &self,
         new_connectors: &mut dyn Iterator<Item = &connector::Handle>,

src/backend/drm/surface/gbm.rs

@@ -313,7 +313,7 @@ where
 
         let maybe_buffer = slot.userdata().as_ref().map(|(buf, _)| buf.clone());
         let dmabuf = match maybe_buffer {
-            Some(buf) => buf.clone(),
+            Some(buf) => buf,
             None => {
                 let dmabuf = slot.export()?;
                 let fb_handle = attach_framebuffer(&self.drm, &*slot)?;

src/backend/egl/mod.rs

@@ -70,6 +70,10 @@ impl ::std::error::Error for EglExtensionNotSupportedError {}
 /// Returns the address of an OpenGL function.
 ///
 /// Result is independent of displays and does not guarantee an extension is actually supported at runtime.
+///
+/// # Safety
+///
+/// This function should only be invoked while an EGL context is active.
 pub unsafe fn get_proc_address(symbol: &str) -> *const c_void {
     let addr = CString::new(symbol.as_bytes()).unwrap();
     let addr = addr.as_ptr();