Switch log signature to str from bytes

HELP.md

@@ -182,7 +182,7 @@ There is an macro decorator `#[private]` that checks that the current account ID
 impl Contract {
     #[private]
     pub fn resolve_transfer(&mut self) {
-        env::log(b"This is a callback");
+        env::log("This is a callback");
     }
 }
 ```
@@ -196,7 +196,7 @@ impl Contract {
         if env::current_account_id() != env::predecessor_account_id() {
             near_sdk::env::panic(b"Method resolve_transfer is private");
         }
-        env::log(b"This is a callback");
+        env::log("This is a callback");
     }
 }
 ```
@@ -348,11 +348,11 @@ to receive attached deposits. Otherwise, if a deposit is attached to a non-payab
 impl Contract {
     #[payable]
     pub fn take_my_money(&mut self) {
-        env::log(b"Thanks!");
+        env::log("Thanks!");
     }
 
     pub fn do_not_take_my_money(&mut self) {
-        env::log(b"Thanks!");
+        env::log("Thanks!");
     }
 }
 ```
@@ -363,14 +363,14 @@ This is equivalent to:
 #[near_bindgen]
 impl Contract {
     pub fn take_my_money(&mut self) {
-        env::log(b"Thanks!");
+        env::log("Thanks!");
     }
 
     pub fn do_not_take_my_money(&mut self) {
         if near_sdk::env::attached_deposit() != 0 {
             near_sdk::env::panic(b"Method do_not_take_my_money doesn't accept deposit");
         }
-        env::log(b"Thanks!");
+        env::log("Thanks!");
     }
 }
 ```
@@ -412,7 +412,7 @@ log!("Transferred {} tokens from {} to {}", amount, sender_id, receiver_id);
 It's equivalent to the following message:
 
 ```rust
-env::log(format!("Transferred {} tokens from {} to {}", amount, sender_id, receiver_id).as_bytes());
+env::log(format!("Transferred {} tokens from {} to {}", amount, sender_id, receiver_id));
 ```
 
 ## Return `Promise`

examples/cross-contract-low-level/src/lib.rs

@@ -147,7 +147,7 @@ impl CrossContract {
     pub fn check_promise(&mut self) {
         match env::promise_result(0) {
             PromiseResult::Successful(_) => {
-                env::log(b"Check_promise successful");
+                env::log("Check_promise successful");
                 self.checked_promise = true;
             }
             _ => panic!("Promise with index 0 failed"),

near-sdk/src/environment/env.rs

@@ -691,15 +691,13 @@ pub fn panic(message: &[u8]) -> ! {
     unreachable!()
 }
 /// Log the UTF-8 encodable message.
-pub fn log(message: &[u8]) {
-    unsafe {
-        BLOCKCHAIN_INTERFACE.with(|b| {
-            b.borrow()
-                .as_ref()
-                .expect(BLOCKCHAIN_INTERFACE_NOT_SET_ERR)
-                .log_utf8(message.len() as _, message.as_ptr() as _)
-        })
-    }
+pub fn log(message: &str) {
+    BLOCKCHAIN_INTERFACE.with(|b| unsafe {
+        b.borrow()
+            .as_ref()
+            .expect(BLOCKCHAIN_INTERFACE_NOT_SET_ERR)
+            .log_utf8(message.len() as _, message.as_ptr() as _)
+    })
 }
 
 // ###############

near-sdk/src/utils/mod.rs

@@ -4,11 +4,11 @@ use crate::{env, AccountId, PromiseResult};
 
 #[macro_export]
 macro_rules! log {
-    ($arg:tt) => {
-        $crate::env::log($arg.as_bytes())
+    ($arg:expr) => {
+        $crate::env::log($arg.as_ref())
     };
     ($($arg:tt)*) => {
-        $crate::env::log(format!($($arg)*).as_bytes())
+        $crate::env::log(format!($($arg)*).as_str())
     };
 }