From eaa01c597e08f08962b47a862dfdc2b5a0f63b5d Mon Sep 17 00:00:00 2001
From: Daniel Reiter Horn <danielrh@users.sourceforge.net>
Date: Mon, 24 Aug 2020 15:23:31 -0700
Subject: [PATCH] suppress dangerous clippy hints until
 https://github.com/rust-lang/rust-clippy/issues/3992 is resolved

---
 src/sdl2/macros.rs   |  3 +++
 src/sdl2/rect.rs     |  7 ++++++-
 src/sdl2/render.rs   | 15 +++++++++++++++
 src/sdl2/rwops.rs    |  3 +++
 src/sdl2/surface.rs  |  3 +++
 src/sdl2/ttf/font.rs |  3 +++
 src/sdl2/video.rs    |  3 +++
 7 files changed, 36 insertions(+), 1 deletion(-)

diff --git a/src/sdl2/macros.rs b/src/sdl2/macros.rs
index 4ceaa804546..41100950a3d 100644
--- a/src/sdl2/macros.rs
+++ b/src/sdl2/macros.rs
@@ -3,6 +3,9 @@ macro_rules! impl_raw_accessors(
         $(
         impl $t {
             #[inline]
+            // can prevent introducing UB until
+            // https://github.com/rust-lang/rust-clippy/issues/3992 is fixed
+            #[allow(clippy::trivially_copy_pass_by_ref)]
             pub const unsafe fn raw(&self) -> $raw { self.raw }
         }
         )+
diff --git a/src/sdl2/rect.rs b/src/sdl2/rect.rs
index ba31db69da2..1d9638931bd 100644
--- a/src/sdl2/rect.rs
+++ b/src/sdl2/rect.rs
@@ -391,6 +391,9 @@ impl Rect {
     }
 
     /// Returns the underlying C Rect.
+    // this can prevent introducing UB until
+    // https://github.com/rust-lang/rust-clippy/issues/3992 is fixed
+    #[allow(clippy::trivially_copy_pass_by_ref)]
     pub fn raw(&self) -> *const sys::SDL_Rect {
         &self.raw
     }
@@ -738,7 +741,9 @@ impl Point {
     pub fn raw_slice(slice: &[Point]) -> *const sys::SDL_Point {
         slice.as_ptr() as *const sys::SDL_Point
     }
-
+    // this can prevent introducing UB until
+    // https://github.com/rust-lang/rust-clippy/issues/3992 is fixed
+    #[allow(clippy::trivially_copy_pass_by_ref)]
     pub fn raw(&self) -> *const sys::SDL_Point {
         &self.raw
     }
diff --git a/src/sdl2/render.rs b/src/sdl2/render.rs
index 3b1f94be55a..7792f3067b4 100644
--- a/src/sdl2/render.rs
+++ b/src/sdl2/render.rs
@@ -219,6 +219,9 @@ impl<T> RendererContext<T> {
     }
 
     /// Gets the raw pointer to the SDL_Renderer
+    // this can prevent introducing UB until
+    // https://github.com/rust-lang/rust-clippy/issues/3992 is fixed
+    #[allow(clippy::trivially_copy_pass_by_ref)]
     pub fn raw(&self) -> *mut sys::SDL_Renderer {
         self.raw
     }
@@ -784,6 +787,9 @@ fn ll_create_texture(context: *mut sys::SDL_Renderer,
 
 /// Texture-creating methods for the renderer
 impl<T> TextureCreator<T> {
+    // this can prevent introducing UB until
+    // https://github.com/rust-lang/rust-clippy/issues/3992 is fixed
+    #[allow(clippy::trivially_copy_pass_by_ref)]
     pub fn raw(&self) -> *mut sys::SDL_Renderer {
         self.context.raw()
     }
@@ -922,6 +928,9 @@ impl<T> TextureCreator<T> {
 
 /// Drawing methods
 impl<T: RenderTarget> Canvas<T> {
+    // this can prevent introducing UB until
+    // https://github.com/rust-lang/rust-clippy/issues/3992 is fixed
+    #[allow(clippy::trivially_copy_pass_by_ref)]
     pub fn raw(&self) -> *mut sys::SDL_Renderer {
         self.context.raw()
     }
@@ -2148,6 +2157,9 @@ impl<'r> Texture<'r> {
     }
 
     #[inline]
+    // this can prevent introducing UB until
+    // https://github.com/rust-lang/rust-clippy/issues/3992 is fixed
+    #[allow(clippy::trivially_copy_pass_by_ref)]
     pub const fn raw(&self) -> *mut sys::SDL_Texture {
         self.raw
     }
@@ -2328,6 +2340,9 @@ impl<> Texture<> {
     }
 
     #[inline]
+    // this can prevent introducing UB until
+    // https://github.com/rust-lang/rust-clippy/issues/3992 is fixed
+    #[allow(clippy::trivially_copy_pass_by_ref)]
     pub const fn raw(&self) -> *mut sys::SDL_Texture {
         self.raw
     }
diff --git a/src/sdl2/rwops.rs b/src/sdl2/rwops.rs
index de752f84f99..ec81feed67a 100644
--- a/src/sdl2/rwops.rs
+++ b/src/sdl2/rwops.rs
@@ -16,6 +16,9 @@ pub struct RWops<'a> {
 }
 
 impl<'a> RWops<'a> {
+    // this can prevent introducing UB until
+    // https://github.com/rust-lang/rust-clippy/issues/3992 is fixed
+    #[allow(clippy::trivially_copy_pass_by_ref)]
     pub unsafe fn raw(&self) -> *mut sys::SDL_RWops { self.raw }
 
     pub unsafe fn from_ll<'b>(raw: *mut sys::SDL_RWops) -> RWops<'b> {
diff --git a/src/sdl2/surface.rs b/src/sdl2/surface.rs
index 91eabd6619a..6258fd3defb 100644
--- a/src/sdl2/surface.rs
+++ b/src/sdl2/surface.rs
@@ -281,6 +281,9 @@ impl SurfaceRef {
     }
 
     #[inline]
+    // this can prevent introducing UB until
+    // https://github.com/rust-lang/rust-clippy/issues/3992 is fixed
+    #[allow(clippy::trivially_copy_pass_by_ref)]
     pub fn raw(&self) -> *mut sys::SDL_Surface {
         self as *const SurfaceRef as *mut SurfaceRef as *mut () as *mut sys::SDL_Surface
     }
diff --git a/src/sdl2/ttf/font.rs b/src/sdl2/ttf/font.rs
index 14aa56815c1..0df75d1b5bb 100644
--- a/src/sdl2/ttf/font.rs
+++ b/src/sdl2/ttf/font.rs
@@ -301,6 +301,9 @@ pub fn internal_load_font_at_index<'ttf,P: AsRef<Path>>(path: P, index: u32, pts
 
 impl<'ttf,'r> Font<'ttf,'r> {
     /// Returns the underlying C font object.
+    // this can prevent introducing UB until
+    // https://github.com/rust-lang/rust-clippy/issues/3992 is fixed
+    #[allow(clippy::trivially_copy_pass_by_ref)]
     unsafe fn raw(&self) -> *mut ttf::TTF_Font {
         self.raw
     }
diff --git a/src/sdl2/video.rs b/src/sdl2/video.rs
index a389b62d2c6..5836f99d509 100644
--- a/src/sdl2/video.rs
+++ b/src/sdl2/video.rs
@@ -1075,6 +1075,9 @@ impl From<Window> for CanvasBuilder {
 
 impl Window {
     #[inline]
+    // this can prevent introducing UB until
+    // https://github.com/rust-lang/rust-clippy/issues/3992 is fixed
+    #[allow(clippy::trivially_copy_pass_by_ref)]
     pub fn raw(&self) -> *mut sys::SDL_Window { self.context.raw }
 
     #[inline]