Merge rust-bitcoin#4954: chore: remove redundant .to_string() call in embedded example

fe9179c chore: remove redundant .to_string() call in embedded example (Bugar)

Pull request description:

  Remove unnecessary .to_string() call on string literal in assert_eq! macro. The assert_eq! macro can compare String and &str directly through PartialEq<&str> for String, making the .to_string() call redundant and creating an unnecessary allocation


ACKs for top commit:
  apoelstra:
    ACK fe9179c; successfully ran local tests


Tree-SHA512: d098c389f7ec1cb5d1b09a5c9d24f09899a90a3469a77f07ceb24993770c1eedb518a9f5bcac0037860a7b7d31b9fa43d72f79d30c4e6516cbfe09f2674e8ab4

Author: apoelstra

bitcoin/embedded/src/main.rs

@@ -46,7 +46,7 @@ fn main() -> ! {
     let address = Address::p2wpkh(pubkey, Network::Bitcoin);
     hprintln!("Address: {}", address).unwrap();
 
-    assert_eq!(address.to_string(), "bc1qpx9t9pzzl4qsydmhyt6ctrxxjd4ep549np9993".to_string());
+    assert_eq!(address.to_string(), "bc1qpx9t9pzzl4qsydmhyt6ctrxxjd4ep549np9993");
     // exit QEMU
     // NOTE do not run this on hardware; it can corrupt OpenOCD state
     debug::exit(debug::EXIT_SUCCESS);