From 092c5c42cc6ca6316cb6d286fa78b527ec9213ab Mon Sep 17 00:00:00 2001 From: Alberto Massari <amassari@apache.org> Date: Fri, 7 Jan 2005 15:12:11 +0000 Subject: [PATCH] Removed warnings git-svn-id: https://svn.apache.org/repos/asf/xerces/c/trunk@176279 13f79535-47bb-0310-9956-ffa450edef68 --- src/xercesc/util/BaseRefVectorOf.c | 4 ++- src/xercesc/util/BitSet.cpp | 7 +++-- src/xercesc/util/HexBin.cpp | 5 +++- src/xercesc/util/KVStringPair.cpp | 7 ++++- src/xercesc/util/NameIdPool.c | 8 ++++-- src/xercesc/util/OutOfMemoryException.hpp | 2 +- src/xercesc/util/QName.cpp | 15 +++++++---- src/xercesc/util/RefHash3KeysIdPool.c | 7 ++++- src/xercesc/util/RefHashTableOf.c | 7 ++++- .../Transcoders/Cygwin/CygwinTransService.cpp | 26 +++++++++++-------- .../Transcoders/Cygwin/CygwinTransService.hpp | 26 +++++++++---------- src/xercesc/util/ValueArrayOf.c | 7 +++-- src/xercesc/util/ValueVectorOf.c | 7 +++-- src/xercesc/util/XMLBigInteger.cpp | 6 ++++- src/xercesc/util/XMLDateTime.cpp | 6 ++++- src/xercesc/util/XMLEnumerator.hpp | 9 ++++--- src/xercesc/util/XMLException.cpp | 4 +-- src/xercesc/util/XMLNumber.cpp | 8 ++++++ src/xercesc/util/XMLNumber.hpp | 7 +++-- src/xercesc/util/XMLResourceIdentifier.hpp | 7 +++-- src/xercesc/util/XMLString.cpp | 2 +- src/xercesc/util/XMLURL.cpp | 12 ++++----- src/xercesc/util/XMLUri.cpp | 4 ++- src/xercesc/util/regx/BMPattern.cpp | 5 +++- src/xercesc/util/regx/Match.cpp | 8 ++++-- 25 files changed, 140 insertions(+), 66 deletions(-) diff --git a/src/xercesc/util/BaseRefVectorOf.c b/src/xercesc/util/BaseRefVectorOf.c index 2396f5d91..49f8f0162 100644 --- a/src/xercesc/util/BaseRefVectorOf.c +++ b/src/xercesc/util/BaseRefVectorOf.c @@ -312,7 +312,9 @@ template <class TElem> BaseRefVectorEnumerator<TElem>::~BaseRefVectorEnumerator( template <class TElem> BaseRefVectorEnumerator<TElem>:: BaseRefVectorEnumerator(const BaseRefVectorEnumerator<TElem>& toCopy) : - fAdopted(toCopy.fAdopted) + XMLEnumerator<TElem>(toCopy) + , XMemory(toCopy) + , fAdopted(toCopy.fAdopted) , fCurIndex(toCopy.fCurIndex) , fToEnum(toCopy.fToEnum) { diff --git a/src/xercesc/util/BitSet.cpp b/src/xercesc/util/BitSet.cpp index 28c1bf288..914916f31 100644 --- a/src/xercesc/util/BitSet.cpp +++ b/src/xercesc/util/BitSet.cpp @@ -16,6 +16,9 @@ /* * $Log$ + * Revision 1.5 2005/01/07 15:12:10 amassari + * Removed warnings + * * Revision 1.4 2004/09/08 13:56:21 peiyongz * Apache License Version 2.0 * @@ -85,8 +88,8 @@ BitSet::BitSet( const unsigned int size } BitSet::BitSet(const BitSet& toCopy) : - - fMemoryManager(toCopy.fMemoryManager) + XMemory(toCopy) + , fMemoryManager(toCopy.fMemoryManager) , fBits(0) , fUnitLen(toCopy.fUnitLen) { diff --git a/src/xercesc/util/HexBin.cpp b/src/xercesc/util/HexBin.cpp index c1bd7b9cc..8c80a2dec 100644 --- a/src/xercesc/util/HexBin.cpp +++ b/src/xercesc/util/HexBin.cpp @@ -16,6 +16,9 @@ /* * $Log$ + * Revision 1.7 2005/01/07 15:12:10 amassari + * Removed warnings + * * Revision 1.6 2004/12/10 10:37:56 cargilld * Fix problem with hexbin::decode and use XMLByte instead of XMLCh for output of decoding. * @@ -188,7 +191,7 @@ void HexBin::init() int i; for ( i = 0; i < BASELENGTH; i++ ) - hexNumberTable[i] = -1; + hexNumberTable[i] = (XMLByte)-1; for ( i = chDigit_9; i >= chDigit_0; i-- ) hexNumberTable[i] = (XMLByte) (i - chDigit_0); diff --git a/src/xercesc/util/KVStringPair.cpp b/src/xercesc/util/KVStringPair.cpp index d51bca028..bdf9005b6 100644 --- a/src/xercesc/util/KVStringPair.cpp +++ b/src/xercesc/util/KVStringPair.cpp @@ -16,6 +16,9 @@ /* * $Log$ + * Revision 1.8 2005/01/07 15:12:10 amassari + * Removed warnings + * * Revision 1.7 2004/10/28 20:14:41 peiyongz * Data member reshuffle * @@ -119,7 +122,9 @@ KVStringPair::KVStringPair(const XMLCh* const key, } KVStringPair::KVStringPair(const KVStringPair& toCopy) -:fKeyAllocSize(0) +:XSerializable(toCopy) +,XMemory(toCopy) +,fKeyAllocSize(0) ,fValueAllocSize(0) ,fKey(0) ,fValue(0) diff --git a/src/xercesc/util/NameIdPool.c b/src/xercesc/util/NameIdPool.c index 74f35f4f1..fb1079264 100644 --- a/src/xercesc/util/NameIdPool.c +++ b/src/xercesc/util/NameIdPool.c @@ -16,6 +16,9 @@ /** * $Log$ + * Revision 1.12 2005/01/07 15:12:10 amassari + * Removed warnings + * * Revision 1.11 2004/11/19 00:50:22 cargilld * Memory improvement to utility classes from Christian Will. Remove dependency on XMemory. * @@ -358,8 +361,9 @@ NameIdPoolEnumerator(NameIdPool<TElem>* const toEnum template <class TElem> NameIdPoolEnumerator<TElem>:: NameIdPoolEnumerator(const NameIdPoolEnumerator<TElem>& toCopy) : - - fCurIndex(toCopy.fCurIndex) + XMLEnumerator<TElem>(toCopy) + , XMemory(toCopy) + , fCurIndex(toCopy.fCurIndex) , fToEnum(toCopy.fToEnum) , fMemoryManager(toCopy.fMemoryManager) { diff --git a/src/xercesc/util/OutOfMemoryException.hpp b/src/xercesc/util/OutOfMemoryException.hpp index 451b4b3b1..133c4cc88 100755 --- a/src/xercesc/util/OutOfMemoryException.hpp +++ b/src/xercesc/util/OutOfMemoryException.hpp @@ -56,7 +56,7 @@ public: // constructors/destructors... inline OutOfMemoryException::OutOfMemoryException() {} inline OutOfMemoryException::~OutOfMemoryException() {} -inline OutOfMemoryException::OutOfMemoryException(const OutOfMemoryException&) {} +inline OutOfMemoryException::OutOfMemoryException(const OutOfMemoryException& other) : XMemory(other) {} inline OutOfMemoryException& OutOfMemoryException::operator=(const OutOfMemoryException&) { return *this; diff --git a/src/xercesc/util/QName.cpp b/src/xercesc/util/QName.cpp index 7b618a963..15392cf1d 100644 --- a/src/xercesc/util/QName.cpp +++ b/src/xercesc/util/QName.cpp @@ -16,6 +16,9 @@ /* * $Log$ + * Revision 1.15 2005/01/07 15:12:10 amassari + * Removed warnings + * * Revision 1.14 2004/10/28 20:14:41 peiyongz * Data member reshuffle * @@ -100,9 +103,9 @@ QName::QName(MemoryManager* const manager) ,fLocalPartBufSz(0) ,fRawNameBufSz(0) ,fURIId(0) -,fRawName(0) ,fPrefix(0) ,fLocalPart(0) +,fRawName(0) ,fMemoryManager(manager) { } @@ -115,9 +118,9 @@ QName::QName( const XMLCh* const prefix ,fLocalPartBufSz(0) ,fRawNameBufSz(0) ,fURIId(0) -,fRawName(0) ,fPrefix(0) ,fLocalPart(0) +,fRawName(0) ,fMemoryManager(manager) { try @@ -145,9 +148,9 @@ QName::QName( const XMLCh* const rawName ,fLocalPartBufSz(0) ,fRawNameBufSz(0) ,fURIId(0) -,fRawName(0) ,fPrefix(0) ,fLocalPart(0) +,fRawName(0) ,fMemoryManager(manager) { try @@ -177,13 +180,15 @@ QName::~QName() // QName: Copy Constructors // --------------------------------------------------------------------------- QName::QName(const QName& qname) -:fPrefixBufSz(0) +:XSerializable(qname) +,XMemory(qname) +,fPrefixBufSz(0) ,fLocalPartBufSz(0) ,fRawNameBufSz(0) ,fURIId(0) -,fRawName(0) ,fPrefix(0) ,fLocalPart(0) +,fRawName(0) ,fMemoryManager(qname.fMemoryManager) { unsigned int newLen; diff --git a/src/xercesc/util/RefHash3KeysIdPool.c b/src/xercesc/util/RefHash3KeysIdPool.c index f678ac10a..6c7608220 100644 --- a/src/xercesc/util/RefHash3KeysIdPool.c +++ b/src/xercesc/util/RefHash3KeysIdPool.c @@ -16,6 +16,9 @@ /** * $Log$ + * Revision 1.16 2005/01/07 15:12:10 amassari + * Removed warnings + * * Revision 1.15 2004/11/19 00:50:22 cargilld * Memory improvement to utility classes from Christian Will. Remove dependency on XMemory. * @@ -458,7 +461,9 @@ template <class TVal> RefHash3KeysIdPoolEnumerator<TVal>::~RefHash3KeysIdPoolEnu template <class TVal> RefHash3KeysIdPoolEnumerator<TVal>:: RefHash3KeysIdPoolEnumerator(const RefHash3KeysIdPoolEnumerator<TVal>& toCopy) : - fAdoptedElems(toCopy.fAdoptedElems) + XMLEnumerator<TVal>(toCopy) + , XMemory(toCopy) + , fAdoptedElems(toCopy.fAdoptedElems) , fCurIndex(toCopy.fCurIndex) , fToEnum(toCopy.fToEnum) , fCurElem(toCopy.fCurElem) diff --git a/src/xercesc/util/RefHashTableOf.c b/src/xercesc/util/RefHashTableOf.c index 73c80fa2d..6f27cece3 100644 --- a/src/xercesc/util/RefHashTableOf.c +++ b/src/xercesc/util/RefHashTableOf.c @@ -16,6 +16,9 @@ /** * $Log$ + * Revision 1.19 2005/01/07 15:12:10 amassari + * Removed warnings + * * Revision 1.18 2004/11/19 00:50:22 cargilld * Memory improvement to utility classes from Christian Will. Remove dependency on XMemory. * @@ -634,7 +637,9 @@ template <class TVal> RefHashTableOfEnumerator<TVal>::~RefHashTableOfEnumerator( template <class TVal> RefHashTableOfEnumerator<TVal>:: RefHashTableOfEnumerator(const RefHashTableOfEnumerator<TVal>& toCopy) : - fAdopted(toCopy.fAdopted) + XMLEnumerator<TVal>(toCopy) + , XMemory(toCopy) + , fAdopted(toCopy.fAdopted) , fCurElem(toCopy.fCurElem) , fCurHash(toCopy.fCurHash) , fToEnum(toCopy.fToEnum) diff --git a/src/xercesc/util/Transcoders/Cygwin/CygwinTransService.cpp b/src/xercesc/util/Transcoders/Cygwin/CygwinTransService.cpp index 25866a399..f17ed592c 100644 --- a/src/xercesc/util/Transcoders/Cygwin/CygwinTransService.cpp +++ b/src/xercesc/util/Transcoders/Cygwin/CygwinTransService.cpp @@ -16,6 +16,9 @@ /* * $Log$ + * Revision 1.14 2005/01/07 15:12:11 amassari + * Removed warnings + * * Revision 1.13 2004/09/08 13:56:43 peiyongz * Apache License Version 2.0 * @@ -485,22 +488,23 @@ CygwinTransService::~CygwinTransService() // --------------------------------------------------------------------------- int CygwinTransService::auxCompareString( const XMLCh* const comp1 , const XMLCh* const comp2 - , signed long maxChars + , signed long sMaxChars , const bool ignoreCase) { const XMLCh* args[2] = { comp1, comp2 }; XMLCh* firstBuf = NULL; - XMLCh* secondBuf; + XMLCh* secondBuf = NULL; unsigned int len = XMLString::stringLen( comp1); unsigned int otherLen = XMLString::stringLen( comp2); unsigned int countChar = 0; + unsigned int maxChars; int theResult = 0; // Determine at what string index the comparison stops. - if ( maxChars != -1L ) + if ( sMaxChars != -1L ) { - len = ( len > maxChars ) ? maxChars : len; - otherLen = ( otherLen > maxChars ) ? maxChars : otherLen; + len = ( len > (unsigned int)sMaxChars ) ? (unsigned int)sMaxChars : len; + otherLen = ( otherLen > (unsigned int)sMaxChars ) ? (unsigned int)sMaxChars : otherLen; maxChars = ( len > otherLen ) ? otherLen : len; } else @@ -608,7 +612,7 @@ const XMLCh* CygwinTransService::getId() const bool CygwinTransService::isSpace(const XMLCh toCheck) const { - int theCount = 0; + unsigned int theCount = 0; while ( theCount < (sizeof(gWhitespace) / sizeof(XMLCh)) ) { if ( toCheck == gWhitespace[theCount] ) @@ -683,7 +687,7 @@ CygwinTransService::makeNewXMLTranscoder(const XMLCh* const encoding // Get an upper cased copy of the encoding name, since we use a hash // table and we store them all in upper case. // - int itsLen = XMLString::stringLen( encodingName) + 1; + unsigned int itsLen = XMLString::stringLen( encodingName) + 1; memcpy( upEncoding , encodingName @@ -976,7 +980,7 @@ CygwinLCPTranscoder::~CygwinLCPTranscoder() // CygwinLCPTranscoder: Implementation of the virtual transcoder interface // --------------------------------------------------------------------------- unsigned int CygwinLCPTranscoder::calcRequiredSize(const char* const srcText - , MemoryManager* const manager) + , MemoryManager* const /*manager*/) { if (!srcText) return 0; @@ -986,7 +990,7 @@ unsigned int CygwinLCPTranscoder::calcRequiredSize(const char* const srcText unsigned int CygwinLCPTranscoder::calcRequiredSize(const XMLCh* const srcText - , MemoryManager* const manager) + , MemoryManager* const /*manager*/) { if (!srcText) return 0; @@ -1116,7 +1120,7 @@ XMLCh* CygwinLCPTranscoder::transcode(const char* const toTranscode, bool CygwinLCPTranscoder::transcode( const char* const toTranscode , XMLCh* const toFill , const unsigned int maxChars - , MemoryManager* const manager) + , MemoryManager* const /*manager*/) { // Check for a couple of psycho corner cases if (!toTranscode || !maxChars) @@ -1141,7 +1145,7 @@ bool CygwinLCPTranscoder::transcode( const char* const toTranscode bool CygwinLCPTranscoder::transcode( const XMLCh* const toTranscode , char* const toFill , const unsigned int maxBytes - , MemoryManager* const manager) + , MemoryManager* const /*manager*/) { // Watch for a couple of pyscho corner cases if (!toTranscode || !maxBytes) diff --git a/src/xercesc/util/Transcoders/Cygwin/CygwinTransService.hpp b/src/xercesc/util/Transcoders/Cygwin/CygwinTransService.hpp index 2a11f3866..97733ca96 100644 --- a/src/xercesc/util/Transcoders/Cygwin/CygwinTransService.hpp +++ b/src/xercesc/util/Transcoders/Cygwin/CygwinTransService.hpp @@ -83,7 +83,14 @@ protected : , MemoryManager* const manager ); - virtual int auxCompareString +private : + // ----------------------------------------------------------------------- + // Unimplemented constructors and operators + // ----------------------------------------------------------------------- + CygwinTransService(const CygwinTransService&); + CygwinTransService& operator=(const CygwinTransService&); + + int auxCompareString ( const XMLCh* const comp1 , const XMLCh* const comp2 @@ -91,12 +98,10 @@ protected : , const bool ignoreCase ); -private : - // ----------------------------------------------------------------------- - // Unimplemented constructors and operators - // ----------------------------------------------------------------------- - CygwinTransService(const CygwinTransService&); - CygwinTransService& operator=(const CygwinTransService&); + static bool isAlias(const HKEY encodingKey + , char* const aliasBuf = 0 + , const unsigned int nameBufSz = 0); + // This is a hash table of entries which map encoding names to their // Windows specific code pages. The code page allows us to create @@ -106,13 +111,6 @@ private : // This map is shared unsynchronized among all threads of the process, // which is cool since it will be read only once its initialized. - - - static bool isAlias(const HKEY encodingKey - , char* const aliasBuf = 0 - , const unsigned int nameBufSz = 0); - - RefHashTableOf<CPMapEntry> *fCPMap; }; diff --git a/src/xercesc/util/ValueArrayOf.c b/src/xercesc/util/ValueArrayOf.c index 534e227ca..d314e1bb1 100644 --- a/src/xercesc/util/ValueArrayOf.c +++ b/src/xercesc/util/ValueArrayOf.c @@ -16,6 +16,9 @@ /** * $Log$ + * Revision 1.8 2005/01/07 15:12:10 amassari + * Removed warnings + * * Revision 1.7 2004/09/08 13:56:23 peiyongz * Apache License Version 2.0 * @@ -94,8 +97,8 @@ ValueArrayOf<TElem>::ValueArrayOf( const TElem* values template <class TElem> ValueArrayOf<TElem>::ValueArrayOf(const ValueArrayOf<TElem>& source) : - - fSize(source.fSize) + XMemory(source) + , fSize(source.fSize) , fArray(0) , fMemoryManager(source.fMemoryManager) { diff --git a/src/xercesc/util/ValueVectorOf.c b/src/xercesc/util/ValueVectorOf.c index 02d5d682b..3d10b6968 100644 --- a/src/xercesc/util/ValueVectorOf.c +++ b/src/xercesc/util/ValueVectorOf.c @@ -16,6 +16,9 @@ /** * $Log$ + * Revision 1.12 2005/01/07 15:12:10 amassari + * Removed warnings + * * Revision 1.11 2004/11/11 01:31:54 peiyongz * Avoid unnecessary expansion -- patch from Christian * @@ -106,8 +109,8 @@ ValueVectorOf<TElem>::ValueVectorOf(const unsigned int maxElems, template <class TElem> ValueVectorOf<TElem>::ValueVectorOf(const ValueVectorOf<TElem>& toCopy) : - - fCallDestructor(toCopy.fCallDestructor) + XMemory(toCopy) + , fCallDestructor(toCopy.fCallDestructor) , fCurCount(toCopy.fCurCount) , fMaxCount(toCopy.fMaxCount) , fElemList(0) diff --git a/src/xercesc/util/XMLBigInteger.cpp b/src/xercesc/util/XMLBigInteger.cpp index 26e8cc67f..d716e4140 100644 --- a/src/xercesc/util/XMLBigInteger.cpp +++ b/src/xercesc/util/XMLBigInteger.cpp @@ -16,6 +16,9 @@ /* * $Log$ + * Revision 1.15 2005/01/07 15:12:10 amassari + * Removed warnings + * * Revision 1.14 2004/09/08 13:56:24 peiyongz * Apache License Version 2.0 * @@ -284,7 +287,8 @@ XMLBigInteger::~XMLBigInteger() } XMLBigInteger::XMLBigInteger(const XMLBigInteger& toCopy) -: fSign(toCopy.fSign) +: XMemory(toCopy) +, fSign(toCopy.fSign) , fMagnitude(0) , fRawData(0) , fMemoryManager(toCopy.fMemoryManager) diff --git a/src/xercesc/util/XMLDateTime.cpp b/src/xercesc/util/XMLDateTime.cpp index b7532a20d..21f2d0df9 100644 --- a/src/xercesc/util/XMLDateTime.cpp +++ b/src/xercesc/util/XMLDateTime.cpp @@ -17,6 +17,9 @@ /* * $Id$ * $Log$ + * Revision 1.30 2005/01/07 15:12:10 amassari + * Removed warnings + * * Revision 1.29 2004/10/28 20:13:35 peiyongz * Data member reshuffle * @@ -537,7 +540,8 @@ XMLDateTime::XMLDateTime(const XMLCh* const aString, // ----------------------------------------------------------------------- XMLDateTime::XMLDateTime(const XMLDateTime &toCopy) -: fBufferMaxLen(0) +: XMLNumber(toCopy) +, fBufferMaxLen(0) , fBuffer(0) , fMemoryManager(toCopy.fMemoryManager) { diff --git a/src/xercesc/util/XMLEnumerator.hpp b/src/xercesc/util/XMLEnumerator.hpp index 7ef2077ad..f0b0bc3f5 100644 --- a/src/xercesc/util/XMLEnumerator.hpp +++ b/src/xercesc/util/XMLEnumerator.hpp @@ -16,6 +16,9 @@ /* * $Log$ + * Revision 1.6 2005/01/07 15:12:10 amassari + * Removed warnings + * * Revision 1.5 2004/09/08 13:56:24 peiyongz * Apache License Version 2.0 * @@ -73,13 +76,13 @@ public : virtual TElem& nextElement() = 0; virtual void Reset() = 0; - XMLEnumerator() {}; + XMLEnumerator() {} + XMLEnumerator(const XMLEnumerator<TElem>&) {} private: // ----------------------------------------------------------------------- - // Unimplemented constructors and operators + // Unimplemented operators // ----------------------------------------------------------------------- - XMLEnumerator(const XMLEnumerator<TElem>&); XMLEnumerator<TElem>& operator=(const XMLEnumerator<TElem>&); }; diff --git a/src/xercesc/util/XMLException.cpp b/src/xercesc/util/XMLException.cpp index a4c3d04d2..dd8e5690a 100644 --- a/src/xercesc/util/XMLException.cpp +++ b/src/xercesc/util/XMLException.cpp @@ -157,8 +157,8 @@ XMLException::XMLException( const char* const srcFile XMLException::XMLException(const XMLException& toCopy) : - - fCode(toCopy.fCode) + XMemory(toCopy) + , fCode(toCopy.fCode) , fSrcFile(0) , fSrcLine(toCopy.fSrcLine) , fMsg(XMLString::replicate(toCopy.fMsg, toCopy.fMemoryManager)) diff --git a/src/xercesc/util/XMLNumber.cpp b/src/xercesc/util/XMLNumber.cpp index 0ab86ebf3..f4e06369b 100644 --- a/src/xercesc/util/XMLNumber.cpp +++ b/src/xercesc/util/XMLNumber.cpp @@ -17,6 +17,9 @@ /* * $Id$ * $Log$ + * Revision 1.7 2005/01/07 15:12:10 amassari + * Removed warnings + * * Revision 1.6 2004/09/08 13:56:24 peiyongz * Apache License Version 2.0 * @@ -61,6 +64,11 @@ XERCES_CPP_NAMESPACE_BEGIN XMLNumber::XMLNumber() {} +XMLNumber::XMLNumber(const XMLNumber& toCopy) +: XSerializable(toCopy) +, XMemory(toCopy) +{} + XMLNumber::~XMLNumber() {} diff --git a/src/xercesc/util/XMLNumber.hpp b/src/xercesc/util/XMLNumber.hpp index 1db7eacea..89fbe83f4 100644 --- a/src/xercesc/util/XMLNumber.hpp +++ b/src/xercesc/util/XMLNumber.hpp @@ -17,6 +17,9 @@ /* * $Id$ * $Log$ + * Revision 1.15 2005/01/07 15:12:10 amassari + * Removed warnings + * * Revision 1.14 2004/09/08 13:56:24 peiyongz * Apache License Version 2.0 * @@ -143,12 +146,12 @@ public: protected: XMLNumber(); + XMLNumber(const XMLNumber&); private: // ----------------------------------------------------------------------- - // Unimplemented constructors and operators + // Unimplemented operators // ----------------------------------------------------------------------- - XMLNumber(const XMLNumber&); XMLNumber& operator=(const XMLNumber&); }; diff --git a/src/xercesc/util/XMLResourceIdentifier.hpp b/src/xercesc/util/XMLResourceIdentifier.hpp index 93b3eb90e..4712710db 100644 --- a/src/xercesc/util/XMLResourceIdentifier.hpp +++ b/src/xercesc/util/XMLResourceIdentifier.hpp @@ -16,6 +16,9 @@ /* * $Log$ + * Revision 1.11 2005/01/07 15:12:10 amassari + * Removed warnings + * * Revision 1.10 2004/12/21 16:32:52 cargilld * Attempt to fix various apidoc problems. * @@ -225,10 +228,10 @@ inline XMLResourceIdentifier::XMLResourceIdentifier(const ResourceIdentifierType , const XMLCh* const publicId , const XMLCh* const baseURI ) : fResourceIdentifierType(resourceIdentifierType) - , fSystemId(systemId) - , fNameSpace(nameSpace) , fPublicId(publicId) + , fSystemId(systemId) , fBaseURI(baseURI) + , fNameSpace(nameSpace) { } diff --git a/src/xercesc/util/XMLString.cpp b/src/xercesc/util/XMLString.cpp index 2af615c90..54adc5ab2 100644 --- a/src/xercesc/util/XMLString.cpp +++ b/src/xercesc/util/XMLString.cpp @@ -1764,7 +1764,7 @@ void XMLString::removeChar(const XMLCh* const srcString dstBuffer.reset(); - while (c=*pszSrc++) + while ((c=*pszSrc++)!=0) { if (c != toRemove) dstBuffer.append(c); diff --git a/src/xercesc/util/XMLURL.cpp b/src/xercesc/util/XMLURL.cpp index 7f0b0b403..f7800ac94 100644 --- a/src/xercesc/util/XMLURL.cpp +++ b/src/xercesc/util/XMLURL.cpp @@ -354,8 +354,8 @@ XMLURL::XMLURL(const char* const urlText, } XMLURL::XMLURL(const XMLURL& toCopy) : - - fMemoryManager(toCopy.fMemoryManager) + XMemory(toCopy) + , fMemoryManager(toCopy.fMemoryManager) , fFragment(0) , fHost(0) , fPassword(0) @@ -913,8 +913,8 @@ void XMLURL::parse(const XMLCh* const urlText) // // The first thing we will do is to check for a file name, so that - // we don't waste time thinking its a URL. If its in the form x:\ - // or x:/ and x is an ASCII letter, then assume that's the deal. + // we don't waste time thinking its a URL. If its in the form x:\ or x:/ + // and x is an ASCII letter, then assume that's the deal. // if (((*urlText >= chLatin_A) && (*urlText <= chLatin_Z)) || ((*urlText >= chLatin_a) && (*urlText <= chLatin_z))) @@ -1205,8 +1205,8 @@ bool XMLURL::parse(const XMLCh* const urlText, XMLURL& xmlURL) // // The first thing we will do is to check for a file name, so that - // we don't waste time thinking its a URL. If its in the form x:\ - // or x:/ and x is an ASCII letter, then assume that's the deal. + // we don't waste time thinking its a URL. If its in the form x:\ or x:/ + // and x is an ASCII letter, then assume that's the deal. // if (((*urlText >= chLatin_A) && (*urlText <= chLatin_Z)) || ((*urlText >= chLatin_a) && (*urlText <= chLatin_z))) diff --git a/src/xercesc/util/XMLUri.cpp b/src/xercesc/util/XMLUri.cpp index 599b3bd91..7481c1eff 100644 --- a/src/xercesc/util/XMLUri.cpp +++ b/src/xercesc/util/XMLUri.cpp @@ -294,7 +294,9 @@ XMLUri::XMLUri(const XMLUri* const baseURI //Copy constructor XMLUri::XMLUri(const XMLUri& toCopy) -: fPort(-1) +: XSerializable(toCopy) +, XMemory(toCopy) +, fPort(-1) , fScheme(0) , fUserInfo(0) , fHost(0) diff --git a/src/xercesc/util/regx/BMPattern.cpp b/src/xercesc/util/regx/BMPattern.cpp index 1ca7d5e77..61c10f6fa 100644 --- a/src/xercesc/util/regx/BMPattern.cpp +++ b/src/xercesc/util/regx/BMPattern.cpp @@ -16,6 +16,9 @@ /* * $Log$ + * Revision 1.6 2005/01/07 15:12:11 amassari + * Removed warnings + * * Revision 1.5 2004/09/08 13:56:47 peiyongz * Apache License Version 2.0 * @@ -137,7 +140,7 @@ int BMPattern::matches(const XMLCh* const content, int start, int limit) { int patternIndex = patternLen; int nIndex = index + 1; - XMLCh ch; + XMLCh ch = 0; while (patternIndex > 0) { diff --git a/src/xercesc/util/regx/Match.cpp b/src/xercesc/util/regx/Match.cpp index 2510c8e27..83b6269c1 100644 --- a/src/xercesc/util/regx/Match.cpp +++ b/src/xercesc/util/regx/Match.cpp @@ -16,6 +16,9 @@ /* * $Log$ + * Revision 1.6 2005/01/07 15:12:11 amassari + * Removed warnings + * * Revision 1.5 2004/09/08 13:56:47 peiyongz * Apache License Version 2.0 * @@ -61,7 +64,8 @@ Match::Match(MemoryManager* const manager) : } Match::Match(const Match& toCopy) : - fNoGroups(0) + XMemory(toCopy) + , fNoGroups(0) , fPositionsSize(0) , fStartPositions(0) , fEndPositions(0) @@ -70,7 +74,7 @@ Match::Match(const Match& toCopy) : initialize(toCopy); } -Match& Match::operator=(const Match& toAssign){ +Match& Match::operator=(const Match& toAssign) { initialize(toAssign); return *this; -- GitLab