svn commit: r1747631 - /xerces/java/trunk/src/org/apache/xerces/jaxp/datatype/DurationImpl.java

2016-06-09 Thread mukulg 
Author: mukulg
Date: Fri Jun 10 04:44:24 2016
New Revision: 1747631

URL: http://svn.apache.org/viewvc?rev=1747631=rev
Log:
I am reverting a change I did long time ago to this file. I am restoring the 
logic in this file, that was in the original revision 906803. I think, the 
original code that I had changed in this file is not really wrong. I started 
looking at the code in this file, while studying the bug report XERCESJ-1669.

Modified:
xerces/java/trunk/src/org/apache/xerces/jaxp/datatype/DurationImpl.java

Modified: 
xerces/java/trunk/src/org/apache/xerces/jaxp/datatype/DurationImpl.java
URL: 
http://svn.apache.org/viewvc/xerces/java/trunk/src/org/apache/xerces/jaxp/datatype/DurationImpl.java?rev=1747631=1747630=1747631=diff
==
--- xerces/java/trunk/src/org/apache/xerces/jaxp/datatype/DurationImpl.java 
(original)
+++ xerces/java/trunk/src/org/apache/xerces/jaxp/datatype/DurationImpl.java Fri 
Jun 10 04:44:24 2016
@@ -420,16 +420,16 @@ class DurationImpl
 protected DurationImpl(String lexicalRepresentation)
 throws IllegalArgumentException {
 // only if I could use the JDK1.4 regular expression 
-
-if (lexicalRepresentation == null) {
-   throw new NullPointerException();
-}
-
+ 
 final String s = lexicalRepresentation;
 boolean positive;
 int[] idx = new int[1];
 int length = s.length();
 boolean timeRequired = false;
+
+if (lexicalRepresentation == null) {
+throw new NullPointerException();
+}
 
 idx[0] = 0;
 if (length != idx[0] && s.charAt(idx[0]) == '-') {



-
To unsubscribe, e-mail: commits-unsubscr...@xerces.apache.org
For additional commands, e-mail: commits-h...@xerces.apache.org

svn commit: r1747618 - /xerces/c/branches/xerces-3.1/src/xercesc/framework/XMLFormatter.cpp

2016-06-09 Thread scantor 
Author: scantor
Date: Fri Jun 10 01:28:38 2016
New Revision: 1747618

URL: http://svn.apache.org/viewvc?rev=1747618=rev
Log:
https://issues.apache.org/jira/browse/XERCESC-2065

Modified:
xerces/c/branches/xerces-3.1/src/xercesc/framework/XMLFormatter.cpp

Modified: xerces/c/branches/xerces-3.1/src/xercesc/framework/XMLFormatter.cpp
URL: 
http://svn.apache.org/viewvc/xerces/c/branches/xerces-3.1/src/xercesc/framework/XMLFormatter.cpp?rev=1747618=1747617=1747618=diff
==
--- xerces/c/branches/xerces-3.1/src/xercesc/framework/XMLFormatter.cpp 
(original)
+++ xerces/c/branches/xerces-3.1/src/xercesc/framework/XMLFormatter.cpp Fri Jun 
10 01:28:38 2016
@@ -81,7 +81,7 @@ static const XMLCh gEscapeChars[XMLForma
 { chNull  , chNull   , chNull, chNull   , chNull   
 , chNull, chNull }
 ,   { chAmpersand , chCloseAngle , chDoubleQuote , chOpenAngle  , 
chSingleQuote , chNull, chNull }
 ,   { chAmpersand , chOpenAngle  , chDoubleQuote , chLF , chCR 
 , chHTab, chNull }
-,   { chAmpersand , chOpenAngle  , chCloseAngle  , chNull   , chNull   
 , chNull, chNull }
+,   { chAmpersand , chOpenAngle  , chCloseAngle  , chCR , chNull   
 , chNull, chNull }
 };
 
 // ---



-
To unsubscribe, e-mail: commits-unsubscr...@xerces.apache.org
For additional commands, e-mail: commits-h...@xerces.apache.org