Remove dead code

soroban-sdk/src/bytes.rs

@@ -791,9 +791,13 @@ impl IntoIterator for Bytes {
 pub struct BytesIter(Bytes);
 
 impl BytesIter {
-    fn into_bin(self) -> Bytes {
+    pub fn into_bytes(self) -> Bytes {
         self.0
     }
+
+    pub fn to_bytes(&self) -> Bytes {
+        self.0.clone()
+    }
 }
 
 impl Iterator for BytesIter {

soroban-sdk/src/events.rs

@@ -5,9 +5,6 @@ use core::fmt::Debug;
 use crate::{contracttype, Bytes, Map};
 use crate::{env::internal, unwrap::UnwrapInfallible, Env, IntoVal, Val, Vec};
 
-// TODO: consolidate with host::events::TOPIC_BYTES_LENGTH_LIMIT
-const TOPIC_BYTES_LENGTH_LIMIT: u32 = 32;
-
 /// Events publishes events for the currently executing contract.
 ///
 /// ```

soroban-sdk/src/lib.rs

@@ -51,7 +51,6 @@
 
 #![cfg_attr(target_family = "wasm", no_std)]
 #![cfg_attr(feature = "docs", feature(doc_cfg))]
-#![allow(dead_code)]
 // The SDK uses #[test] in doctests, and does some sneaky line hiding to have
 // the doctest execute the test inside a main function instead.
 #![allow(clippy::test_attr_in_doctest)]

soroban-sdk/src/map.rs

@@ -291,11 +291,6 @@ impl<K, V> Map<K, V> {
         self.obj.to_val()
     }
 
-    #[inline(always)]
-    pub(crate) fn as_object(&self) -> &MapObject {
-        &self.obj
-    }
-
     #[inline(always)]
     pub(crate) fn to_object(&self) -> MapObject {
         self.obj

soroban-sdk/src/vec.rs

@@ -953,7 +953,11 @@ impl<T> VecTryIter<T> {
         }
     }
 
-    fn into_vec(self) -> Vec<T> {
+    pub fn into_vec(self) -> Vec<T> {
+        self.to_vec()
+    }
+
+    pub fn to_vec(&self) -> Vec<T> {
         self.vec.slice(self.start..self.end)
     }
 }