archive/ruby
1
0
Fork 0
mirror of https://github.com/ruby/ruby.git synced 2025-09-20 11:03:58 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 4

merge revision(s) 34202:

Browse source 
* lib/rexml/parsers/baseparser.rb: rexml BaseParser uses
	  instance_eval unnecessarily on listener add.
	  patch from Charles Nutter. [Bug #5696] [ruby-core:41437]


git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_1_9_3@34223 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
This commit is contained in:
ayumin 2012-01-07 13:40:21 +00:00
parent e87360009c
commit 68fff3881d
3 changed files with 18 additions and 16 deletions
Download patch file Download diff file Expand all files Collapse all files

22
lib/rexml/parsers/baseparser.rb
View file

@ -114,22 +114,10 @@ module REXML
def initialize( source )
self.stream = source
@listeners = []
end
def add_listener( listener )
if !defined?(@listeners) or !@listeners
@listeners = []
instance_eval <<-EOL
alias :_old_pull :pull
def pull
event = _old_pull
@listeners.each do |listener|
listener.receive event
end
event
end
EOL
end
@listeners << listener
end
@ -192,6 +180,14 @@ module REXML
# Returns the next event. This is a +PullEvent+ object.
def pull
_pull_inner.tap do |event|
@listeners.each do |listener|
listener.receive event
end
end
end
def _pull_inner
if @closed
x, @closed = @closed, nil
return [ :end_element, x ]