Implement Display instead of ToString for Chart

yuankunzhang · Oct 6, 2024 · 772395d · 772395d
1 parent dd62e84
commit 772395d
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/charming/src/lib.rs b/charming/src/lib.rs
@@ -1,4 +1,3 @@
-#![allow(clippy::to_string_trait_impl)]
 #![cfg_attr(docsrs, feature(doc_cfg))]
 /*!
 Charming is a powerful and versatile chart rendering library for Rust that
@@ -516,9 +515,13 @@ impl Chart {
     }
 }
 
-impl ToString for Chart {
-    fn to_string(&self) -> String {
-        process_raw_strings(serde_json::to_string_pretty(self).unwrap().as_str())
+impl std::fmt::Display for Chart {
+    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
+        write!(
+            f,
+            "{}",
+            process_raw_strings(&serde_json::to_string_pretty(self).unwrap())
+        )
     }
 }