Implementation with a separate thread and ML changed to 1mb

Author: viuginick1

bin/rdebug-ide

@@ -37,8 +37,8 @@ EOB
   opts.separator ""
   opts.separator "Options:"
 
-  ENV['DEBUGGER_MEMORY_LIMIT'] = '10'
-  opts.on("-m", "--memory-limit LIMIT", Integer, "evaluation memory limit in mb (default: 10)") do |limit| 
+  ENV['DEBUGGER_MEMORY_LIMIT'] = '1'
+  opts.on("-m", "--memory-limit LIMIT", Integer, "evaluation memory limit in mb (default: 1)") do |limit| 
     ENV['DEBUGGER_MEMORY_LIMIT'] = limit
   end
 

lib/ruby-debug-ide/xml_printer.rb

@@ -385,28 +385,33 @@ def max_compact_name_size
     end
 
     def inspect_with_allocation_control(slice, memory_limit)
-      curr_thread = Thread.current
-      
-      start_alloc_size = ObjectSpace.memsize_of_all
-      
-      trace = TracePoint.new(:c_call, :call) do |tp|
-        curr_alloc_size = ObjectSpace.memsize_of_all
-        
-        if(curr_alloc_size - start_alloc_size > 1e6 * memory_limit)
-          
-          trace.disable
-          curr_thread.raise MemoryLimitError, "Out of memory: evaluation of inspect took more than #{memory_limit}mb." if curr_thread.alive?
-        end
+      if Thread.respond_to?(:critical) and Thread.critical
+        raise ThreadError, "memory_limit within critical session"      
       end
+      
+      curr_thread = Thread.current
 
-      trace.enable
-      result = slice.inspect
-      trace.disable
-      result 
+      result = nil
+      inspect_thread = DebugThread.start {
+        start_alloc_size = ObjectSpace.memsize_of_all
+        trace = TracePoint.new(:c_call, :call) do |tp|
+          curr_alloc_size = ObjectSpace.memsize_of_all
+          start_alloc_size = curr_alloc_size if (curr_alloc_size < start_alloc_size)
+
+          if(curr_alloc_size - start_alloc_size > 1e6 * memory_limit)
+            curr_thread.raise MemoryLimitError, "Out of memory: evaluation of inspect took more than #{memory_limit}mb. \n#{caller.map{|l| "\t#{l}"}.join("\n")}"
+            inspect_thread.kill
+          end
+        end.enable {
+          result = slice.inspect
+        }
+      }
+      inspect_thread.join
+      return result
     rescue MemoryLimitError => e
       print_debug(e.message)
       return nil
-    end 
+    end
 
     def compact_array_str(value)
       slice   = value[0..10]