From 8704fa1b864afaca181bfc1c71775f914c7665a5 Mon Sep 17 00:00:00 2001
From: Matthias Koefferlein <matthias@klayout.de>
Date: Sat, 13 May 2023 16:04:42 +0200
Subject: [PATCH] Solved issue #1219: 'close' button works now in macro IDE,
 printing name of macro which is run to avoid confusion between 'run current'
 and 'run this'

---
 src/lay/lay/layMacroEditorDialog.cc | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/src/lay/lay/layMacroEditorDialog.cc b/src/lay/lay/layMacroEditorDialog.cc
index 3266b7c60..e75c60dfa 100644
--- a/src/lay/lay/layMacroEditorDialog.cc
+++ b/src/lay/lay/layMacroEditorDialog.cc
@@ -399,6 +399,7 @@ MacroEditorDialog::MacroEditorDialog (lay::Dispatcher *pr, lym::MacroCollection
   replaceText->setPlaceholderText (tr ("Replace text ..."));
 #endif
 
+  connect (closeButton, SIGNAL (clicked ()), this, SLOT (close_button_clicked ()));
   connect (forwardButton, SIGNAL (clicked ()), this, SLOT (forward ()));
   connect (backwardButton, SIGNAL (clicked ()), this, SLOT (backward ()));
 
@@ -2362,7 +2363,7 @@ END_PROTECTED
 void
 MacroEditorDialog::tab_close_requested (int index)
 {
-  if (m_in_exec) {
+  if (m_in_exec || index < 0) {
     return;
   }
 
@@ -3736,8 +3737,12 @@ MacroEditorDialog::run (int stop_stack_depth, lym::Macro *macro)
     set_run_macro (macro);
 
     try {
+
+      write_str (tl::sprintf (tl::to_string (tr ("Running macro %s\n")), macro->path ()).c_str (), OS_echo);
+
       macro->run ();
       m_stop_stack_depth = -1;
+
     } catch (tl::ExitException &) {
       m_stop_stack_depth = -1;
       //  .. ignore exit exceptions ..