From: Daniel Berger <Daniel.Berger@...>
Date: 2006-02-18T05:21:29+09:00
Subject: Re: Minor tracer.rb patch

Yukihiro Matsumoto wrote:
> Hi,
> 
> In message "Re: Minor tracer.rb patch"
>     on Thu, 16 Feb 2006 08:22:28 +0900, Daniel Berger <Daniel.Berger@qwest.com> writes:
> 
> |Here's a patch that gets rid of the unnecessary (and occasionally problematic) 
> |MY_FILE_NAME constant, which I'm guessing was used for quick testing by the author:
> 
> The small code at the bottom should not be removed, since it's for
> usage like:
> 
>   ruby lib/tracer.rb foobar.rb

Who would do this other than the author?  I don't think too many folks are 
going to dig into their libdir and run against tracer.rb directly.  But, whatever.

> But I will merge this patch except that part.  Thank you.
> 
> 							matz.

Ok, but can we replace "if caller(0).size == 1" with "if $0 == __FILE__" ?  At 
least, I'm assuming that's the behavior the author was after.

The latter is more idiomatic, probably more reliable, and causes fewer problems 
with 1.9 (I'm told).

Thanks,

Dan