You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@xerces.apache.org by ne...@apache.org on 2003/03/27 19:23:17 UTC
cvs commit: xml-xerces/java/src/org/apache/xerces/impl XMLVersionDetector.java
neilg 2003/03/27 10:23:17
Modified: java/src/org/apache/xerces/impl XMLVersionDetector.java
Log:
fix thread safety problem
Revision Changes Path
1.7 +14 -13 xml-xerces/java/src/org/apache/xerces/impl/XMLVersionDetector.java
Index: XMLVersionDetector.java
===================================================================
RCS file: /home/cvs/xml-xerces/java/src/org/apache/xerces/impl/XMLVersionDetector.java,v
retrieving revision 1.6
retrieving revision 1.7
diff -u -r1.6 -r1.7
--- XMLVersionDetector.java 27 Mar 2003 16:27:34 -0000 1.6
+++ XMLVersionDetector.java 27 Mar 2003 18:23:17 -0000 1.7
@@ -84,8 +84,6 @@
//
private final static char[] XML11_VERSION = new char[]{'1', '.', '1'};
- private final static char [] EXPECTED_VERSION_STRING = {'<', '?', 'x', 'm', 'l', ' ', 'v', 'e', 'r', 's',
- 'i', 'o', 'n', '=', ' ', ' ', ' ', ' ', ' '};
// property identifiers
@@ -125,6 +123,9 @@
private XMLString fVersionNum = new XMLString();
+ private final char [] fExpectedVersionString = {'<', '?', 'x', 'm', 'l', ' ', 'v', 'e', 'r', 's',
+ 'i', 'o', 'n', '=', ' ', ' ', ' ', ' ', ' '};
+
/**
*
*
@@ -140,8 +141,8 @@
fSymbolTable = (SymbolTable)componentManager.getProperty(SYMBOL_TABLE);
fErrorReporter = (XMLErrorReporter)componentManager.getProperty(ERROR_REPORTER);
fEntityManager = (XMLEntityManager)componentManager.getProperty(ENTITY_MANAGER);
- for(int i=14; i<EXPECTED_VERSION_STRING.length; i++ )
- EXPECTED_VERSION_STRING[i] = ' ';
+ for(int i=14; i<fExpectedVersionString.length; i++ )
+ fExpectedVersionString[i] = ' ';
} // reset(XMLComponentManager)
/**
@@ -189,30 +190,30 @@
return Constants.XML_VERSION_1_0;
}
if (!scanner.skipSpaces()) {
- fixupCurrentEntity(fEntityManager, EXPECTED_VERSION_STRING, 5);
+ fixupCurrentEntity(fEntityManager, fExpectedVersionString, 5);
return Constants.XML_VERSION_1_0;
}
if (!scanner.skipString("version")) {
- fixupCurrentEntity(fEntityManager, EXPECTED_VERSION_STRING, 6);
+ fixupCurrentEntity(fEntityManager, fExpectedVersionString, 6);
return Constants.XML_VERSION_1_0;
}
scanner.skipSpaces();
if (scanner.scanChar() != '=') {
- fixupCurrentEntity(fEntityManager, EXPECTED_VERSION_STRING, 13);
+ fixupCurrentEntity(fEntityManager, fExpectedVersionString, 13);
return Constants.XML_VERSION_1_0;
}
scanner.skipSpaces();
int quoteChar = scanner.scanChar();
- EXPECTED_VERSION_STRING[14] = (char) quoteChar;
+ fExpectedVersionString[14] = (char) quoteChar;
for (int versionPos = 0; versionPos < XML11_VERSION.length; versionPos++) {
- EXPECTED_VERSION_STRING[15 + versionPos] = (char) scanner.scanChar();
+ fExpectedVersionString[15 + versionPos] = (char) scanner.scanChar();
}
// REVISIT: should we check whether this equals quoteChar?
- EXPECTED_VERSION_STRING[18] = (char) scanner.scanChar();
- fixupCurrentEntity(fEntityManager, EXPECTED_VERSION_STRING, 19);
+ fExpectedVersionString[18] = (char) scanner.scanChar();
+ fixupCurrentEntity(fEntityManager, fExpectedVersionString, 19);
int matched = 0;
for (; matched < XML11_VERSION.length; matched++) {
- if (EXPECTED_VERSION_STRING[15 + matched] != XML11_VERSION[matched])
+ if (fExpectedVersionString[15 + matched] != XML11_VERSION[matched])
break;
}
if (matched == XML11_VERSION.length)
---------------------------------------------------------------------
To unsubscribe, e-mail: xerces-cvs-unsubscribe@xml.apache.org
For additional commands, e-mail: xerces-cvs-help@xml.apache.org