chore: minor clippy fixes (#3576)

Description
---
- remove unnecessary vec copy in protobuf conversion 
- clippy fixes
- remove duplicate utility function

Motivation and Context
---
Fix some minor clippy warnings 

How Has This Been Tested?
---
Existing tests pass

applications/tari_app_grpc/src/conversions/aggregate_body.rs

@@ -21,7 +21,7 @@
 // USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
 use std::convert::TryFrom;
-use tari_core::{proto::utils::try_convert_all, transactions::aggregated_body::AggregateBody};
+use tari_core::{tari_utilities::convert::try_convert_all, transactions::aggregated_body::AggregateBody};
 
 use crate::tari_rpc as grpc;
 

base_layer/core/src/proto/mod.rs

@@ -51,4 +51,4 @@ mod block;
 #[cfg(any(feature = "base_node", feature = "base_node_proto"))]
 mod block_header;
 #[cfg(any(feature = "base_node", feature = "base_node_proto"))]
-pub mod utils;
+mod utils;

base_layer/core/src/proto/transaction.rs

@@ -162,7 +162,7 @@ impl TryFrom<proto::types::TransactionOutput> for TransactionOutput {
         let sender_offset_public_key =
             PublicKey::from_bytes(output.sender_offset_public_key.as_bytes()).map_err(|err| format!("{:?}", err))?;
 
-        let script = TariScript::from_bytes(&output.script.to_vec()).map_err(|err| err.to_string())?;
+        let script = TariScript::from_bytes(&output.script).map_err(|err| err.to_string())?;
 
         let metadata_signature = output
             .metadata_signature

base_layer/core/src/proto/utils.rs

@@ -21,23 +21,8 @@
 // USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
 use prost_types::Timestamp;
-use std::convert::TryInto;
 use tari_crypto::tari_utilities::epoch_time::EpochTime;
 
-/// Tries to convert a series of `T`s to `U`s, returning an error at the first failure
-pub fn try_convert_all<T, U, I>(into_iter: I) -> Result<Vec<U>, T::Error>
-where
-    I: IntoIterator<Item = T>,
-    T: TryInto<U>,
-{
-    let iter = into_iter.into_iter();
-    let mut result = Vec::with_capacity(iter.size_hint().0);
-    for item in iter {
-        result.push(item.try_into()?);
-    }
-    Ok(result)
-}
-
 /// Utility function that converts a `prost::Timestamp` to a `chrono::DateTime`
 pub(crate) fn timestamp_to_datetime(timestamp: Timestamp) -> EpochTime {
     (timestamp.seconds as u64).into()