diff options
author | kou <kou@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2012-02-02 12:48:45 +0000 |
---|---|---|
committer | kou <kou@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2012-02-02 12:48:45 +0000 |
commit | ca77bce66d8b32816fd9cf4c7ee228bb5e3dfa7f (patch) | |
tree | a3a4f58f60fbda0278bce2f85bc71bedc2b48656 /lib/rexml/parsers/baseparser.rb | |
parent | 92cecf79b428cfe1c8fc83a9b25ca5a99e867d83 (diff) | |
download | ruby-ca77bce66d8b32816fd9cf4c7ee228bb5e3dfa7f.tar.gz |
* lib/rexml/parsers/baseparser.rb: use meaningful name.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@34420 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'lib/rexml/parsers/baseparser.rb')
-rw-r--r-- | lib/rexml/parsers/baseparser.rb | 26 |
1 files changed, 13 insertions, 13 deletions
diff --git a/lib/rexml/parsers/baseparser.rb b/lib/rexml/parsers/baseparser.rb index 2df26b82f9..b5b61c34dd 100644 --- a/lib/rexml/parsers/baseparser.rb +++ b/lib/rexml/parsers/baseparser.rb @@ -373,31 +373,31 @@ module REXML if md[4].size > 0 attrs = md[4].scan( ATTRIBUTE_PATTERN ) raise REXML::ParseException.new( "error parsing attributes: [#{attrs.join ', '}], excess = \"#$'\"", @source) if $' and $'.strip.size > 0 - attrs.each { |a,b,c,d,e| - if b == "xmlns" - if c == "xml" - if e != "http://www.w3.org/XML/1998/namespace" + attrs.each do |name, prefix, local_part, quote, value| + if prefix == "xmlns" + if local_part == "xml" + if value != "http://www.w3.org/XML/1998/namespace" msg = "The 'xml' prefix must not be bound to any other namespace "+ "(http://www.w3.org/TR/REC-xml-names/#ns-decl)" raise REXML::ParseException.new( msg, @source, self ) end - elsif c == "xmlns" + elsif local_part == "xmlns" msg = "The 'xmlns' prefix must not be declared "+ "(http://www.w3.org/TR/REC-xml-names/#ns-decl)" raise REXML::ParseException.new( msg, @source, self) end - curr_ns << c - elsif b - prefixes << b unless b == "xml" + curr_ns << local_part + elsif prefix + prefixes << prefix unless prefix == "xml" end - if attributes.has_key? a - msg = "Duplicate attribute #{a.inspect}" - raise REXML::ParseException.new( msg, @source, self) + if attributes.has_key?(name) + msg = "Duplicate attribute #{name.inspect}" + raise REXML::ParseException.new(msg, @source, self) end - attributes[a] = e - } + attributes[name] = value + end end # Verify that all of the prefixes have been defined |