WIP : Add logger and cleanup TUI module.

1 files changed, 25 insertions, 13 deletions

diff --git a/modules/TUI/module.jai b/modules/TUI/module.jai
index 5fd29f6..121aa42 100644
--- a/modules/TUI/module.jai
+++ b/modules/TUI/module.jai
@@ -2,6 +2,9 @@
 
 #scope_file
 
+// - fix/implement/finish `TODO` on `TUI\module` (use some sort of buffet to reduce io/print calls)
+// - fix/implement/finish `TODO` on `TUI\module` (use `dirty_bit_flag` to only update ehat has been changed)
+
 #if OS == {
     case .LINUX;
         #load "unix.jai";
@@ -525,16 +528,17 @@ read_input_line :: (count_limit: int, is_visible: bool = true) -> string, Key {
     return result, key;
 }
 
-start :: () {
+start :: () -> success := true #must {
     if active == true return;
     
-    // TODO Should start() call flush_input internally?
-
-    setup_key_map(); // TODO This is being called multiple times... please fix me!
-
     input_string.data   = input_buffer.data;
     input_string.count  = 0;
     input_override      = xx Keys.None;
+
+    previous_logger = context.logger;
+    context.logger = module_logger;
+
+    setup_key_map();
     
     write_strings(
         Commands.HideCursor,
@@ -542,26 +546,34 @@ start :: () {
         Commands.AlternateScreenBuffer,
         Commands.EncodingUTF8,
         Commands.CursorNormalMode,
-        Commands.KeypadNumMode);
-
-    previous_logger = context.logger;
-    context.logger = module_logger;
+        Commands.KeypadNumMode
+        );
     
-    OS_prepare_terminal();
+    if !OS_prepare_terminal() then return false;
     
     active = true;
+
+    return;
 }
 
-stop :: () {
+stop :: () -> success := true #must {
     if active == false return;
+    
     active = false;
 
     clear_style();
-    OS_reset_terminal();
+    
+    if !OS_reset_terminal() then return false;
     
     context.logger = previous_logger;
     
-    write_strings(Commands.MainScreenBuffer, Commands.RestoreCursorPosition, Commands.ShowCursor);
+    write_strings(
+        Commands.MainScreenBuffer,
+        Commands.RestoreCursorPosition,
+        Commands.ShowCursor
+        );
+    
+    return;
 }
 
 flush_input :: () {