From c3749b6a6da86243ca16ed058216114e71c184f3 Mon Sep 17 00:00:00 2001 From: ktsj Date: Sat, 5 Nov 2011 07:37:47 +0000 Subject: * ext/pathname/lib/pathname.rb, ext/tk/lib/multi-tk.rb, ext/tk/sample/demos-en/widget, lib/benchmark.rb, lib/irb/cmd/fork.rb, lib/mkmf.rb, lib/net/ftp.rb, lib/net/smtp.rb, lib/open3.rb, lib/pstore.rb, lib/rexml/element.rb, lib/rexml/light/node.rb, lib/rinda/tuplespace.rb, lib/rss/maker/base.rb, lib/rss/maker/entry.rb, lib/scanf.rb, lib/set.rb, lib/shell.rb, lib/shell/command-processor.rb, lib/shell/process-controller.rb, lib/shell/system-command.rb, lib/uri/common.rb: remove unused block arguments to avoid creating Proc objects. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@33638 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 12 ++++++++++++ ext/pathname/lib/pathname.rb | 2 +- ext/tk/lib/multi-tk.rb | 2 +- ext/tk/sample/demos-en/widget | 2 +- lib/benchmark.rb | 2 +- lib/irb/cmd/fork.rb | 2 +- lib/mkmf.rb | 2 +- lib/net/ftp.rb | 8 ++++---- lib/net/smtp.rb | 2 +- lib/open3.rb | 8 ++++---- lib/pstore.rb | 2 +- lib/rexml/element.rb | 6 +++--- lib/rexml/light/node.rb | 2 +- lib/rinda/tuplespace.rb | 2 +- lib/rss/maker/base.rb | 8 ++++---- lib/rss/maker/entry.rb | 4 ++-- lib/scanf.rb | 2 +- lib/set.rb | 4 ++-- lib/shell.rb | 2 +- lib/shell/command-processor.rb | 4 ++-- lib/shell/process-controller.rb | 2 +- lib/shell/system-command.rb | 2 +- lib/uri/common.rb | 2 +- 23 files changed, 48 insertions(+), 36 deletions(-) diff --git a/ChangeLog b/ChangeLog index 8204ec5c56..8b7c4ef0a7 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,15 @@ +Sat Nov 5 16:27:52 2011 Kazuki Tsujimoto + + * ext/pathname/lib/pathname.rb, ext/tk/lib/multi-tk.rb, + ext/tk/sample/demos-en/widget, lib/benchmark.rb, lib/irb/cmd/fork.rb, + lib/mkmf.rb, lib/net/ftp.rb, lib/net/smtp.rb, lib/open3.rb, + lib/pstore.rb, lib/rexml/element.rb, lib/rexml/light/node.rb, + lib/rinda/tuplespace.rb, lib/rss/maker/base.rb, + lib/rss/maker/entry.rb, lib/scanf.rb, lib/set.rb, lib/shell.rb, + lib/shell/command-processor.rb, lib/shell/process-controller.rb, + lib/shell/system-command.rb, lib/uri/common.rb: remove unused block + arguments to avoid creating Proc objects. + Sat Nov 5 15:45:04 2011 Tanaka Akira * ext/socket/init.c (rsock_socket0): extract single socket() call with diff --git a/ext/pathname/lib/pathname.rb b/ext/pathname/lib/pathname.rb index 1d2b37c66b..90485a2783 100644 --- a/ext/pathname/lib/pathname.rb +++ b/ext/pathname/lib/pathname.rb @@ -495,7 +495,7 @@ class Pathname # * Find * # If +self+ is ., yielded pathnames begin with a filename in the # current directory, not ./. # - def find(&block) # :yield: pathname + def find # :yield: pathname require 'find' if @path == '.' Find.find(@path) {|f| yield self.class.new(f.sub(%r{\A\./}, '')) } diff --git a/ext/tk/lib/multi-tk.rb b/ext/tk/lib/multi-tk.rb index b125c6e18e..68bd849670 100644 --- a/ext/tk/lib/multi-tk.rb +++ b/ext/tk/lib/multi-tk.rb @@ -2103,7 +2103,7 @@ end # evaluate a procedure on the proper interpreter class MultiTkIp # instance & class method - def _proc_on_safelevel(cmd=nil, &blk) # require a block for eval + def _proc_on_safelevel(cmd=nil) # require a block for eval if cmd if cmd.kind_of?(Method) _proc_on_safelevel{|*args| cmd.call(*args)} diff --git a/ext/tk/sample/demos-en/widget b/ext/tk/sample/demos-en/widget index e6510c7e45..21dd41685f 100644 --- a/ext/tk/sample/demos-en/widget +++ b/ext/tk/sample/demos-en/widget @@ -627,7 +627,7 @@ class Object end class Proc - def initialize(*args, &b) + def initialize(*args) super @__pseudo_toplevel__ = Thread.current[:TOPLEVEL] end diff --git a/lib/benchmark.rb b/lib/benchmark.rb index 6a00da1359..7fd3d0643e 100644 --- a/lib/benchmark.rb +++ b/lib/benchmark.rb @@ -243,7 +243,7 @@ module Benchmark # #bmbm yields a Benchmark::Job object and returns an array of # Benchmark::Tms objects. # - def bmbm(width = 0, &blk) # :yield: job + def bmbm(width = 0) # :yield: job job = Job.new(width) yield(job) width = job.width + 1 diff --git a/lib/irb/cmd/fork.rb b/lib/irb/cmd/fork.rb index c2664626ae..a3531d4299 100644 --- a/lib/irb/cmd/fork.rb +++ b/lib/irb/cmd/fork.rb @@ -15,7 +15,7 @@ module IRB module ExtendCommand class Forkgnuplot_commands, :binmode=>true) # - def capture2(*cmd, &block) + def capture2(*cmd) if Hash === cmd.last opts = cmd.pop.dup else @@ -359,7 +359,7 @@ module Open3 # # capture make log # make_log, s = Open3.capture2e("make") # - def capture2e(*cmd, &block) + def capture2e(*cmd) if Hash === cmd.last opts = cmd.pop.dup else @@ -662,7 +662,7 @@ module Open3 end module_function :pipeline - def pipeline_run(cmds, pipeline_opts, child_io, parent_io, &block) # :nodoc: + def pipeline_run(cmds, pipeline_opts, child_io, parent_io) # :nodoc: if cmds.empty? raise ArgumentError, "no commands" end diff --git a/lib/pstore.rb b/lib/pstore.rb index 429d7e5c7d..952445fc93 100644 --- a/lib/pstore.rb +++ b/lib/pstore.rb @@ -310,7 +310,7 @@ class PStore # # Note that PStore does not support nested transactions. # - def transaction(read_only = false, &block) # :yields: pstore + def transaction(read_only = false) # :yields: pstore value = nil raise PStore::Error, "nested transaction" if !@thread_safe && @lock.locked? @lock.synchronize do diff --git a/lib/rexml/element.rb b/lib/rexml/element.rb index 5991859a78..83e5f6e623 100644 --- a/lib/rexml/element.rb +++ b/lib/rexml/element.rb @@ -901,11 +901,11 @@ module REXML # #-> Yields , , , , , # XPath.each(doc.root, 'child::node()', &block) # #-> Yields , , , sean, , , - def each( xpath=nil, &block) + def each( xpath=nil ) XPath::each( @element, xpath ) {|e| yield e if e.kind_of? Element } end - def collect( xpath=nil, &block ) + def collect( xpath=nil ) collection = [] XPath::each( @element, xpath ) {|e| collection << yield(e) if e.kind_of?(Element) @@ -913,7 +913,7 @@ module REXML collection end - def inject( xpath=nil, initial=nil, &block ) + def inject( xpath=nil, initial=nil ) first = true XPath::each( @element, xpath ) {|e| if (e.kind_of? Element) diff --git a/lib/rexml/light/node.rb b/lib/rexml/light/node.rb index 0a896c83dc..b33f78f7ce 100644 --- a/lib/rexml/light/node.rb +++ b/lib/rexml/light/node.rb @@ -40,7 +40,7 @@ module REXML end end - def each( &block ) + def each size.times { |x| yield( at(x+4) ) } end diff --git a/lib/rinda/tuplespace.rb b/lib/rinda/tuplespace.rb index 113aa0de15..ba494aa4ec 100644 --- a/lib/rinda/tuplespace.rb +++ b/lib/rinda/tuplespace.rb @@ -305,7 +305,7 @@ module Rinda @bin.delete_at(idx) if idx end - def find(&blk) + def find @bin.reverse_each do |x| return x if yield(x) end diff --git a/lib/rss/maker/base.rb b/lib/rss/maker/base.rb index fa2b02d101..a776f47adc 100644 --- a/lib/rss/maker/base.rb +++ b/lib/rss/maker/base.rb @@ -353,7 +353,7 @@ module RSS module SetupDefaultDate private - def _set_default_values(&block) + def _set_default_values keep = { :date => date, :dc_dates => dc_dates.to_a.dup, @@ -365,7 +365,7 @@ module RSS dc_dates.unshift(dc_date) end self.date ||= self.dc_date - super(&block) + super ensure date = keep[:date] dc_dates.replace(keep[:dc_dates]) @@ -379,7 +379,7 @@ module RSS module SetupDefaultLanguage private - def _set_default_values(&block) + def _set_default_values keep = { :dc_languages => dc_languages.to_a.dup, } @@ -390,7 +390,7 @@ module RSS dc_language.value = _language.dup dc_languages.unshift(dc_language) end - super(&block) + super ensure dc_languages.replace(keep[:dc_languages]) end diff --git a/lib/rss/maker/entry.rb b/lib/rss/maker/entry.rb index b35936b208..f8f5469f2c 100644 --- a/lib/rss/maker/entry.rb +++ b/lib/rss/maker/entry.rb @@ -99,7 +99,7 @@ module RSS end end - def _set_default_values(&block) + def _set_default_values keep = { :authors => authors.to_a.dup, :contributors => contributors.to_a.dup, @@ -126,7 +126,7 @@ module RSS @maker.channel.title {|t| @title = t} end self.updated ||= @maker.channel.updated - super(&block) + super ensure authors.replace(keep[:authors]) contributors.replace(keep[:contributors]) diff --git a/lib/scanf.rb b/lib/scanf.rb index 6d67627f39..cd1596be8f 100644 --- a/lib/scanf.rb +++ b/lib/scanf.rb @@ -745,7 +745,7 @@ class String # See Scanf for details on creating a format string. # # You will need to require 'scanf' to use String#block_scanf - def block_scanf(fstr,&b) #:yield: current_match + def block_scanf(fstr) #:yield: current_match fs = Scanf::FormatString.new(fstr) str = self.dup final = [] diff --git a/lib/set.rb b/lib/set.rb index 43edc7557a..88303424ee 100755 --- a/lib/set.rb +++ b/lib/set.rb @@ -531,7 +531,7 @@ class SortedSet < Set require 'rbtree' module_eval %{ - def initialize(*args, &block) + def initialize(*args) @hash = RBTree.new super end @@ -544,7 +544,7 @@ class SortedSet < Set } rescue LoadError module_eval %{ - def initialize(*args, &block) + def initialize(*args) @keys = nil super end diff --git a/lib/shell.rb b/lib/shell.rb index 522f60a839..3b6ebcc8f1 100644 --- a/lib/shell.rb +++ b/lib/shell.rb @@ -286,7 +286,7 @@ class Shell end end - def self.notify(*opts, &block) + def self.notify(*opts) Shell::debug_output_synchronize do if opts[-1].kind_of?(String) yorn = verbose? diff --git a/lib/shell/command-processor.rb b/lib/shell/command-processor.rb index ae157eba5c..da7fecf1d5 100644 --- a/lib/shell/command-processor.rb +++ b/lib/shell/command-processor.rb @@ -335,7 +335,7 @@ class Shell end # %pwd, %cwd -> @pwd - def notify(*opts, &block) + def notify(*opts) Shell.notify(*opts) {|mes| yield mes if iterator? @@ -407,7 +407,7 @@ class Shell def self.alias_map @alias_map end - def self.alias_command(ali, command, *opts, &block) + def self.alias_command(ali, command, *opts) ali = ali.id2name if ali.kind_of?(Symbol) command = command.id2name if command.kind_of?(Symbol) begin diff --git a/lib/shell/process-controller.rb b/lib/shell/process-controller.rb index 30f2229d39..b973539b4b 100644 --- a/lib/shell/process-controller.rb +++ b/lib/shell/process-controller.rb @@ -234,7 +234,7 @@ class Shell end # simple fork - def sfork(command, &block) + def sfork(command) pipe_me_in, pipe_peer_out = IO.pipe pipe_peer_in, pipe_me_out = IO.pipe diff --git a/lib/shell/system-command.rb b/lib/shell/system-command.rb index 50d994bb20..1a8bb1a90f 100644 --- a/lib/shell/system-command.rb +++ b/lib/shell/system-command.rb @@ -145,7 +145,7 @@ class Shell # then # mes: "job(%id) close pipe-out." # yorn: Boolean(@shell.debug? or @shell.verbose?) - def notify(*opts, &block) + def notify(*opts) @shell.notify(*opts) do |mes| yield mes if iterator? diff --git a/lib/uri/common.rb b/lib/uri/common.rb index 38b0eb50d2..2b0dc2f1ab 100644 --- a/lib/uri/common.rb +++ b/lib/uri/common.rb @@ -258,7 +258,7 @@ module URI # # see also URI::Parser.make_regexp # - def extract(str, schemes = nil, &block) + def extract(str, schemes = nil) if block_given? str.scan(make_regexp(schemes)) { yield $& } nil -- cgit v1.2.3