Prefer block_given? to iterator?

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@66866 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

basictest/test.rb

@@ -984,13 +984,6 @@ def $x.default(k)
 
 test_check "iterator"
 
-test_ok(!iterator?)
-
-def ttt
-  test_ok(iterator?)
-end
-ttt{}
-
 # yield at top level
 test_ok(!defined?(yield))
 

lib/irb/cmd/fork.rb

@@ -21,7 +21,7 @@ def execute
           class << self
             alias_method :exit, ExtendCommand.irb_original_method_name('exit')
           end
-          if iterator?
+          if block_given?
             begin
               yield
             ensure

lib/shell.rb

@@ -263,7 +263,7 @@ def expand_path(path)
   def chdir(path = nil, verbose = @verbose)
     check_point
 
-    if iterator?
+    if block_given?
       notify("chdir(with block) #{path}") if verbose
       cwd_old = @cwd
       begin
@@ -297,7 +297,7 @@ def chdir(path = nil, verbose = @verbose)
   def pushdir(path = nil, verbose = @verbose)
     check_point
 
-    if iterator?
+    if block_given?
       notify("pushdir(with block) #{path}") if verbose
       pushdir(path, nil)
       begin
@@ -446,7 +446,7 @@ def self.notify(*opts)
       _head = true
       STDERR.print opts.collect{|mes|
         mes = mes.dup
-        yield mes if iterator?
+        yield mes if block_given?
         if _head
           _head = false
           prefix + mes

lib/shell/command-processor.rb

@@ -343,7 +343,7 @@ def concat(*jobs)
     # %pwd, %cwd -> @pwd
     def notify(*opts)
       Shell.notify(*opts) {|mes|
-        yield mes if iterator?
+        yield mes if block_given?
 
         mes.gsub!("%pwd", "#{@cwd}")
         mes.gsub!("%cwd", "#{@cwd}")
@@ -437,7 +437,7 @@ def self.alias_command(ali, command, *opts)
       ali = ali.id2name if ali.kind_of?(Symbol)
       command = command.id2name if command.kind_of?(Symbol)
       begin
-        if iterator?
+        if block_given?
           @alias_map[ali.intern] = proc
 
           eval((d = %Q[def #{ali}(*opts)

lib/shell/system-command.rb

@@ -147,7 +147,7 @@ def each(rs = nil)
     #    yorn: Boolean(@shell.debug? or @shell.verbose?)
     def notify(*opts)
       @shell.notify(*opts) do |mes|
-        yield mes if iterator?
+        yield mes if block_given?
 
         mes.gsub!("%id", "#{@command}:##{@pid}")
         mes.gsub!("%name", "#{@command}")

test/ruby/test_iterator.rb

@@ -12,17 +12,12 @@ def iter_test2
 end
 
 class TestIterator < Test::Unit::TestCase
-  def ttt
-    assert(iterator?)
-  end
-
-  def test_iterator
-    assert(!iterator?)
-
-    ttt{}
-
-    # yield at top level	!! here's not toplevel
-    assert(!defined?(yield))
+  def test_yield_at_toplevel
+    assert_separately([],"#{<<~"begin;"}\n#{<<~'end;'}")
+    begin;
+      assert(!block_given?)
+      assert(!defined?(yield))
+    end;
   end
 
   def test_array