[前][次][番号順一覧][スレッド一覧]

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/

[前][次][番号順一覧][スレッド一覧]