refactor: apply several optimizations

[null8626]

The following pull request applies the following changes for optimization purposes:

• Refactor crate::utils::emojis::get_emoji by making the second argument a &str not a String and making the function a one-liner
• .await several unused async calls
• Refactor several uses of format!() and including removing some redundant ones
• Prevent redundant clone()ing of Option<T> types by using as_ref() or as_deref() instead
• Prevent redundant clone()ing of types that implement Copy
• Collapse several nested if-statements
• Use unwrap_or_default() whenever necessary
• Remove several redundant uses of Arc<T>s
• And finally specifically for FilterPreset:
• • Use an array constant ALL instead of a method returns the same Vec<T> every time
• • Use a custom equalizer_vec![] macro for every equalizer bands declaration instead of creating a Vec and reconstructing it every time
• • Refactor several methods that matches against itself (using Self, marking some of them as const, etc.)
• • Adding a FromStr impl to make outside code that use it cleaner
• • Use the Filters { ..., ..Default::default() } instead of manually modifying a manually created Filters struct

[magmabits]

this is code

[null8626]

yes, this is indeed code!

[notigorwastaken]

Seeing these corrections made me realize I'm a terrible programmer lol

[notigorwastaken]

Thank you very much for your contribution, welcome to the team. 🥰

[null8626]

Seeing these corrections made me realize I'm a terrible programmer lol

No, you are not! You are learning, Igor! To be honest, I would even say your Rust code is pretty good! ❤️

[null8626]

Just be sure to take note of my suggestions as you learn along! 😉

[notigorwastaken]

Just be sure to take note of my suggestions as you learn along! 😉

Absolutely, thank you very much!