1
0
Fork 0

Remove now unnecessary explicit dyn casts

Rust-analyzer used to not infer this, seems like now it does.
This commit is contained in:
Robbert van der Helm 2022-05-24 23:55:48 +02:00
parent ed9ee2c188
commit 535ae1260e
8 changed files with 8 additions and 16 deletions

View file

@ -298,8 +298,7 @@ impl Plugin for Crisp {
const SAMPLE_ACCURATE_AUTOMATION: bool = true; const SAMPLE_ACCURATE_AUTOMATION: bool = true;
fn params(&self) -> Arc<dyn Params> { fn params(&self) -> Arc<dyn Params> {
// The explicit cast is not needed, but Rust Analyzer gets very upset when you don't do it self.params.clone()
self.params.clone() as Arc<dyn Params>
} }
fn editor(&self) -> Option<Box<dyn Editor>> { fn editor(&self) -> Option<Box<dyn Editor>> {

View file

@ -237,8 +237,7 @@ impl Plugin for Diopser {
const SAMPLE_ACCURATE_AUTOMATION: bool = true; const SAMPLE_ACCURATE_AUTOMATION: bool = true;
fn params(&self) -> Arc<dyn Params> { fn params(&self) -> Arc<dyn Params> {
// The explicit cast is not needed, but Rust Analyzer gets very upset when you don't do it self.params.clone()
self.params.clone() as Arc<dyn Params>
} }
fn editor(&self) -> Option<Box<dyn Editor>> { fn editor(&self) -> Option<Box<dyn Editor>> {

View file

@ -115,8 +115,7 @@ impl Plugin for Gain {
const SAMPLE_ACCURATE_AUTOMATION: bool = true; const SAMPLE_ACCURATE_AUTOMATION: bool = true;
fn params(&self) -> Arc<dyn Params> { fn params(&self) -> Arc<dyn Params> {
// The explicit cast is not needed, but Rust Analyzer gets very upset when you don't do it self.params.clone()
self.params.clone() as Arc<dyn Params>
} }
fn accepts_bus_config(&self, config: &BusConfig) -> bool { fn accepts_bus_config(&self, config: &BusConfig) -> bool {

View file

@ -34,8 +34,7 @@ impl Plugin for MidiInverter {
const SAMPLE_ACCURATE_AUTOMATION: bool = true; const SAMPLE_ACCURATE_AUTOMATION: bool = true;
fn params(&self) -> Arc<dyn Params> { fn params(&self) -> Arc<dyn Params> {
// The explicit cast is not needed, but Rust Analyzer gets very upset when you don't do it self.params.clone()
self.params.clone() as Arc<dyn Params>
} }
fn process( fn process(

View file

@ -112,8 +112,7 @@ impl Plugin for Sine {
const SAMPLE_ACCURATE_AUTOMATION: bool = true; const SAMPLE_ACCURATE_AUTOMATION: bool = true;
fn params(&self) -> Arc<dyn Params> { fn params(&self) -> Arc<dyn Params> {
// The explicit cast is not needed, but Rust Analyzer gets very upset when you don't do it self.params.clone()
self.params.clone() as Arc<dyn Params>
} }
fn accepts_bus_config(&self, config: &BusConfig) -> bool { fn accepts_bus_config(&self, config: &BusConfig) -> bool {

View file

@ -96,8 +96,7 @@ impl Plugin for Stft {
const SAMPLE_ACCURATE_AUTOMATION: bool = true; const SAMPLE_ACCURATE_AUTOMATION: bool = true;
fn params(&self) -> Arc<dyn Params> { fn params(&self) -> Arc<dyn Params> {
// The explicit cast is not needed, but Rust Analyzer gets very upset when you don't do it self.params.clone()
self.params.clone() as Arc<dyn Params>
} }
fn accepts_bus_config(&self, config: &BusConfig) -> bool { fn accepts_bus_config(&self, config: &BusConfig) -> bool {

View file

@ -124,8 +124,7 @@ impl Plugin for LoudnessWarWinner {
const DEFAULT_NUM_OUTPUTS: u32 = 2; const DEFAULT_NUM_OUTPUTS: u32 = 2;
fn params(&self) -> Arc<dyn Params> { fn params(&self) -> Arc<dyn Params> {
// The explicit cast is not needed, but Rust Analyzer gets very upset when you don't do it self.params.clone()
self.params.clone() as Arc<dyn Params>
} }
fn accepts_bus_config(&self, config: &BusConfig) -> bool { fn accepts_bus_config(&self, config: &BusConfig) -> bool {

View file

@ -167,8 +167,7 @@ impl Plugin for PubertySimulator {
const DEFAULT_NUM_OUTPUTS: u32 = 2; const DEFAULT_NUM_OUTPUTS: u32 = 2;
fn params(&self) -> Arc<dyn Params> { fn params(&self) -> Arc<dyn Params> {
// The explicit cast is not needed, but Rust Analyzer gets very upset when you don't do it self.params.clone()
self.params.clone() as Arc<dyn Params>
} }
fn accepts_bus_config(&self, config: &BusConfig) -> bool { fn accepts_bus_config(&self, config: &BusConfig) -> bool {