From: nobu.nokada@...
Date: 2001-12-17T00:19:12+09:00
Subject: [ruby-talk:28687] Re: NodeDump patch for ruby 1.7.2

At Sun, 16 Dec 2001 23:45:59 +0900,
Dave Thomas <Dave@PragmaticProgrammer.com> wrote:
> > the set_trace_func trick that NodeDump uses doesn't work well.
> > This patch solves that.
> 
> Thanks. I've uploaded 0.1.4 that includes your patch.

I think it's better that t_dump() returns false unless dumped.


--- nodeDump.c~	Sun Dec 16 23:40:16 2001
+++ nodeDump.c	Mon Dec 17 00:12:36 2001
@@ -687,10 +687,11 @@ static VALUE t_dump(void)
   }
   
-  if(n){
+  if (n) {
     p("NodeDump V" ND_VERSION "\n\n");
     doDump(n, 0);
     printf("\n");
-    return Qnil;
+    return Qtrue;
   }
+  return Qfalse;
 }
 
@@ -701,4 +702,5 @@ static VALUE t_trigger(VALUE event, VALU
     exit(1);
   }
+  return Qnil;
 }
 



Nobu Nakada