octave-bug-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Octave-bug-tracker] [bug #44420] Memory leaks in 4.0 RC


From: Robert Jenssen
Subject: [Octave-bug-tracker] [bug #44420] Memory leaks in 4.0 RC
Date: Sat, 2 Jul 2016 07:00:53 +0000 (UTC)
User-agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/51.0.2704.106 Safari/537.36

Follow-up Comment #8, bug #44420 (project octave):

I've looked around a bit more and I think I understand things a little better.
Sorry for the noise. I found the comment in toplev.cc about "Don't call
singleton_cleanup_list ..." and uncommented the cleanup call and
commented the typeinfo cleanup call:

diff -r f471f513ffd0 libinterp/corefcn/toplev.cc
--- a/libinterp/corefcn/toplev.cc       Wed Jun 29 18:50:37 2016 -0400
+++ b/libinterp/corefcn/toplev.cc       Sat Jul 02 16:17:44 2016 +1000
@@ -882,7 +882,7 @@
       // .oct files to be unregistered when the .oct file shared library
       // is unloaded.
       //
-      // OCTAVE_SAFE_CALL (singleton_cleanup_list::cleanup, ());
+      OCTAVE_SAFE_CALL (singleton_cleanup_list::cleanup, ());
 
       OCTAVE_SAFE_CALL (octave_chunk_buffer::clear, ());
     }
diff -r f471f513ffd0 libinterp/octave-value/ov-typeinfo.h
--- a/libinterp/octave-value/ov-typeinfo.h      Wed Jun 29 18:50:37 2016 -0400
+++ b/libinterp/octave-value/ov-typeinfo.h      Sat Jul 02 16:17:44 2016 +1000
@@ -218,7 +218,9 @@
 
   static octave_value_typeinfo *instance;
 
-  static void cleanup_instance (void) { delete instance; instance = 0; }
+  static void cleanup_instance (void) {
+    // delete instance; instance = 0;
+  }

Now "cdef_manager::cleanup_instance()" gets called.



    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?44420>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/




reply via email to

[Prev in Thread] Current Thread [Next in Thread]