Merge pull request #704 from GuillaumeGomez/clippy

Fix new clippy lints

Author: sdroege

gdk/src/keys.rs

@@ -82,12 +82,14 @@ impl Key {
     }
 
     #[doc(alias = "gdk_keyval_to_upper")]
+    #[must_use]
     pub fn to_upper(&self) -> Self {
         skip_assert_initialized!();
         unsafe { ffi::gdk_keyval_to_upper(**self) }.into()
     }
 
     #[doc(alias = "gdk_keyval_to_lower")]
+    #[must_use]
     pub fn to_lower(&self) -> Self {
         skip_assert_initialized!();
         unsafe { ffi::gdk_keyval_to_lower(**self) }.into()

gdk/src/rt.rs

@@ -55,7 +55,6 @@ pub fn is_initialized_main_thread() -> bool {
 }
 
 /// Informs this crate that GDK has been initialized and the current thread is the main one.
-#[allow(clippy::if_then_panic)]
 pub unsafe fn set_initialized() {
     skip_assert_initialized!();
     if is_initialized_main_thread() {

gtk/src/rt.rs

@@ -72,7 +72,6 @@ pub fn is_initialized_main_thread() -> bool {
 /// 1. You have initialised the underlying GTK library yourself.
 /// 2. You did 1 on the thread with which you are calling this function
 /// 3. You ensure that this thread is the main thread for the process.
-#[allow(clippy::if_then_panic)]
 pub unsafe fn set_initialized() {
     skip_assert_initialized!();
     if is_initialized_main_thread() {
@@ -108,7 +107,6 @@ pub unsafe fn set_initialized() {
 /// Instead, an Ok is returned if the windowing system was successfully
 /// initialized otherwise an Err is returned.
 #[doc(alias = "gtk_init")]
-#[allow(clippy::if_then_panic)]
 pub fn init() -> Result<(), glib::BoolError> {
     skip_assert_initialized!();
     if is_initialized_main_thread() {
@@ -139,7 +137,6 @@ pub fn init() -> Result<(), glib::BoolError> {
 }
 
 #[doc(alias = "gtk_main_quit")]
-#[allow(clippy::if_then_panic)]
 pub fn main_quit() {
     assert_initialized_main_thread!();
     unsafe {