Use Bytes::new instead of &ByteBuf::from

- Optimize usage for byte slice instead of allocating new byte buffer

Author: bluk

src/value/index.rs

@@ -2,7 +2,7 @@
 
 use super::Value;
 use core::ops;
-use serde_bytes::ByteBuf;
+use serde_bytes::Bytes;
 
 #[cfg(all(feature = "alloc", not(feature = "std")))]
 use alloc::string::String;
@@ -35,14 +35,14 @@ impl Index for usize {
 impl Index for str {
     fn index<'a>(&self, v: &'a Value) -> Option<&'a Value> {
         match v {
-            Value::Dict(ref d) => d.get(&ByteBuf::from(self)),
+            Value::Dict(ref d) => d.get(Bytes::new(self.as_bytes())),
             _ => None,
         }
     }
 
     fn index_mut<'a>(&self, v: &'a mut Value) -> Option<&'a mut Value> {
         match v {
-            Value::Dict(ref mut d) => d.get_mut(&ByteBuf::from(self)),
+            Value::Dict(ref mut d) => d.get_mut(Bytes::new(self.as_bytes())),
             _ => None,
         }
     }

tests/de.rs

@@ -2,7 +2,6 @@
 extern crate serde_derive;
 
 use bt_bencode::{Error, Value};
-use serde_bytes::ByteBuf;
 
 #[derive(Debug, Deserialize, Clone, Eq, Hash, PartialEq)]
 struct TorrentFile {
@@ -24,13 +23,10 @@ fn test_deserialize_torrent_file_via_value() -> Result<(), Error> {
     let torrent_bytes = include_bytes!("ubuntu-18.04.3-live-server-amd64.iso.torrent");
     let decoded_value: Value = bt_bencode::from_slice(&torrent_bytes[..])?;
 
-    let announce = match decoded_value {
-        Value::Dict(dict) => match dict.get(&ByteBuf::from(String::from("announce"))) {
-            Some(Value::ByteStr(s)) => Some(s.clone().into_vec()),
-            _ => None,
-        },
-        _ => None,
-    };
+    let announce = decoded_value
+        .get("announce")
+        .and_then(|v| v.as_byte_str())
+        .map(|v| v.to_vec());
 
     assert_eq!(
         announce,