diff options
author | akr <akr@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2009-03-17 05:26:15 +0000 |
---|---|---|
committer | akr <akr@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2009-03-17 05:26:15 +0000 |
commit | a28532dcf3e237b8d75c724cd65eb04ca57e6da1 (patch) | |
tree | 8e8690ef4920c1981fc7749a2f146a9c4f4b4bc4 | |
parent | 27a39b8ffed0c8317f5d44e21dee52e66ee66e40 (diff) | |
download | ruby-a28532dcf3e237b8d75c724cd65eb04ca57e6da1.tar.gz |
* lib/pathname.rb (Pathname#sub): set $~ in block.binding.
[ruby-dev:38173]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@22987 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 5 | ||||
-rw-r--r-- | lib/pathname.rb | 16 | ||||
-rw-r--r-- | test/pathname/test_pathname.rb | 9 |
3 files changed, 29 insertions, 1 deletions
@@ -1,3 +1,8 @@ +Tue Mar 17 14:25:16 2009 Tanaka Akira <akr@fsij.org> + + * lib/pathname.rb (Pathname#sub): set $~ in block.binding. + [ruby-dev:38173] + Tue Mar 17 13:48:08 2009 Nobuyoshi Nakada <nobu@ruby-lang.org> * win32/Makefile.sub (config.h): added RUBY_COREDLL. diff --git a/lib/pathname.rb b/lib/pathname.rb index 8a07d455fd..75faaef8b2 100644 --- a/lib/pathname.rb +++ b/lib/pathname.rb @@ -254,7 +254,21 @@ class Pathname # Return a pathname which is substituted by String#sub. def sub(pattern, *rest, &block) - self.class.new(@path.sub(pattern, *rest, &block)) + if block + path = @path.sub(pattern, *rest) {|*args| + begin + old = Thread.current[:pathname_sub_matchdata] + Thread.current[:pathname_sub_matchdata] = $~ + eval("$~ = Thread.current[:pathname_sub_matchdata]", block.binding) + ensure + Thread.current[:pathname_sub_matchdata] = old + end + yield *args + } + else + path = @path.sub(pattern, *rest) + end + self.class.new(path) end if File::ALT_SEPARATOR diff --git a/test/pathname/test_pathname.rb b/test/pathname/test_pathname.rb index d979ff7023..a9f7149633 100644 --- a/test/pathname/test_pathname.rb +++ b/test/pathname/test_pathname.rb @@ -471,6 +471,15 @@ class TestPathname < Test::Unit::TestCase defassert(:pathsubext, 'fooaa.o', 'fooaa', '.o') defassert(:pathsubext, 'd.e/aa.o', 'd.e/aa', '.o') + def test_sub_matchdata + result = Pathname("abc.gif").sub(/\..*/) { + assert_not_nil($~) + assert_equal(".gif", $~[0]) + ".png" + } + assert_equal("abc.png", result.to_s) + end + def root?(path) Pathname.new(path).root? end |