From 43f1f2ed1a7f9cb4ae09a3d080856857a76caaf1 Mon Sep 17 00:00:00 2001 From: Andelf Date: Sat, 31 May 2014 17:36:45 +0800 Subject: [PATCH] fix compile error: string change, raw accessor change --- src/demo/main.rs | 2 +- src/sdl2_image/lib.rs | 78 +++++++++++++++++++++---------------------- 2 files changed, 39 insertions(+), 41 deletions(-) diff --git a/src/demo/main.rs b/src/demo/main.rs index 8ff5fa81..75e2364e 100644 --- a/src/demo/main.rs +++ b/src/demo/main.rs @@ -19,6 +19,6 @@ fn main() { if args.len() < 2 { println!("Usage: ./demo image.[png|jpg]") } else { - video::main(&Path::new(args.get(1).to_owned())); + video::main(&Path::new(args.get(1).to_string())); } } diff --git a/src/sdl2_image/lib.rs b/src/sdl2_image/lib.rs index 81baa46d..402da7d4 100644 --- a/src/sdl2_image/lib.rs +++ b/src/sdl2_image/lib.rs @@ -18,9 +18,7 @@ use sdl2::render::Renderer; use sdl2::rwops::RWops; use sdl2::version::Version; use sdl2::get_error; - -// FIXME: this should be done in rust-sdl2 -pub type SdlResult = Result; +use sdl2::SdlResult; // Setup linking for all targets. #[cfg(target_os="macos")] @@ -77,7 +75,7 @@ impl LoadSurface for Surface { if raw == ptr::null() { Err(get_error()) } else { - Ok(Surface { raw: raw, owned: true }) + Ok(Surface::from_ll(raw, true)) } } } @@ -89,7 +87,7 @@ impl LoadSurface for Surface { if raw == ptr::null() { Err(get_error()) } else { - Ok(Surface { raw: raw, owned: true }) + Ok(Surface::from_ll(raw, true)) } } } @@ -99,7 +97,7 @@ impl SaveSurface for Surface { fn save(&self, filename: &Path) -> SdlResult<()> { //! Saves an SDL Surface to a file unsafe { - let status = ffi::IMG_SavePNG(self.raw, + let status = ffi::IMG_SavePNG(self.raw(), filename.to_c_str().unwrap()); if status != 0 { Err(get_error()) @@ -112,7 +110,7 @@ impl SaveSurface for Surface { fn save_rw(&self, dst: &mut RWops) -> SdlResult<()> { //! Saves an SDL Surface to an RWops unsafe { - let status = ffi::IMG_SavePNG_RW(self.raw, dst.raw, 0); + let status = ffi::IMG_SavePNG_RW(self.raw(), dst.raw(), 0); if status != 0 { Err(get_error()) @@ -132,7 +130,7 @@ impl LoadTexture for Renderer { fn load_texture(&self, filename: &Path) -> SdlResult { //! Loads an SDL Texture from a file unsafe { - let raw = ffi::IMG_LoadTexture(self.raw, + let raw = ffi::IMG_LoadTexture(self.raw(), filename.to_c_str().unwrap()); if raw == ptr::null() { Err(get_error()) @@ -169,7 +167,7 @@ fn to_surface_result(raw: *sdl2::surface::ll::SDL_Surface) -> SdlResult if raw == ptr::null() { Err(get_error()) } else { - Ok(Surface { raw: raw, owned: true }) + unsafe { Ok(Surface::from_ll(raw, true)) } } } @@ -214,118 +212,118 @@ pub trait ImageRWops { impl ImageRWops for RWops { fn load(&self) -> SdlResult { let raw = unsafe { - ffi::IMG_Load_RW(self.raw, 0) + ffi::IMG_Load_RW(self.raw(), 0) }; to_surface_result(raw) } fn load_typed(&self, _type: &str) -> SdlResult { let raw = unsafe { - ffi::IMG_LoadTyped_RW(self.raw, 0, _type.to_c_str().unwrap()) + ffi::IMG_LoadTyped_RW(self.raw(), 0, _type.to_c_str().unwrap()) }; to_surface_result(raw) } fn load_cur(&self) -> SdlResult { - let raw = unsafe { ffi::IMG_LoadCUR_RW(self.raw) }; + let raw = unsafe { ffi::IMG_LoadCUR_RW(self.raw()) }; to_surface_result(raw) } fn load_ico(&self) -> SdlResult { - let raw = unsafe { ffi::IMG_LoadICO_RW(self.raw) }; + let raw = unsafe { ffi::IMG_LoadICO_RW(self.raw()) }; to_surface_result(raw) } fn load_bmp(&self) -> SdlResult { - let raw = unsafe { ffi::IMG_LoadBMP_RW(self.raw) }; + let raw = unsafe { ffi::IMG_LoadBMP_RW(self.raw()) }; to_surface_result(raw) } fn load_pnm(&self) -> SdlResult { - let raw = unsafe { ffi::IMG_LoadPNM_RW(self.raw) }; + let raw = unsafe { ffi::IMG_LoadPNM_RW(self.raw()) }; to_surface_result(raw) } fn load_xpm(&self) -> SdlResult { - let raw = unsafe { ffi::IMG_LoadXPM_RW(self.raw) }; + let raw = unsafe { ffi::IMG_LoadXPM_RW(self.raw()) }; to_surface_result(raw) } fn load_xcf(&self) -> SdlResult { - let raw = unsafe { ffi::IMG_LoadXCF_RW(self.raw) }; + let raw = unsafe { ffi::IMG_LoadXCF_RW(self.raw()) }; to_surface_result(raw) } fn load_pcx(&self) -> SdlResult { - let raw = unsafe { ffi::IMG_LoadPCX_RW(self.raw) }; + let raw = unsafe { ffi::IMG_LoadPCX_RW(self.raw()) }; to_surface_result(raw) } fn load_gif(&self) -> SdlResult { - let raw = unsafe { ffi::IMG_LoadGIF_RW(self.raw) }; + let raw = unsafe { ffi::IMG_LoadGIF_RW(self.raw()) }; to_surface_result(raw) } fn load_jpg(&self) -> SdlResult { - let raw = unsafe { ffi::IMG_LoadJPG_RW(self.raw) }; + let raw = unsafe { ffi::IMG_LoadJPG_RW(self.raw()) }; to_surface_result(raw) } fn load_tif(&self) -> SdlResult { - let raw = unsafe { ffi::IMG_LoadTIF_RW(self.raw) }; + let raw = unsafe { ffi::IMG_LoadTIF_RW(self.raw()) }; to_surface_result(raw) } fn load_png(&self) -> SdlResult { - let raw = unsafe { ffi::IMG_LoadPNG_RW(self.raw) }; + let raw = unsafe { ffi::IMG_LoadPNG_RW(self.raw()) }; to_surface_result(raw) } fn load_tga(&self) -> SdlResult { - let raw = unsafe { ffi::IMG_LoadTGA_RW(self.raw) }; + let raw = unsafe { ffi::IMG_LoadTGA_RW(self.raw()) }; to_surface_result(raw) } fn load_lbm(&self) -> SdlResult { - let raw = unsafe { ffi::IMG_LoadLBM_RW(self.raw) }; + let raw = unsafe { ffi::IMG_LoadLBM_RW(self.raw()) }; to_surface_result(raw) } fn load_xv(&self) -> SdlResult { - let raw = unsafe { ffi::IMG_LoadXV_RW(self.raw) }; + let raw = unsafe { ffi::IMG_LoadXV_RW(self.raw()) }; to_surface_result(raw) } fn load_webp(&self) -> SdlResult { - let raw = unsafe { ffi::IMG_LoadWEBP_RW(self.raw) }; + let raw = unsafe { ffi::IMG_LoadWEBP_RW(self.raw()) }; to_surface_result(raw) } fn is_cur(&self) -> bool { - unsafe { ffi::IMG_isCUR(self.raw) == 1 } + unsafe { ffi::IMG_isCUR(self.raw()) == 1 } } fn is_ico(&self) -> bool { - unsafe { ffi::IMG_isICO(self.raw) == 1 } + unsafe { ffi::IMG_isICO(self.raw()) == 1 } } fn is_bmp(&self) -> bool { - unsafe { ffi::IMG_isBMP(self.raw) == 1 } + unsafe { ffi::IMG_isBMP(self.raw()) == 1 } } fn is_pnm(&self) -> bool { - unsafe { ffi::IMG_isPNM(self.raw) == 1 } + unsafe { ffi::IMG_isPNM(self.raw()) == 1 } } fn is_xpm(&self) -> bool { - unsafe { ffi::IMG_isXPM(self.raw) == 1 } + unsafe { ffi::IMG_isXPM(self.raw()) == 1 } } fn is_xcf(&self) -> bool { - unsafe { ffi::IMG_isXCF(self.raw) == 1 } + unsafe { ffi::IMG_isXCF(self.raw()) == 1 } } fn is_pcx(&self) -> bool { - unsafe { ffi::IMG_isPCX(self.raw) == 1 } + unsafe { ffi::IMG_isPCX(self.raw()) == 1 } } fn is_gif(&self) -> bool { - unsafe { ffi::IMG_isGIF(self.raw) == 1 } + unsafe { ffi::IMG_isGIF(self.raw()) == 1 } } fn is_jpg(&self) -> bool { - unsafe { ffi::IMG_isJPG(self.raw) == 1 } + unsafe { ffi::IMG_isJPG(self.raw()) == 1 } } fn is_tif(&self) -> bool { - unsafe { ffi::IMG_isTIF(self.raw) == 1 } + unsafe { ffi::IMG_isTIF(self.raw()) == 1 } } fn is_png(&self) -> bool { - unsafe { ffi::IMG_isPNG(self.raw) == 1 } + unsafe { ffi::IMG_isPNG(self.raw()) == 1 } } fn is_lbm(&self) -> bool { - unsafe { ffi::IMG_isLBM(self.raw) == 1 } + unsafe { ffi::IMG_isLBM(self.raw()) == 1 } } fn is_xv(&self) -> bool { - unsafe { ffi::IMG_isXV(self.raw) == 1 } + unsafe { ffi::IMG_isXV(self.raw()) == 1 } } fn is_webp(&self) -> bool { - unsafe { ffi::IMG_isWEBP(self.raw) == 1 } + unsafe { ffi::IMG_isWEBP(self.raw()) == 1 } } }