feat: require renderers to return RenderResult instead of mutating one

BREAKING CHANGE: The `render` method on `Renderer`, `HelpRequest`, and
`ProgramCollect` now returns `RenderResult` instead of taking
`&mut RenderResult`. The `r_print!` and `r_println!` macros have been
removed in favor of using `std::io::Write` directly on `RenderResult`.

1 files changed, 19 insertions, 8 deletions

diff --git a/mling/src/proj_mgr/generator.rs b/mling/src/proj_mgr/generator.rs
index 48a1753..e560ae0 100644
--- a/mling/src/proj_mgr/generator.rs
+++ b/mling/src/proj_mgr/generator.rs
@@ -1,11 +1,12 @@
 use std::path::{self, PathBuf};
 
 use mingling::{
-    Groupped,
-    macros::{chain, pack, r_println, renderer, route},
+    macros::{chain, pack, renderer, route},
+    Groupped, RenderResult,
 };
+use std::io::Write as _;
 
-use crate::{Next, proj_mgr::EntryGenerateProject, res::ResCurrentDir};
+use crate::{proj_mgr::EntryGenerateProject, res::ResCurrentDir, Next};
 
 pack!(StateGenerateProjectReady = PathBuf);
 pack!(ResultGenerateProjectChecklistCreated = PathBuf);
@@ -35,12 +36,22 @@ pub fn handle_state_gen_proj_ready(prev: StateGenerateProjectReady) -> Next {
 }
 
 #[renderer]
-pub fn render_gen_proj_checklist_created(result: ResultGenerateProjectChecklistCreated) {
-    r_println!(
+pub fn render_gen_proj_checklist_created(
+    result: ResultGenerateProjectChecklistCreated,
+) -> RenderResult {
+    let mut res = RenderResult::default();
+    writeln!(
+        res,
         "Successfully create {} at \"{}\"",
         CHECK_LIST_NAME,
         result.to_string_lossy()
-    );
-    r_println!("");
-    r_println!("Please fill in {CHECK_LIST_NAME} and run `mling gen` again to continue generating");
+    )
+    .ok();
+    writeln!(res).ok();
+    writeln!(
+        res,
+        "Please fill in {CHECK_LIST_NAME} and run `mling gen` again to continue generating"
+    )
+    .ok();
+    res
 }