LibLine: Ignore interrupts unless actively editing

It does not make much sense to receive an interrupt and process it
*much later*.
Also patches Userland/js to only create exceptions while some code is
actually running.

2 files changed, 12 insertions, 1 deletions

diff --git a/Libraries/LibLine/Editor.cpp b/Libraries/LibLine/Editor.cpp
index f355c09417..28b236716d 100644
--- a/Libraries/LibLine/Editor.cpp
+++ b/Libraries/LibLine/Editor.cpp
@@ -124,6 +124,8 @@ void Editor::stylize(const Span& span, const Style& style)
 
 String Editor::get_line(const String& prompt)
 {
+    m_is_editing = true;
+
     set_prompt(prompt);
     reset();
     set_origin();
@@ -139,6 +141,7 @@ String Editor::get_line(const String& prompt)
             fflush(stdout);
             auto string = String::copy(m_buffer);
             m_buffer.clear();
+            m_is_editing = false;
             return string;
         }
         char keybuf[16];
diff --git a/Libraries/LibLine/Editor.h b/Libraries/LibLine/Editor.h
index e7d6271625..4569e95c79 100644
--- a/Libraries/LibLine/Editor.h
+++ b/Libraries/LibLine/Editor.h
@@ -107,7 +107,11 @@ public:
 
     // FIXME: we will have to kindly ask our instantiators to set our signal handlers
     // since we can not do this cleanly ourselves (signal() limitation: cannot give member functions)
-    void interrupted() { m_was_interrupted = true; }
+    void interrupted()
+    {
+        if (m_is_editing)
+            m_was_interrupted = true;
+    }
     void resized() { m_was_resized = true; }
 
     size_t cursor() const { return m_cursor; }
@@ -148,6 +152,8 @@ public:
         m_finish = true;
     }
 
+    bool is_editing() const { return m_is_editing; }
+
 private:
     void vt_save_cursor();
     void vt_restore_cursor();
@@ -291,6 +297,8 @@ private:
 
     bool m_initialized { false };
     bool m_refresh_needed { false };
+
+    bool m_is_editing { false };
 };
 
 }