From afdd000d92706efe06a082e1bb9517ae07853754 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sebastian=20Dr=C3=B6ge?= Date: Fri, 30 Jul 2021 13:53:35 +0300 Subject: [PATCH] Fix various needless-borrow clippy warnings and others --- audio/audiofx/src/audioloudnorm/imp.rs | 4 ++-- audio/audiofx/src/ebur128level/imp.rs | 2 +- generic/file/src/filesink/imp.rs | 2 +- generic/file/src/filesrc/imp.rs | 2 +- generic/sodium/src/decrypter/imp.rs | 11 +++++------ generic/sodium/src/encrypter/imp.rs | 4 ++-- generic/threadshare/src/appsrc/imp.rs | 2 +- generic/threadshare/src/inputselector/imp.rs | 6 +++--- generic/threadshare/src/jitterbuffer/imp.rs | 2 +- generic/threadshare/src/proxy/imp.rs | 2 +- generic/threadshare/src/queue/imp.rs | 4 ++-- generic/threadshare/src/runtime/task.rs | 2 +- generic/threadshare/src/tcpclientsrc/imp.rs | 2 +- generic/threadshare/src/udpsink/imp.rs | 16 ++++++++-------- generic/threadshare/src/udpsrc/imp.rs | 2 +- generic/threadshare/tests/pad.rs | 18 +++++++++--------- generic/threadshare/tests/udpsink.rs | 2 +- net/reqwest/src/reqwesthttpsrc/imp.rs | 2 +- net/rusoto/src/aws_transcriber/imp.rs | 6 +++--- text/json/src/jsongstparse/imp.rs | 6 +++--- text/json/tests/json.rs | 4 ++-- text/regex/src/gstregex/imp.rs | 2 +- .../examples/gtk_fallbackswitch.rs | 2 +- .../src/fallbacksrc/custom_source/imp.rs | 6 +++--- utils/fallbackswitch/src/fallbackswitch/imp.rs | 12 +++++------- utils/fallbackswitch/tests/fallbackswitch.rs | 2 +- video/cdg/src/cdgdec/imp.rs | 2 +- video/closedcaption/src/cea608overlay/imp.rs | 10 +++++----- video/closedcaption/src/mcc_parse/imp.rs | 6 +++--- video/closedcaption/src/scc_parse/imp.rs | 6 +++--- video/closedcaption/src/transcriberbin/imp.rs | 2 +- video/dav1d/src/dav1ddec/imp.rs | 4 ++-- video/flavors/src/flvdemux/imp.rs | 4 ++-- video/gif/tests/gifenc.rs | 2 +- video/rav1e/tests/rav1enc.rs | 2 +- video/rspng/tests/pngenc.rs | 2 +- 36 files changed, 81 insertions(+), 84 deletions(-) diff --git a/audio/audiofx/src/audioloudnorm/imp.rs b/audio/audiofx/src/audioloudnorm/imp.rs index e6d7e614..22eccec0 100644 --- a/audio/audiofx/src/audioloudnorm/imp.rs +++ b/audio/audiofx/src/audioloudnorm/imp.rs @@ -1622,7 +1622,7 @@ impl AudioLoudNorm { let mut state = self.state.borrow_mut(); let mut outbuf = None; if let Some(ref mut state) = &mut *state { - outbuf = match state.drain(&element) { + outbuf = match state.drain(element) { Ok(outbuf) => Some(outbuf), Err(gst::FlowError::Eos) => None, Err(_) => return false, @@ -1644,7 +1644,7 @@ impl AudioLoudNorm { let mut state = self.state.borrow_mut(); let mut outbuf = None; if let Some(ref mut state) = &mut *state { - outbuf = match state.drain(&element) { + outbuf = match state.drain(element) { Ok(outbuf) => Some(outbuf), Err(gst::FlowError::Eos) => None, Err(_) => return false, diff --git a/audio/audiofx/src/ebur128level/imp.rs b/audio/audiofx/src/ebur128level/imp.rs index 3033db17..8b9d7236 100644 --- a/audio/audiofx/src/ebur128level/imp.rs +++ b/audio/audiofx/src/ebur128level/imp.rs @@ -436,7 +436,7 @@ impl BaseTransformImpl for EbuR128Level { let mut timestamp = buf.pts(); let segment = element.segment().downcast::().ok(); - let buf = gst_audio::AudioBufferRef::from_buffer_ref_readable(&buf, &state.info).map_err( + let buf = gst_audio::AudioBufferRef::from_buffer_ref_readable(buf, &state.info).map_err( |_| { gst::element_error!(element, gst::ResourceError::Read, ["Failed to map buffer"]); gst::FlowError::Error diff --git a/generic/file/src/filesink/imp.rs b/generic/file/src/filesink/imp.rs index 7a0dd9e7..26225493 100644 --- a/generic/file/src/filesink/imp.rs +++ b/generic/file/src/filesink/imp.rs @@ -313,7 +313,7 @@ impl URIHandlerImpl for FileSink { if uri != "file://" { let file_location = FileLocation::try_from_uri_str(uri)?; - self.set_location(&element, Some(file_location)) + self.set_location(element, Some(file_location)) } else { Ok(()) } diff --git a/generic/file/src/filesrc/imp.rs b/generic/file/src/filesrc/imp.rs index be01cb90..ca4bef71 100644 --- a/generic/file/src/filesrc/imp.rs +++ b/generic/file/src/filesrc/imp.rs @@ -366,7 +366,7 @@ impl URIHandlerImpl for FileSrc { if uri != "file://" { let file_location = FileLocation::try_from_uri_str(uri)?; - self.set_location(&element, Some(file_location)) + self.set_location(element, Some(file_location)) } else { Ok(()) } diff --git a/generic/sodium/src/decrypter/imp.rs b/generic/sodium/src/decrypter/imp.rs index 428b252c..050470c4 100644 --- a/generic/sodium/src/decrypter/imp.rs +++ b/generic/sodium/src/decrypter/imp.rs @@ -58,7 +58,7 @@ impl State { let sender_key = props .sender_key .as_ref() - .and_then(|k| box_::PublicKey::from_slice(&k)) + .and_then(|k| box_::PublicKey::from_slice(k)) .ok_or_else(|| { gst::error_msg!( gst::ResourceError::NotFound, @@ -73,7 +73,7 @@ impl State { let receiver_key = props .receiver_key .as_ref() - .and_then(|k| box_::SecretKey::from_slice(&k)) + .and_then(|k| box_::SecretKey::from_slice(k)) .ok_or_else(|| { gst::error_msg!( gst::ResourceError::NotFound, @@ -121,16 +121,15 @@ impl State { let block_size = self.block_size.expect("Block size wasn't set") as usize + box_::MACBYTES; for subbuffer in map.chunks(block_size) { - let plain = box_::open_precomputed(&subbuffer, &nonce, &self.precomputed_key).map_err( - |_| { + let plain = + box_::open_precomputed(subbuffer, &nonce, &self.precomputed_key).map_err(|_| { gst::element_error!( element, gst::StreamError::Format, ["Failed to decrypt buffer"] ); gst::FlowError::Error - }, - )?; + })?; // assumes little endian nonce.increment_le_inplace(); self.adapter.push(gst::Buffer::from_mut_slice(plain)); diff --git a/generic/sodium/src/encrypter/imp.rs b/generic/sodium/src/encrypter/imp.rs index ac1f1f40..d2bf78cb 100644 --- a/generic/sodium/src/encrypter/imp.rs +++ b/generic/sodium/src/encrypter/imp.rs @@ -73,7 +73,7 @@ impl State { let sender_key = props .sender_key .as_ref() - .and_then(|k| box_::SecretKey::from_slice(&k)) + .and_then(|k| box_::SecretKey::from_slice(k)) .ok_or_else(|| { gst::error_msg!( gst::ResourceError::NotFound, @@ -88,7 +88,7 @@ impl State { let receiver_key = props .receiver_key .as_ref() - .and_then(|k| box_::PublicKey::from_slice(&k)) + .and_then(|k| box_::PublicKey::from_slice(k)) .ok_or_else(|| { gst::error_msg!( gst::ResourceError::NotFound, diff --git a/generic/threadshare/src/appsrc/imp.rs b/generic/threadshare/src/appsrc/imp.rs index 2fdf6cff..2f5dd428 100644 --- a/generic/threadshare/src/appsrc/imp.rs +++ b/generic/threadshare/src/appsrc/imp.rs @@ -134,7 +134,7 @@ impl AppSrcPadHandler { pad.push_event(stream_start_evt).await; if let Some(ref caps) = state.caps { - pad.push_event(gst::event::Caps::new(&caps)).await; + pad.push_event(gst::event::Caps::new(caps)).await; *self.0.configured_caps.lock().unwrap() = Some(caps.clone()); } diff --git a/generic/threadshare/src/inputselector/imp.rs b/generic/threadshare/src/inputselector/imp.rs index 080816fd..0bccc227 100644 --- a/generic/threadshare/src/inputselector/imp.rs +++ b/generic/threadshare/src/inputselector/imp.rs @@ -110,7 +110,7 @@ impl InputSelectorPadSinkHandler { if let Some(segment) = &inner.segment { if let Some(segment) = segment.downcast_ref::() { let rtime = segment.to_running_time(buffer.pts()); - let (sync_fut, abort_handle) = abortable(self.sync(&element, rtime)); + let (sync_fut, abort_handle) = abortable(self.sync(element, rtime)); inner.abort_handle = Some(abort_handle); sync_future = Some(sync_fut.map_err(|_| gst::FlowError::Flushing)); } @@ -479,7 +479,7 @@ impl ObjectImpl for InputSelector { let pads = self.pads.lock().unwrap(); let mut old_pad = None; if let Some(ref pad) = pad { - if pads.sink_pads.get(&pad).is_some() { + if pads.sink_pads.get(pad).is_some() { old_pad = state.active_sinkpad.clone(); state.active_sinkpad = Some(pad.clone()); state.switched_pad = true; @@ -607,7 +607,7 @@ impl ElementImpl for InputSelector { let mut state = self.state.lock().unwrap(); let mut pads = self.pads.lock().unwrap(); let sink_pad = - gst::Pad::from_template(&templ, Some(format!("sink_{}", pads.pad_serial).as_str())); + gst::Pad::from_template(templ, Some(format!("sink_{}", pads.pad_serial).as_str())); pads.pad_serial += 1; sink_pad.set_active(true).unwrap(); element.add_pad(&sink_pad).unwrap(); diff --git a/generic/threadshare/src/jitterbuffer/imp.rs b/generic/threadshare/src/jitterbuffer/imp.rs index 22204bef..a27e1fa0 100644 --- a/generic/threadshare/src/jitterbuffer/imp.rs +++ b/generic/threadshare/src/jitterbuffer/imp.rs @@ -525,7 +525,7 @@ impl SinkHandler { // Reschedule if needed let (_, next_wakeup) = jb.src_pad_handler - .next_wakeup(&element, &state, latency, context_wait); + .next_wakeup(element, &state, latency, context_wait); if let Some((next_wakeup, _)) = next_wakeup { if let Some((previous_next_wakeup, ref abort_handle)) = state.wait_handle { if previous_next_wakeup.is_none() diff --git a/generic/threadshare/src/proxy/imp.rs b/generic/threadshare/src/proxy/imp.rs index b24fe580..0e590459 100644 --- a/generic/threadshare/src/proxy/imp.rs +++ b/generic/threadshare/src/proxy/imp.rs @@ -902,7 +902,7 @@ impl TaskImpl for ProxySrcTask { let pad = self.src_pad.upgrade().expect("PadSrc no longer exists"); let proxysrc = ProxySrc::from_instance(&self.element); - let res = ProxySrcPadHandler::push_item(&pad, &proxysrc, item).await; + let res = ProxySrcPadHandler::push_item(&pad, proxysrc, item).await; match res { Ok(()) => { gst_log!(SRC_CAT, obj: &self.element, "Successfully pushed item"); diff --git a/generic/threadshare/src/queue/imp.rs b/generic/threadshare/src/queue/imp.rs index b4501cb0..ec8f8951 100644 --- a/generic/threadshare/src/queue/imp.rs +++ b/generic/threadshare/src/queue/imp.rs @@ -368,7 +368,7 @@ impl TaskImpl for QueueTask { let pad = self.src_pad.upgrade().expect("PadSrc no longer exists"); let queue = Queue::from_instance(&self.element); - let res = QueuePadSrcHandler::push_item(&pad, &queue, item).await; + let res = QueuePadSrcHandler::push_item(&pad, queue, item).await; match res { Ok(()) => { gst_log!(CAT, obj: &self.element, "Successfully pushed item"); @@ -558,7 +558,7 @@ impl Queue { let mut pending_queue = self.pending_queue.lock().unwrap(); - if let Err(item) = self.queue_until_full(&dataqueue, &mut pending_queue, item) { + if let Err(item) = self.queue_until_full(dataqueue, &mut pending_queue, item) { if pending_queue .as_ref() .map(|pq| !pq.scheduled) diff --git a/generic/threadshare/src/runtime/task.rs b/generic/threadshare/src/runtime/task.rs index 7c2ff256..aa649795 100644 --- a/generic/threadshare/src/runtime/task.rs +++ b/generic/threadshare/src/runtime/task.rs @@ -1027,7 +1027,7 @@ impl StateMachine { task_inner: &Arc>, context: &Context, ) -> Self { - let task_inner_clone = Arc::clone(&task_inner); + let task_inner_clone = Arc::clone(task_inner); let loop_fut = async move { let mut res = self.task_impl.start().await; if res.is_ok() { diff --git a/generic/threadshare/src/tcpclientsrc/imp.rs b/generic/threadshare/src/tcpclientsrc/imp.rs index 748f5ead..b832ee9f 100644 --- a/generic/threadshare/src/tcpclientsrc/imp.rs +++ b/generic/threadshare/src/tcpclientsrc/imp.rs @@ -148,7 +148,7 @@ impl TcpClientSrcPadHandler { pad.push_event(stream_start_evt).await; if let Some(ref caps) = state.caps { - pad.push_event(gst::event::Caps::new(&caps)).await; + pad.push_event(gst::event::Caps::new(caps)).await; *self.0.configured_caps.lock().unwrap() = Some(caps.clone()); } diff --git a/generic/threadshare/src/udpsink/imp.rs b/generic/threadshare/src/udpsink/imp.rs index 7e147def..2d6bcb8e 100644 --- a/generic/threadshare/src/udpsink/imp.rs +++ b/generic/threadshare/src/udpsink/imp.rs @@ -441,7 +441,7 @@ impl UdpSinkPadHandler { if !clients_to_configure.is_empty() { for client in &clients_to_configure { - self.configure_client(&settings, &mut socket, &mut socket_v6, &client) + self.configure_client(&settings, &mut socket, &mut socket_v6, client) .map_err(|err| { element_error!( element, @@ -456,7 +456,7 @@ impl UdpSinkPadHandler { if !clients_to_unconfigure.is_empty() { for client in &clients_to_unconfigure { - self.unconfigure_client(&settings, &mut socket, &mut socket_v6, &client) + self.unconfigure_client(&settings, &mut socket, &mut socket_v6, client) .map_err(|err| { element_error!( element, @@ -470,7 +470,7 @@ impl UdpSinkPadHandler { } if do_sync { - self.sync(&element, rtime).await; + self.sync(element, rtime).await; } let data = buffer.map_readable().map_err(|_| { @@ -876,7 +876,7 @@ impl UdpSink { self.prepare_socket(SocketFamily::Ipv6, &context, element)?; self.task - .prepare(UdpSinkTask::new(&element, &self.sink_pad_handler), context) + .prepare(UdpSinkTask::new(element, &self.sink_pad_handler), context) .map_err(|err| { error_msg!( gst::ResourceError::OpenRead, @@ -916,17 +916,17 @@ impl UdpSink { impl UdpSink { fn clear_clients(&self, clients_to_add: impl Iterator) { self.sink_pad_handler - .clear_clients(&self.sink_pad.gst_pad(), clients_to_add); + .clear_clients(self.sink_pad.gst_pad(), clients_to_add); } fn remove_client(&self, addr: SocketAddr) { self.sink_pad_handler - .remove_client(&self.sink_pad.gst_pad(), addr); + .remove_client(self.sink_pad.gst_pad(), addr); } fn add_client(&self, addr: SocketAddr) { self.sink_pad_handler - .add_client(&self.sink_pad.gst_pad(), addr); + .add_client(self.sink_pad.gst_pad(), addr); } } @@ -1249,7 +1249,7 @@ impl ObjectImpl for UdpSink { .map_err(|err| { gst_error!(CAT, obj: obj, "Invalid port {}: {}", rsplit[0], err); }) - .and_then(|port| try_into_socket_addr(&obj, rsplit[1], port)) + .and_then(|port| try_into_socket_addr(obj, rsplit[1], port)) .ok() } else { None diff --git a/generic/threadshare/src/udpsrc/imp.rs b/generic/threadshare/src/udpsrc/imp.rs index 8facdb49..cccf7c17 100644 --- a/generic/threadshare/src/udpsrc/imp.rs +++ b/generic/threadshare/src/udpsrc/imp.rs @@ -165,7 +165,7 @@ impl UdpSrcPadHandler { pad.push_event(stream_start_evt).await; if let Some(ref caps) = state.caps { - pad.push_event(gst::event::Caps::new(&caps)).await; + pad.push_event(gst::event::Caps::new(caps)).await; *self.0.configured_caps.lock().unwrap() = Some(caps.clone()); } diff --git a/generic/threadshare/tests/pad.rs b/generic/threadshare/tests/pad.rs index 1c7b84c6..6f9a5182 100644 --- a/generic/threadshare/tests/pad.rs +++ b/generic/threadshare/tests/pad.rs @@ -524,7 +524,7 @@ mod imp_sink { match event.view() { EventView::FlushStart(..) => { - elem_sink_test.stop(&element.downcast_ref::().unwrap()); + elem_sink_test.stop(element.downcast_ref::().unwrap()); true } _ => false, @@ -953,9 +953,9 @@ fn nominal_scenario( fn src_sink_nominal() { let name = "src_sink_nominal"; - let (pipeline, src_element, _sink_element, receiver) = setup(&name, None, None); + let (pipeline, src_element, _sink_element, receiver) = setup(name, None, None); - nominal_scenario(&name, pipeline, src_element, receiver); + nominal_scenario(name, pipeline, src_element, receiver); } #[test] @@ -974,7 +974,7 @@ fn src_tsqueue_sink_nominal() { let (pipeline, src_element, _sink_element, receiver) = setup(name, Some(ts_queue), None); - nominal_scenario(&name, pipeline, src_element, receiver); + nominal_scenario(name, pipeline, src_element, receiver); } #[test] @@ -986,7 +986,7 @@ fn src_queue_sink_nominal() { let queue = gst::ElementFactory::make("queue", Some("queue")).unwrap(); let (pipeline, src_element, _sink_element, receiver) = setup(name, Some(queue), None); - nominal_scenario(&name, pipeline, src_element, receiver); + nominal_scenario(name, pipeline, src_element, receiver); } #[test] @@ -1014,7 +1014,7 @@ fn src_tsproxy_sink_nominal() { let (pipeline, src_element, _sink_element, receiver) = setup(name, Some(ts_proxy_sink), Some(ts_proxy_src)); - nominal_scenario(&name, pipeline, src_element, receiver); + nominal_scenario(name, pipeline, src_element, receiver); } #[test] @@ -1023,7 +1023,7 @@ fn start_pause_start() { let scenario_name = "start_pause_start"; - let (pipeline, src_element, _sink_element, mut receiver) = setup(&scenario_name, None, None); + let (pipeline, src_element, _sink_element, mut receiver) = setup(scenario_name, None, None); let elem_src_test = imp_src::ElementSrcTest::from_instance(&src_element); @@ -1101,7 +1101,7 @@ fn start_stop_start() { let scenario_name = "start_stop_start"; - let (pipeline, src_element, _sink_element, mut receiver) = setup(&scenario_name, None, None); + let (pipeline, src_element, _sink_element, mut receiver) = setup(scenario_name, None, None); let elem_src_test = imp_src::ElementSrcTest::from_instance(&src_element); @@ -1214,7 +1214,7 @@ fn start_flush() { let scenario_name = "start_flush"; - let (pipeline, src_element, sink_element, mut receiver) = setup(&scenario_name, None, None); + let (pipeline, src_element, sink_element, mut receiver) = setup(scenario_name, None, None); let elem_src_test = imp_src::ElementSrcTest::from_instance(&src_element); diff --git a/generic/threadshare/tests/udpsink.rs b/generic/threadshare/tests/udpsink.rs index 4590ceef..1673adec 100644 --- a/generic/threadshare/tests/udpsink.rs +++ b/generic/threadshare/tests/udpsink.rs @@ -122,7 +122,7 @@ fn test_chain() { init(); let mut h = gst_check::Harness::new("ts-udpsink"); - h.set_src_caps_str(&"foo/bar"); + h.set_src_caps_str("foo/bar"); { let udpsink = h.element().unwrap(); udpsink.set_property("clients", &"127.0.0.1:5005").unwrap(); diff --git a/net/reqwest/src/reqwesthttpsrc/imp.rs b/net/reqwest/src/reqwesthttpsrc/imp.rs index ee1cadf3..7c58e26f 100644 --- a/net/reqwest/src/reqwesthttpsrc/imp.rs +++ b/net/reqwest/src/reqwesthttpsrc/imp.rs @@ -1277,7 +1277,7 @@ impl URIHandlerImpl for ReqwestHttpSrc { } fn set_uri(&self, element: &Self::Type, uri: &str) -> Result<(), glib::Error> { - self.set_location(&element, Some(uri)) + self.set_location(element, Some(uri)) } } diff --git a/net/rusoto/src/aws_transcriber/imp.rs b/net/rusoto/src/aws_transcriber/imp.rs index 4a5635e7..37936b4e 100644 --- a/net/rusoto/src/aws_transcriber/imp.rs +++ b/net/rusoto/src/aws_transcriber/imp.rs @@ -479,7 +479,7 @@ impl Transcriber { if packet_is_exception(&pkt) { let message: ExceptionMessage = - serde_json::from_str(&payload).map_err(|err| { + serde_json::from_str(payload).map_err(|err| { gst_error!( CAT, obj: element, @@ -505,7 +505,7 @@ impl Transcriber { )); } - let transcript: Transcript = serde_json::from_str(&payload).map_err(|err| { + let transcript: Transcript = serde_json::from_str(payload).map_err(|err| { error_msg!( gst::StreamError::Failed, ["Unexpected binary message: {} ({})", payload, err] @@ -795,7 +795,7 @@ impl Transcriber { self.ensure_connection(element).map_err(|err| { element_error!( - &element, + element, gst::StreamError::Failed, ["Streaming failed: {}", err] ); diff --git a/text/json/src/jsongstparse/imp.rs b/text/json/src/jsongstparse/imp.rs index b722602a..78289922 100644 --- a/text/json/src/jsongstparse/imp.rs +++ b/text/json/src/jsongstparse/imp.rs @@ -130,7 +130,7 @@ impl State { } }; - let line: Line = serde_json::from_slice(&line).map_err(|err| (line, err))?; + let line: Line = serde_json::from_slice(line).map_err(|err| (line, err))?; Ok(Some(line)) } @@ -181,7 +181,7 @@ impl State { self.need_segment = false; } - events.extend(self.pending_events.drain(..)); + events.append(&mut self.pending_events); events } @@ -501,7 +501,7 @@ impl JsonGstParse { pts, duration, data: _data, - }) = serde_json::from_slice(&line) + }) = serde_json::from_slice(line) { last_pts = pts.zip(duration).map(|(pts, duration)| pts + duration); } diff --git a/text/json/tests/json.rs b/text/json/tests/json.rs index 80df4c2d..bbe35fb5 100644 --- a/text/json/tests/json.rs +++ b/text/json/tests/json.rs @@ -88,8 +88,8 @@ fn test_parse() { match ev.view() { EventView::Caps(ev) => { assert!(ev.caps().is_strictly_equal(&gst::Caps::new_simple( - &"application/x-json", - &[(&"format", &"test")] + "application/x-json", + &[("format", &"test")] ))); } _ => (), diff --git a/text/regex/src/gstregex/imp.rs b/text/regex/src/gstregex/imp.rs index 88e45b3d..e716af75 100644 --- a/text/regex/src/gstregex/imp.rs +++ b/text/regex/src/gstregex/imp.rs @@ -257,7 +257,7 @@ impl ObjectImpl for RegEx { Operation::ReplaceAll(ref replacement) => { commands.push( gst::Structure::new( - &"replace-all", + "replace-all", &[("pattern", &command.pattern), ("replacement", &replacement)], ) .to_send_value(), diff --git a/utils/fallbackswitch/examples/gtk_fallbackswitch.rs b/utils/fallbackswitch/examples/gtk_fallbackswitch.rs index c8c0bc13..11723227 100644 --- a/utils/fallbackswitch/examples/gtk_fallbackswitch.rs +++ b/utils/fallbackswitch/examples/gtk_fallbackswitch.rs @@ -36,7 +36,7 @@ fn create_pipeline() -> (gst::Pipeline, gst::Pad, gst::Element, gtk::Widget) { let video_src = gst::parse_bin_from_description(MAIN_PIPELINE, true) .unwrap() .upcast(); - let fallback_video_src = gst::parse_bin_from_description(&FALLBACK_PIPELINE, true) + let fallback_video_src = gst::parse_bin_from_description(FALLBACK_PIPELINE, true) .unwrap() .upcast(); diff --git a/utils/fallbackswitch/src/fallbacksrc/custom_source/imp.rs b/utils/fallbackswitch/src/fallbacksrc/custom_source/imp.rs index e613f6bf..fbdad2de 100644 --- a/utils/fallbackswitch/src/fallbacksrc/custom_source/imp.rs +++ b/utils/fallbackswitch/src/fallbacksrc/custom_source/imp.rs @@ -162,7 +162,7 @@ impl BinImpl for CustomSource { // TODO: Drop stream collection message for now, we only create a simple custom // one here so that fallbacksrc can know about our streams. It is never // forwarded. - self.handle_source_no_more_pads(&bin); + self.handle_source_no_more_pads(bin); } _ => self.parent_handle_message(bin, msg), } @@ -198,14 +198,14 @@ impl CustomSource { // Handle all source pads that already exist for pad in source.src_pads() { - if let Err(msg) = self.handle_source_pad_added(&element, &pad) { + if let Err(msg) = self.handle_source_pad_added(element, &pad) { element.post_error_message(msg); return Err(gst::StateChangeError); } } if !has_sometimes_pads { - self.handle_source_no_more_pads(&element); + self.handle_source_no_more_pads(element); } else { gst_debug!(CAT, obj: element, "Found sometimes pads"); diff --git a/utils/fallbackswitch/src/fallbackswitch/imp.rs b/utils/fallbackswitch/src/fallbackswitch/imp.rs index 2dea6cde..8b059890 100644 --- a/utils/fallbackswitch/src/fallbackswitch/imp.rs +++ b/utils/fallbackswitch/src/fallbackswitch/imp.rs @@ -361,7 +361,7 @@ impl FallbackSwitch { // Drop all older buffers from the fallback sinkpad if let Some(backup_pad) = backup_pad { - self.drain_pad_to_time(state, &backup_pad, state.last_output_time)?; + self.drain_pad_to_time(state, backup_pad, state.last_output_time)?; } Ok(Some((buffer, active_caps, pad_change))) @@ -607,7 +607,7 @@ impl FallbackSwitch { "No primary buffer, but can't autoswitch - draining backup pad" ); if let Some(backup_pad) = &backup_pad { - if let Err(e) = self.drain_pad_to_time(&mut *state, &backup_pad, cur_running_time) { + if let Err(e) = self.drain_pad_to_time(&mut *state, backup_pad, cur_running_time) { return ( Err(e), state.check_health_changes( @@ -917,11 +917,9 @@ impl ElementImpl for FallbackSwitch { return None; } - let sinkpad = gst::PadBuilder::::from_template( - &templ, - Some("fallback_sink"), - ) - .build(); + let sinkpad = + gst::PadBuilder::::from_template(templ, Some("fallback_sink")) + .build(); *fallback_sinkpad = Some(sinkpad.clone()); drop(fallback_sinkpad); diff --git a/utils/fallbackswitch/tests/fallbackswitch.rs b/utils/fallbackswitch/tests/fallbackswitch.rs index b4039790..b606a6f8 100644 --- a/utils/fallbackswitch/tests/fallbackswitch.rs +++ b/utils/fallbackswitch/tests/fallbackswitch.rs @@ -399,7 +399,7 @@ fn setup_pipeline(with_live_fallback: Option) -> Pipeline { let queue = gst::ElementFactory::make("queue", None).unwrap(); pipeline - .add_many(&[src.upcast_ref(), &switch, &queue, &sink.upcast_ref()]) + .add_many(&[src.upcast_ref(), &switch, &queue, sink.upcast_ref()]) .unwrap(); src.link_pads(Some("src"), &switch, Some("sink")).unwrap(); switch.link_pads(Some("src"), &queue, Some("sink")).unwrap(); diff --git a/video/cdg/src/cdgdec/imp.rs b/video/cdg/src/cdgdec/imp.rs index 189446d2..1cad24b4 100644 --- a/video/cdg/src/cdgdec/imp.rs +++ b/video/cdg/src/cdgdec/imp.rs @@ -135,7 +135,7 @@ impl VideoDecoderImpl for CdgDec { })?; let data = map.as_slice(); - cdg::decode_subchannel_cmd(&data) + cdg::decode_subchannel_cmd(data) }; let cmd = match cmd { diff --git a/video/closedcaption/src/cea608overlay/imp.rs b/video/closedcaption/src/cea608overlay/imp.rs index c9ea5a22..bd817105 100644 --- a/video/closedcaption/src/cea608overlay/imp.rs +++ b/video/closedcaption/src/cea608overlay/imp.rs @@ -131,7 +131,7 @@ impl Cea608Overlay { context.set_base_dir(pango::Direction::Ltr); let layout = pango::Layout::new(&context); layout.set_alignment(pango::Alignment::Left); - let mut font_desc = pango::FontDescription::from_string(&"monospace"); + let mut font_desc = pango::FontDescription::from_string("monospace"); let mut font_size = 1; let mut left_alignment = 0; @@ -139,7 +139,7 @@ impl Cea608Overlay { font_desc.set_size(font_size * pango::SCALE); layout.set_font_description(Some(&font_desc)); layout.set_text( - &"12345678901234567890123456789012\n2\n3\n4\n5\n6\n7\n8\n9\n0\n1\n2\n3\n4\n5", + "12345678901234567890123456789012\n2\n3\n4\n5\n6\n7\n8\n9\n0\n1\n2\n3\n4\n5", ); let (_ink_rect, logical_rect) = layout.extents(); if logical_rect.width > video_info.width() as i32 * pango::SCALE @@ -222,7 +222,7 @@ impl Cea608Overlay { cr.set_source_rgba(0.0, 0.0, 0.0, 1.0); - pangocairo::functions::layout_path(&cr, &layout); + pangocairo::functions::layout_path(&cr, layout); cr.stroke().ok()?; cr.restore().ok()?; @@ -230,7 +230,7 @@ impl Cea608Overlay { cr.save().ok()?; cr.set_source_rgba(255.0, 255.0, 255.0, 1.0); - pangocairo::functions::show_layout(&cr, &layout); + pangocairo::functions::show_layout(&cr, layout); cr.restore().ok()?; drop(cr); @@ -496,7 +496,7 @@ impl Cea608Overlay { if let Some(composition) = &state.composition { let buffer = buffer.make_mut(); if state.attach { - gst_video::VideoOverlayCompositionMeta::add(buffer, &composition); + gst_video::VideoOverlayCompositionMeta::add(buffer, composition); } else { let mut frame = gst_video::VideoFrameRef::from_buffer_ref_writable( buffer, diff --git a/video/closedcaption/src/mcc_parse/imp.rs b/video/closedcaption/src/mcc_parse/imp.rs index 61671cad..8b55b5a9 100644 --- a/video/closedcaption/src/mcc_parse/imp.rs +++ b/video/closedcaption/src/mcc_parse/imp.rs @@ -262,9 +262,9 @@ impl State { framerate: gst::Fraction, ) { let buffer = buffer.get_mut().unwrap(); - gst_video::VideoTimeCodeMeta::add(buffer, &timecode); + gst_video::VideoTimeCodeMeta::add(buffer, timecode); - self.update_timestamp(element, &timecode); + self.update_timestamp(element, timecode); buffer.set_pts(self.last_position); @@ -336,7 +336,7 @@ impl State { self.need_segment = false; } - events.extend(self.pending_events.drain(..)); + events.append(&mut self.pending_events); events } } diff --git a/video/closedcaption/src/scc_parse/imp.rs b/video/closedcaption/src/scc_parse/imp.rs index 5a52c358..76ffe8b2 100644 --- a/video/closedcaption/src/scc_parse/imp.rs +++ b/video/closedcaption/src/scc_parse/imp.rs @@ -159,7 +159,7 @@ impl State { framerate: gst::Fraction, element: &super::SccParse, ) -> Result { - match parse_timecode(framerate, &tc) { + match parse_timecode(framerate, tc) { Ok(timecode) => Ok(timecode), Err(err) => { let last_timecode = @@ -222,7 +222,7 @@ impl State { element: &super::SccParse, ) { let buffer = buffer.get_mut().unwrap(); - gst_video::VideoTimeCodeMeta::add(buffer, &timecode); + gst_video::VideoTimeCodeMeta::add(buffer, timecode); self.update_timestamp(timecode, element); @@ -284,7 +284,7 @@ impl State { self.need_segment = false; } - events.extend(self.pending_events.drain(..)); + events.append(&mut self.pending_events); events } } diff --git a/video/closedcaption/src/transcriberbin/imp.rs b/video/closedcaption/src/transcriberbin/imp.rs index 7d140583..66f88f3e 100644 --- a/video/closedcaption/src/transcriberbin/imp.rs +++ b/video/closedcaption/src/transcriberbin/imp.rs @@ -725,7 +725,7 @@ impl ObjectImpl for TranscriberBin { } "transcriber" => { let state = self.state.lock().unwrap(); - if let Some(ref state) = state.as_ref() { + if let Some(state) = state.as_ref() { state.transcriber.to_value() } else { let ret: Option = None; diff --git a/video/dav1d/src/dav1ddec/imp.rs b/video/dav1d/src/dav1ddec/imp.rs index 2aad908b..2239e487 100644 --- a/video/dav1d/src/dav1ddec/imp.rs +++ b/video/dav1d/src/dav1ddec/imp.rs @@ -253,14 +253,14 @@ impl Dav1dDec { pic: &dav1d::Picture, format: gst_video::VideoFormat, ) -> Result { - self.handle_resolution_change(element, &pic, format)?; + self.handle_resolution_change(element, pic, format)?; let output_state = element .output_state() .expect("Output state not set. Shouldn't happen!"); let offset = pic.offset() as i32; if let Some(mut frame) = element.frame(offset) { - let output_buffer = self.decoded_picture_as_buffer(&pic, output_state)?; + let output_buffer = self.decoded_picture_as_buffer(pic, output_state)?; frame.set_output_buffer(output_buffer); element.finish_frame(frame)?; } else { diff --git a/video/flavors/src/flvdemux/imp.rs b/video/flavors/src/flvdemux/imp.rs index 20fdcb2d..53cdd925 100644 --- a/video/flavors/src/flvdemux/imp.rs +++ b/video/flavors/src/flvdemux/imp.rs @@ -661,7 +661,7 @@ impl FlvDemux { let full_stream_id = srcpad.create_stream_id(element, Some(name)); // FIXME group id srcpad.push_event(gst::event::StreamStart::new(&full_stream_id)); - srcpad.push_event(gst::event::Caps::new(&caps)); + srcpad.push_event(gst::event::Caps::new(caps)); // FIXME proper segment handling let segment = gst::FormattedSegment::::default(); @@ -940,7 +940,7 @@ impl StreamingState { // AAC special case if data_header.sound_format == flavors::SoundFormat::AAC - && self.handle_aac_audio_packet_header(element, &tag_header, adapter)? + && self.handle_aac_audio_packet_header(element, tag_header, adapter)? { return Ok(events); } diff --git a/video/gif/tests/gifenc.rs b/video/gif/tests/gifenc.rs index bae6e9f1..54d96a06 100644 --- a/video/gif/tests/gifenc.rs +++ b/video/gif/tests/gifenc.rs @@ -49,7 +49,7 @@ fn test_encode(video_info: &gst_video::VideoInfo) { buffer.set_pts(gst::ClockTime::from_seconds(pts)); } let mut vframe = - gst_video::VideoFrame::from_buffer_writable(buffer, &video_info).unwrap(); + gst_video::VideoFrame::from_buffer_writable(buffer, video_info).unwrap(); for v in vframe.plane_data_mut(0).unwrap() { *v = 128; } diff --git a/video/rav1e/tests/rav1enc.rs b/video/rav1e/tests/rav1enc.rs index bae68a1e..8cc1e5d6 100644 --- a/video/rav1e/tests/rav1enc.rs +++ b/video/rav1e/tests/rav1enc.rs @@ -128,7 +128,7 @@ fn test_encode(video_info: &gst_video::VideoInfo) { let buffer = { let buffer = gst::Buffer::with_size(video_info.size()).unwrap(); - let mut vframe = gst_video::VideoFrame::from_buffer_writable(buffer, &video_info).unwrap(); + let mut vframe = gst_video::VideoFrame::from_buffer_writable(buffer, video_info).unwrap(); for v in vframe.plane_data_mut(0).unwrap() { *v = 0; diff --git a/video/rspng/tests/pngenc.rs b/video/rspng/tests/pngenc.rs index 55c87e7a..35a0d6a8 100644 --- a/video/rspng/tests/pngenc.rs +++ b/video/rspng/tests/pngenc.rs @@ -73,7 +73,7 @@ fn test_png_encode(video_info: &gst_video::VideoInfo) { buffer.set_pts(gst::ClockTime::from_seconds(pts)); } let mut vframe = - gst_video::VideoFrame::from_buffer_writable(buffer, &video_info).unwrap(); + gst_video::VideoFrame::from_buffer_writable(buffer, video_info).unwrap(); for v in vframe.plane_data_mut(0).unwrap() { *v = 128; }