refactor code:

trait `Target` at `src/util/option.rs` be instead of `AsMutPtr`, `AsPtr`

Author: admin

src/codec/context.rs

@@ -7,7 +7,8 @@ use super::encoder::Encoder;
 use super::{threading, Compliance, Debug, Flags, Id};
 use crate::ffi::*;
 use crate::media;
-use crate::AsPtr;
+use crate::option;
+use crate::{AsMutPtr, AsPtr};
 use crate::{Codec, Error};
 use libc::c_int;
 
@@ -167,3 +168,18 @@ impl Clone for Context {
         }
     }
 }
+
+/// `AVCodecContext` in `Context` is the target of `option` operations.
+impl AsPtr<AVCodecContext> for Context {
+    fn as_ptr(&self) -> *const AVCodecContext {
+        self.ptr as *const _
+    }
+}
+
+impl AsMutPtr<AVCodecContext> for Context {
+    fn as_mut_ptr(&mut self) -> *mut AVCodecContext {
+        self.ptr as *mut _
+    }
+}
+
+impl option::Settable<AVCodecContext> for Context {}

src/filter/context/context.rs

@@ -2,8 +2,7 @@ use std::marker::PhantomData;
 
 use super::{Sink, Source};
 use crate::ffi::*;
-use crate::{format, option};
-use libc::c_void;
+use crate::{format, option, AsMutPtr, AsPtr};
 
 #[cfg(feature = "ffmpeg_5_1")]
 use crate::ChannelLayout;
@@ -66,14 +65,16 @@ impl<'a> Context<'a> {
     }
 }
 
-unsafe impl<'a> option::Target for Context<'a> {
-    fn as_ptr(&self) -> *const c_void {
+impl<'a> AsPtr<AVFilterContext> for Context<'a> {
+    fn as_ptr(&self) -> *const AVFilterContext {
         self.ptr as *const _
     }
+}
 
-    fn as_mut_ptr(&mut self) -> *mut c_void {
+impl<'a> AsMutPtr<AVFilterContext> for Context<'a> {
+    fn as_mut_ptr(&mut self) -> *mut AVFilterContext {
         self.ptr as *mut _
     }
 }
 
-impl<'a> option::Settable for Context<'a> {}
+impl<'a> option::Settable<AVFilterContext> for Context<'a> {}

src/util/option/mod.rs

@@ -1,5 +1,5 @@
 mod traits;
-pub use self::traits::{Gettable, Iterable, Settable, Target};
+pub use self::traits::{Gettable, Iterable, Settable};
 
 use crate::ffi::*;
 use libc::c_uint;

src/util/option/traits.rs

@@ -5,8 +5,8 @@ use std::mem;
 
 use crate::ffi::*;
 use crate::util::format;
-use crate::{Error, Rational};
-use libc::{c_int, c_void};
+use crate::{AsMutPtr, AsPtr, Error, Rational};
+use libc::c_int;
 
 #[cfg(not(feature = "ffmpeg_7_0"))]
 use crate::ChannelLayoutMask;
@@ -20,21 +20,16 @@ macro_rules! check {
     };
 }
 
-pub unsafe trait Target {
-    fn as_ptr(&self) -> *const c_void;
-    fn as_mut_ptr(&mut self) -> *mut c_void;
-}
-
-pub trait Settable: Target {
-    fn set<T: 'static>(&mut self, name: &str, value: &T) -> Result<(), Error> {
+pub trait Settable<T>: AsPtr<T> + AsMutPtr<T> {
+    fn set<V: 'static>(&mut self, name: &str, value: &V) -> Result<(), Error> {
         unsafe {
             let name = CString::new(name).unwrap();
 
             check!(av_opt_set_bin(
-                self.as_mut_ptr(),
+                self.as_mut_ptr() as *mut _,
                 name.as_ptr(),
                 value as *const _ as *const _,
-                mem::size_of::<T>() as c_int,
+                mem::size_of::<V>() as c_int,
                 AV_OPT_SEARCH_CHILDREN
             ))
         }
@@ -46,7 +41,7 @@ pub trait Settable: Target {
             let value = CString::new(value).unwrap();
 
             check!(av_opt_set(
-                self.as_mut_ptr(),
+                self.as_mut_ptr() as *mut _,
                 name.as_ptr(),
                 value.as_ptr(),
                 AV_OPT_SEARCH_CHILDREN
@@ -59,7 +54,7 @@ pub trait Settable: Target {
             let name = CString::new(name).unwrap();
 
             check!(av_opt_set_int(
-                self.as_mut_ptr(),
+                self.as_mut_ptr() as *mut _,
                 name.as_ptr(),
                 value,
                 AV_OPT_SEARCH_CHILDREN
@@ -72,20 +67,20 @@ pub trait Settable: Target {
             let name = CString::new(name).unwrap();
 
             check!(av_opt_set_double(
-                self.as_mut_ptr(),
+                self.as_mut_ptr() as *mut _,
                 name.as_ptr(),
                 value,
                 AV_OPT_SEARCH_CHILDREN
             ))
         }
     }
 
-    fn set_rational<T: Into<Rational>>(&mut self, name: &str, value: T) -> Result<(), Error> {
+    fn set_rational<V: Into<Rational>>(&mut self, name: &str, value: V) -> Result<(), Error> {
         unsafe {
             let name = CString::new(name).unwrap();
 
             check!(av_opt_set_q(
-                self.as_mut_ptr(),
+                self.as_mut_ptr() as *mut _,
                 name.as_ptr(),
                 value.into().into(),
                 AV_OPT_SEARCH_CHILDREN
@@ -98,7 +93,7 @@ pub trait Settable: Target {
             let name = CString::new(name).unwrap();
 
             check!(av_opt_set_image_size(
-                self.as_mut_ptr(),
+                self.as_mut_ptr() as *mut _,
                 name.as_ptr(),
                 w as c_int,
                 h as c_int,
@@ -112,7 +107,7 @@ pub trait Settable: Target {
             let name = CString::new(name).unwrap();
 
             check!(av_opt_set_pixel_fmt(
-                self.as_mut_ptr(),
+                self.as_mut_ptr() as *mut _,
                 name.as_ptr(),
                 format.into(),
                 AV_OPT_SEARCH_CHILDREN
@@ -125,7 +120,7 @@ pub trait Settable: Target {
             let name = CString::new(name).unwrap();
 
             check!(av_opt_set_sample_fmt(
-                self.as_mut_ptr(),
+                self.as_mut_ptr() as *mut _,
                 name.as_ptr(),
                 format.into(),
                 AV_OPT_SEARCH_CHILDREN
@@ -139,7 +134,7 @@ pub trait Settable: Target {
             let name = CString::new(name).unwrap();
 
             check!(av_opt_set_channel_layout(
-                self.as_mut_ptr(),
+                self.as_mut_ptr() as *mut _,
                 name.as_ptr(),
                 layout.bits() as i64,
                 AV_OPT_SEARCH_CHILDREN
@@ -148,6 +143,6 @@ pub trait Settable: Target {
     }
 }
 
-pub trait Gettable: Target {}
+pub trait Gettable<T>: AsPtr<T> + AsMutPtr<T> {}
 
-pub trait Iterable: Target {}
+pub trait Iterable<T>: AsPtr<T> + AsMutPtr<T> {}