ruby-changes:30405
From: kou <ko1@a...>
Date: Sat, 10 Aug 2013 12:02:01 +0900 (JST)
Subject: [ruby-changes:30405] kou:r42484 (trunk): * lib/rexml/parsers/baseparser.rb (REXML::Parsers::BaseParser::SYSTEM):
kou 2013-08-10 12:01:56 +0900 (Sat, 10 Aug 2013) New Revision: 42484 http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=42484 Log: * lib/rexml/parsers/baseparser.rb (REXML::Parsers::BaseParser::SYSTEM): Fix loose "head" match regular expression. It doesn't cause any problem in the current code but it should be fixed because readers may confuse it. Modified files: trunk/ChangeLog trunk/lib/rexml/parsers/baseparser.rb Index: ChangeLog =================================================================== --- ChangeLog (revision 42483) +++ ChangeLog (revision 42484) @@ -1,3 +1,10 @@ https://github.com/ruby/ruby/blob/trunk/ChangeLog#L1 +Sat Aug 10 12:00:45 2013 Kouhei Sutou <kou@c...> + + * lib/rexml/parsers/baseparser.rb (REXML::Parsers::BaseParser::SYSTEM): + Fix loose "head" match regular expression. It doesn't cause any + problem in the current code but it should be fixed because readers + may confuse it. + Sat Aug 10 11:58:24 2013 Kouhei Sutou <kou@c...> * test/rexml/parse/test_notation_declaration.rb (#test_system_public): Index: lib/rexml/parsers/baseparser.rb =================================================================== --- lib/rexml/parsers/baseparser.rb (revision 42483) +++ lib/rexml/parsers/baseparser.rb (revision 42484) @@ -78,7 +78,7 @@ module REXML https://github.com/ruby/ruby/blob/trunk/lib/rexml/parsers/baseparser.rb#L78 ATTLISTDECL_PATTERN = /^\s*<!ATTLIST\s+#{NAME}(?:#{ATTDEF})*\s*>/um NOTATIONDECL_START = /^\s*<!NOTATION/um PUBLIC = /\A\s*<!NOTATION\s+(\w[\-\w]*)\s+(PUBLIC)\s+(["'])(.*?)\3(?:\s+(["'])(.*?)\5)?\s*>/um - SYSTEM = /^\s*<!NOTATION\s+(\w[\-\w]*)\s+(SYSTEM)\s+(["'])(.*?)\3\s*>/um + SYSTEM = /\A\s*<!NOTATION\s+(\w[\-\w]*)\s+(SYSTEM)\s+(["'])(.*?)\3\s*>/um TEXT_PATTERN = /\A([^<]*)/um -- ML: ruby-changes@q... Info: http://www.atdot.net/~ko1/quickml/