feat: add to_string and log that doesn't require a passed env (#180)

* feat: add to_string and log that doesn't require a passed `env`

* fix: add usages to test if it compiles

* fix: fmt

* fix: clippy

Author: willemneal

crates/loam-sdk/src/lib.rs

@@ -19,3 +19,14 @@ macro_rules! map {
         soroban_sdk::map![soroban_sdk::env(), $($tokens)*]
     };
 }
+
+#[cfg(feature = "loam-soroban-sdk")]
+#[macro_export]
+macro_rules! log {
+    ($($tokens:tt)*) => {
+        {
+        use soroban_sdk::Env;
+        soroban_sdk::log![soroban_sdk::env(), $($tokens)*];
+        }
+    };
+}

crates/loam-soroban-sdk/src/lib.rs

@@ -1,6 +1,8 @@
 #![no_std]
 
+use soroban_sdk::unwrap::UnwrapOptimized;
 pub use soroban_sdk::*;
+
 pub mod into_key;
 pub mod loam_storage;
 
@@ -20,6 +22,12 @@ pub fn set_env(env: Env) {
     unsafe { ENV = Some(env) };
 }
 
+/// Utility to cast a `&str` to a `String`.
+#[must_use]
+pub fn to_string(s: &str) -> String {
+    soroban_sdk::String::from_str(env(), s)
+}
+
 /// Returns a reference to the current environment.
 ///
 /// # Panics
@@ -29,8 +37,9 @@ pub fn set_env(env: Env) {
 /// function is called in normal operation.
 #[must_use]
 #[allow(static_mut_refs)]
+#[inline]
 pub fn env() -> &'static Env {
-    unsafe { ENV.as_ref().unwrap() }
+    unsafe { ENV.as_ref().unwrap_optimized() }
 }
 
 impl<T> Lazy for T

examples/soroban/status_message/src/status_message.rs

@@ -1,8 +1,8 @@
 #![allow(deprecated)]
-
 // Currently need to import `self` because `contracttype` expects it in the namespace
 use loam_sdk::{
-    soroban_sdk::{self, contracttype, env, Address, IntoKey, Lazy, Map, String},
+    log,
+    soroban_sdk::{self, contracttype, env, to_string, Address, IntoKey, Lazy, Map, String},
     subcontract,
 };
 
@@ -19,10 +19,8 @@ impl Default for StatusMessage {
 #[subcontract]
 pub trait IsPostable {
     /// Documentation ends up in the contract's metadata and thus the CLI, etc
-    fn messages_get(
-        &self,
-        author: loam_sdk::soroban_sdk::Address,
-    ) -> Option<loam_sdk::soroban_sdk::String>;
+    fn messages_get(&self, author: loam_sdk::soroban_sdk::Address)
+        -> loam_sdk::soroban_sdk::String;
 
     /// Only the author can set the message
     fn messages_set(
@@ -33,12 +31,15 @@ pub trait IsPostable {
 }
 
 impl IsPostable for StatusMessage {
-    fn messages_get(&self, author: Address) -> Option<String> {
-        self.0.get(author)
+    fn messages_get(&self, author: Address) -> String {
+        self.0
+            .get(author)
+            .unwrap_or_else(|| to_string("No message"))
     }
 
     fn messages_set(&mut self, author: Address, text: String) {
         author.require_auth();
+        log!("Setting message {} for {}", text, author);
         self.0.set(author, text);
     }
 }