repo: rxvt-unicode-sixel
action: commit
revision: 
path_from: 
revision_from: 7e804cd0f4edd5f2f60f39d8baf4f872b6d63756:
path_to: 
revision_to: 

commit 7e804cd0f4edd5f2f60f39d8baf4f872b6d63756
Author: Marc Lehmann 
Date:   Tue Jun 30 07:31:24 2020 +0000

    do not call perl_destruct

diff --git a/Changes b/Changes

--- a/Changes
+++ b/Changes
@@ -45,6 +45,8 @@ TODO: confirm-paste should be the default
 TODO: xterm-osc does not autoload background
 TODO: think about daemon delaying urxvt client exit.
 TODO: libev update
+	- do not destruct perl on exit anymore: this might fail for a variety of
+          reasons, and takes unneccessary time.
 	- the old bg image resources are now provided by the background
 	  extension, and perl is thus required for bg image support. No
 	  configuration change is needed: urxvt autoloads the background
diff --git a/src/rxvtperl.h b/src/rxvtperl.h

--- a/src/rxvtperl.h
+++ b/src/rxvtperl.h
@@ -51,7 +51,9 @@ struct rxvt_perl_interp
 {
   char **perl_environ;

+  #if 0 // see rxvtperl.xs
   ~rxvt_perl_interp ();
+  #endif

   void init ();
   void init (rxvt_term *term);
diff --git a/src/rxvtperl.xs b/src/rxvtperl.xs

--- a/src/rxvtperl.xs
+++ b/src/rxvtperl.xs
@@ -372,6 +372,9 @@ struct rxvt_perl_interp rxvt_perl;

 static PerlInterpreter *perl;

+#if 0 /* we are not a library anymore, so doing this is just not worth it */
+/*THINK/TODO: this has the side effect of, of course, not calling destructors. */
+/* but therse are not guaranteed anyway... */
 rxvt_perl_interp::~rxvt_perl_interp ()
 {
   if (perl)
@@ -381,6 +384,7 @@ rxvt_perl_interp::~rxvt_perl_interp ()
       PERL_SYS_TERM ();
     }
 }
+#endif

 void
 rxvt_perl_interp::init ()

-----END OF PAGE-----