Author: dbertoni
Date: Tue Jul  1 10:55:39 2008
New Revision: 673155

URL: http://svn.apache.org/viewvc?rev=673155&view=rev
Log:
Cleaned up some compiler warnings.

Modified:
    xerces/c/trunk/src/xercesc/internal/IGXMLScanner2.cpp
    xerces/c/trunk/src/xercesc/util/HashCMStateSet.hpp
    xerces/c/trunk/src/xercesc/util/XMLAbstractDoubleFloat.cpp
    xerces/c/trunk/src/xercesc/util/XMLDouble.hpp
    xerces/c/trunk/src/xercesc/util/XMLFloat.hpp
    xerces/c/trunk/src/xercesc/util/XMLString.cpp
    xerces/c/trunk/src/xercesc/util/regx/RegularExpression.cpp
    
xerces/c/trunk/src/xercesc/validators/datatype/AbstractNumericFacetValidator.hpp
    xerces/c/trunk/src/xercesc/validators/schema/GeneralAttributeCheck.cpp
    xerces/c/trunk/src/xercesc/validators/schema/SchemaInfo.hpp
    xerces/c/trunk/src/xercesc/validators/schema/TraverseSchema.cpp
    xerces/c/trunk/src/xercesc/validators/schema/TraverseSchema.hpp

Modified: xerces/c/trunk/src/xercesc/internal/IGXMLScanner2.cpp
URL: 
http://svn.apache.org/viewvc/xerces/c/trunk/src/xercesc/internal/IGXMLScanner2.cpp?rev=673155&r1=673154&r2=673155&view=diff
==============================================================================
--- xerces/c/trunk/src/xercesc/internal/IGXMLScanner2.cpp (original)
+++ xerces/c/trunk/src/xercesc/internal/IGXMLScanner2.cpp Tue Jul  1 10:55:39 
2008
@@ -2320,174 +2320,174 @@
     bool    escaped;
     while (true)
     {
-    try
-    {
-        while(true)
+        try
         {
-            nextCh = fReaderMgr.getNextChar();
-
-            if (!nextCh)
-                ThrowXMLwithMemMgr(UnexpectedEOFException, 
XMLExcepts::Gen_UnexpectedEOF, fMemoryManager);
-
-            // Check for our ending quote in the same entity
-            if (nextCh == quoteCh)
+            while(true)
             {
-                if (curReader == fReaderMgr.getCurrentReaderNum())
-                    return true;
+                nextCh = fReaderMgr.getNextChar();
 
-                // Watch for spillover into a previous entity
-                if (curReader > fReaderMgr.getCurrentReaderNum())
+                if (!nextCh)
+                    ThrowXMLwithMemMgr(UnexpectedEOFException, 
XMLExcepts::Gen_UnexpectedEOF, fMemoryManager);
+
+                // Check for our ending quote in the same entity
+                if (nextCh == quoteCh)
                 {
-                    emitError(XMLErrs::PartialMarkupInEntity);
-                    return false;
+                    if (curReader == fReaderMgr.getCurrentReaderNum())
+                        return true;
+
+                    // Watch for spillover into a previous entity
+                    if (curReader > fReaderMgr.getCurrentReaderNum())
+                    {
+                        emitError(XMLErrs::PartialMarkupInEntity);
+                        return false;
+                    }
                 }
-            }
 
-            //  Check for an entity ref now, before we let it affect our
-            //  whitespace normalization logic below. We ignore the empty flag
-            //  in this one.
-            escaped = false;
-            if (nextCh == chAmpersand)
-            {
-                if (scanEntityRef(true, nextCh, secondCh, escaped) != 
EntityExp_Returned)
+                //  Check for an entity ref now, before we let it affect our
+                //  whitespace normalization logic below. We ignore the empty 
flag
+                //  in this one.
+                escaped = false;
+                if (nextCh == chAmpersand)
                 {
-                    gotLeadingSurrogate = false;
-                    continue;
+                    if (scanEntityRef(true, nextCh, secondCh, escaped) != 
EntityExp_Returned)
+                    {
+                        gotLeadingSurrogate = false;
+                        continue;
+                    }
                 }
-            }
-            else if ((nextCh >= 0xD800) && (nextCh <= 0xDBFF))
-            {
-                // Deal with surrogate pairs
-                //  Its a leading surrogate. If we already got one, then
-                //  issue an error, else set leading flag to make sure that
-                //  we look for a trailing next time.
-                if (gotLeadingSurrogate)
-                    emitError(XMLErrs::Expected2ndSurrogateChar);
-                 else
-                    gotLeadingSurrogate = true;
-            }
-            else
-            {
-                //  If its a trailing surrogate, make sure that we are
-                //  prepared for that. Else, its just a regular char so make
-                //  sure that we were not expected a trailing surrogate.
-                if ((nextCh >= 0xDC00) && (nextCh <= 0xDFFF))
+                else if ((nextCh >= 0xD800) && (nextCh <= 0xDBFF))
                 {
-                    // Its trailing, so make sure we were expecting it
-                    if (!gotLeadingSurrogate)
-                        emitError(XMLErrs::Unexpected2ndSurrogateChar);
+                    // Deal with surrogate pairs
+                    //  Its a leading surrogate. If we already got one, then
+                    //  issue an error, else set leading flag to make sure that
+                    //  we look for a trailing next time.
+                    if (gotLeadingSurrogate)
+                        emitError(XMLErrs::Expected2ndSurrogateChar);
+                     else
+                        gotLeadingSurrogate = true;
                 }
                 else
                 {
-                    //  Its just a char, so make sure we were not expecting a
-                    //  trailing surrogate.
-                    if (gotLeadingSurrogate)
-                        emitError(XMLErrs::Expected2ndSurrogateChar);
-
-                    // Its got to at least be a valid XML character
-                    if (!fReaderMgr.getCurrentReader()->isXMLChar(nextCh))
+                    //  If its a trailing surrogate, make sure that we are
+                    //  prepared for that. Else, its just a regular char so 
make
+                    //  sure that we were not expected a trailing surrogate.
+                    if ((nextCh >= 0xDC00) && (nextCh <= 0xDFFF))
                     {
-                        XMLCh tmpBuf[9];
-                        XMLString::binToText
-                        (
-                            nextCh
-                            , tmpBuf
-                            , 8
-                            , 16
-                            , fMemoryManager
-                        );
-                        emitError(XMLErrs::InvalidCharacterInAttrValue, 
attrName, tmpBuf);
+                        // Its trailing, so make sure we were expecting it
+                        if (!gotLeadingSurrogate)
+                            emitError(XMLErrs::Unexpected2ndSurrogateChar);
                     }
+                    else
+                    {
+                        //  Its just a char, so make sure we were not 
expecting a
+                        //  trailing surrogate.
+                        if (gotLeadingSurrogate)
+                            emitError(XMLErrs::Expected2ndSurrogateChar);
+
+                        // Its got to at least be a valid XML character
+                        if (!fReaderMgr.getCurrentReader()->isXMLChar(nextCh))
+                        {
+                            XMLCh tmpBuf[9];
+                            XMLString::binToText
+                            (
+                                nextCh
+                                , tmpBuf
+                                , 8
+                                , 16
+                                , fMemoryManager
+                            );
+                            emitError(XMLErrs::InvalidCharacterInAttrValue, 
attrName, tmpBuf);
+                        }
+                    }
+                    gotLeadingSurrogate = false;
                 }
-                gotLeadingSurrogate = false;
-            }
 
-            //  If its not escaped, then make sure its not a < character, which
-            //  is not allowed in attribute values.
-            if (!escaped && (nextCh == chOpenAngle))
-                emitError(XMLErrs::BracketInAttrValue, attrName);
-
-            //  If the attribute is a CDATA type we do simple replacement of
-            //  tabs and new lines with spaces, if the character is not escaped
-            //  by way of a char ref.
-            //
-            //  Otherwise, we do the standard non-CDATA normalization of
-            //  compressing whitespace to single spaces and getting rid of 
leading
-            //  and trailing whitespace.
-            if (type == XMLAttDef::CData)
-            {
-                if (!escaped)
+                //  If its not escaped, then make sure its not a < character, 
which
+                //  is not allowed in attribute values.
+                if (!escaped && (nextCh == chOpenAngle))
+                    emitError(XMLErrs::BracketInAttrValue, attrName);
+
+                //  If the attribute is a CDATA type we do simple replacement 
of
+                //  tabs and new lines with spaces, if the character is not 
escaped
+                //  by way of a char ref.
+                //
+                //  Otherwise, we do the standard non-CDATA normalization of
+                //  compressing whitespace to single spaces and getting rid of 
leading
+                //  and trailing whitespace.
+                if (type == XMLAttDef::CData)
                 {
-                    if ((nextCh == 0x09) || (nextCh == 0x0A) || (nextCh == 
0x0D))
+                    if (!escaped)
                     {
-                        // Check Validity Constraint for Standalone document 
declaration
-                        // XML 1.0, Section 2.9
-                        if (fStandalone && fValidate && isAttExternal)
+                        if ((nextCh == 0x09) || (nextCh == 0x0A) || (nextCh == 
0x0D))
                         {
-                            // Can't have a standalone document declaration of 
"yes" if  attribute
-                            // values are subject to normalisation
-                            
fValidator->emitError(XMLValid::NoAttNormForStandalone, attrName);
+                            // Check Validity Constraint for Standalone 
document declaration
+                            // XML 1.0, Section 2.9
+                            if (fStandalone && fValidate && isAttExternal)
+                            {
+                                // Can't have a standalone document 
declaration of "yes" if  attribute
+                                // values are subject to normalisation
+                                
fValidator->emitError(XMLValid::NoAttNormForStandalone, attrName);
+                            }
+                            nextCh = chSpace;
                         }
-                        nextCh = chSpace;
                     }
                 }
-            }
-            else
-            {
-                if (curState == InWhitespace)
+                else
                 {
-                    if ((escaped && nextCh != chSpace) || 
!fReaderMgr.getCurrentReader()->isWhitespace(nextCh))
-                    {
-                        if (firstNonWS)
-                            toFill.append(chSpace);
-                        curState = InContent;
-                        firstNonWS = true;
-                    }
-                    else
+                    if (curState == InWhitespace)
                     {
-                        continue;
+                        if ((escaped && nextCh != chSpace) || 
!fReaderMgr.getCurrentReader()->isWhitespace(nextCh))
+                        {
+                            if (firstNonWS)
+                                toFill.append(chSpace);
+                            curState = InContent;
+                            firstNonWS = true;
+                        }
+                        else
+                        {
+                            continue;
+                        }
                     }
-                }
-                else if (curState == InContent)
-                {
-                    if ((nextCh == chSpace) ||
-                        (fReaderMgr.getCurrentReader()->isWhitespace(nextCh) 
&& !escaped))
+                    else if (curState == InContent)
                     {
-                        curState = InWhitespace;
-
-                        // Check Validity Constraint for Standalone document 
declaration
-                        // XML 1.0, Section 2.9
-                        if (fStandalone && fValidate && isAttExternal)
+                        if ((nextCh == chSpace) ||
+                            
(fReaderMgr.getCurrentReader()->isWhitespace(nextCh) && !escaped))
                         {
-                            if (!firstNonWS || (nextCh != chSpace) || 
(fReaderMgr.lookingAtSpace()))
+                            curState = InWhitespace;
+
+                            // Check Validity Constraint for Standalone 
document declaration
+                            // XML 1.0, Section 2.9
+                            if (fStandalone && fValidate && isAttExternal)
                             {
-                                 // Can't have a standalone document 
declaration of "yes" if  attribute
-                                 // values are subject to normalisation
-                                 
fValidator->emitError(XMLValid::NoAttNormForStandalone, attrName);
+                                if (!firstNonWS || (nextCh != chSpace) || 
(fReaderMgr.lookingAtSpace()))
+                                {
+                                     // Can't have a standalone document 
declaration of "yes" if  attribute
+                                     // values are subject to normalisation
+                                     
fValidator->emitError(XMLValid::NoAttNormForStandalone, attrName);
+                                }
                             }
+                            continue;
                         }
-                        continue;
+                        firstNonWS = true;
                     }
-                    firstNonWS = true;
                 }
-            }
 
-            // Else add it to the buffer
-            toFill.append(nextCh);
+                // Else add it to the buffer
+                toFill.append(nextCh);
 
-            if (secondCh)
-            {
-                toFill.append(secondCh);
-                secondCh=0;
+                if (secondCh)
+                {
+                    toFill.append(secondCh);
+                    secondCh=0;
+                }
             }
         }
-    }
-    catch(const EndOfEntityException&)
-    {
-        // Just eat it and continue.
-        gotLeadingSurrogate = false;
-        escaped = false;
-    }
+        catch(const EndOfEntityException&)
+        {
+            // Just eat it and continue.
+            gotLeadingSurrogate = false;
+            escaped = false;
+        }
     }
     return true;
 }

Modified: xerces/c/trunk/src/xercesc/util/HashCMStateSet.hpp
URL: 
http://svn.apache.org/viewvc/xerces/c/trunk/src/xercesc/util/HashCMStateSet.hpp?rev=673155&r1=673154&r2=673155&view=diff
==============================================================================
--- xerces/c/trunk/src/xercesc/util/HashCMStateSet.hpp (original)
+++ xerces/c/trunk/src/xercesc/util/HashCMStateSet.hpp Tue Jul  1 10:55:39 2008
@@ -62,14 +62,14 @@
 inline unsigned int HashCMStateSet::getHashVal(const void *const key, unsigned 
int mod
                                                , MemoryManager* const)
 {
-    const CMStateSet* const pkey = (const CMStateSet* const) key;
+    const CMStateSet* const pkey = (const CMStateSet*) key;
        return ((pkey->hashCode()) % mod);
 }
 
 inline bool HashCMStateSet::equals(const void *const key1, const void *const 
key2)
 {
-    const CMStateSet* const pkey1 = (const CMStateSet* const) key1;
-    const CMStateSet* const pkey2 = (const CMStateSet* const) key2;
+    const CMStateSet* const pkey1 = (const CMStateSet*) key1;
+    const CMStateSet* const pkey2 = (const CMStateSet*) key2;
 
        return (*pkey1==*pkey2);
 }

Modified: xerces/c/trunk/src/xercesc/util/XMLAbstractDoubleFloat.cpp
URL: 
http://svn.apache.org/viewvc/xerces/c/trunk/src/xercesc/util/XMLAbstractDoubleFloat.cpp?rev=673155&r1=673154&r2=673155&view=diff
==============================================================================
--- xerces/c/trunk/src/xercesc/util/XMLAbstractDoubleFloat.cpp (original)
+++ xerces/c/trunk/src/xercesc/util/XMLAbstractDoubleFloat.cpp Tue Jul  1 
10:55:39 2008
@@ -307,8 +307,6 @@
     {
         return (-1) * compareSpecial(rValue, manager);
     }
-
-    return 0;
 }
 
 int XMLAbstractDoubleFloat::compareSpecial(const XMLAbstractDoubleFloat* const 
specialValue                                         

Modified: xerces/c/trunk/src/xercesc/util/XMLDouble.hpp
URL: 
http://svn.apache.org/viewvc/xerces/c/trunk/src/xercesc/util/XMLDouble.hpp?rev=673155&r1=673154&r2=673155&view=diff
==============================================================================
--- xerces/c/trunk/src/xercesc/util/XMLDouble.hpp (original)
+++ xerces/c/trunk/src/xercesc/util/XMLDouble.hpp Tue Jul  1 10:55:39 2008
@@ -88,8 +88,8 @@
 inline int XMLDouble::compareValues(const XMLDouble* const lValue
                                   , const XMLDouble* const rValue)
 {
-    return XMLAbstractDoubleFloat::compareValues((const 
XMLAbstractDoubleFloat* const) lValue,
-                                                 (const 
XMLAbstractDoubleFloat* const) rValue 
+    return XMLAbstractDoubleFloat::compareValues((const 
XMLAbstractDoubleFloat*) lValue,
+                                                 (const 
XMLAbstractDoubleFloat*) rValue 
                                                  , 
((XMLAbstractDoubleFloat*)lValue)->getMemoryManager());
 }
 

Modified: xerces/c/trunk/src/xercesc/util/XMLFloat.hpp
URL: 
http://svn.apache.org/viewvc/xerces/c/trunk/src/xercesc/util/XMLFloat.hpp?rev=673155&r1=673154&r2=673155&view=diff
==============================================================================
--- xerces/c/trunk/src/xercesc/util/XMLFloat.hpp (original)
+++ xerces/c/trunk/src/xercesc/util/XMLFloat.hpp Tue Jul  1 10:55:39 2008
@@ -87,8 +87,8 @@
 inline int XMLFloat::compareValues(const XMLFloat* const lValue
                                  , const XMLFloat* const rValue)
 {
-    return XMLAbstractDoubleFloat::compareValues((const 
XMLAbstractDoubleFloat* const) lValue,
-                                                 (const 
XMLAbstractDoubleFloat* const) rValue 
+    return XMLAbstractDoubleFloat::compareValues((const 
XMLAbstractDoubleFloat*) lValue,
+                                                 (const 
XMLAbstractDoubleFloat*) rValue 
                                                  , 
((XMLAbstractDoubleFloat*)lValue)->getMemoryManager());
 }
 

Modified: xerces/c/trunk/src/xercesc/util/XMLString.cpp
URL: 
http://svn.apache.org/viewvc/xerces/c/trunk/src/xercesc/util/XMLString.cpp?rev=673155&r1=673154&r2=673155&view=diff
==============================================================================
--- xerces/c/trunk/src/xercesc/util/XMLString.cpp (original)
+++ xerces/c/trunk/src/xercesc/util/XMLString.cpp Tue Jul  1 10:55:39 2008
@@ -716,7 +716,7 @@
     const XMLSize_t srcLen = strlen(srcStr);
 
     // Make sure the start index is within the XMLString bounds
-    if ( startIndex < 0 || startIndex > endIndex || endIndex > srcLen)
+    if (startIndex > endIndex || endIndex > srcLen)
         ThrowXMLwithMemMgr(ArrayIndexOutOfBoundsException, 
XMLExcepts::Str_StartIndexPastEnd, manager);
 
     const XMLSize_t copySize = endIndex - startIndex;
@@ -1701,7 +1701,7 @@
         ThrowXMLwithMemMgr(IllegalArgumentException, 
XMLExcepts::Str_ZeroSizedTargetBuf, manager);
 
     // Make sure the start index is within the XMLString bounds
-    if ( startIndex < 0 || startIndex > endIndex || endIndex > srcStrLength)
+    if (startIndex > endIndex || endIndex > srcStrLength)
         ThrowXMLwithMemMgr(ArrayIndexOutOfBoundsException, 
XMLExcepts::Str_StartIndexPastEnd, manager);
 
     const XMLSize_t copySize = endIndex - startIndex;

Modified: xerces/c/trunk/src/xercesc/util/regx/RegularExpression.cpp
URL: 
http://svn.apache.org/viewvc/xerces/c/trunk/src/xercesc/util/regx/RegularExpression.cpp?rev=673155&r1=673154&r2=673155&view=diff
==============================================================================
--- xerces/c/trunk/src/xercesc/util/regx/RegularExpression.cpp (original)
+++ xerces/c/trunk/src/xercesc/util/regx/RegularExpression.cpp Tue Jul  1 
10:55:39 2008
@@ -149,35 +149,39 @@
     }
 }
 
-RegularExpression::Context& RegularExpression::Context::operator= (const 
RegularExpression::Context& other)
+RegularExpression::Context& RegularExpression::Context::operator=(const 
RegularExpression::Context& other)
 {
-    fStart=other.fStart;
-    fLimit=other.fLimit;
-    fLength=other.fLength;
-    fSize=other.fSize;
-    fStringMaxLen=other.fStringMaxLen;
-    fString=other.fString;
-    fOptions=other.fOptions;
-    if (fOffsets)
-        fMemoryManager->deallocate(fOffsets);//delete [] fOffsets;
-    fOffsets=0;
-    if (fAdoptMatch)
-        delete fMatch;
-    fMatch=0;
-    fAdoptMatch=false;
-
-    fMemoryManager=other.fMemoryManager;
-    if(other.fOffsets)
-    {
-        fOffsets = (int*) fMemoryManager->allocate(fSize* sizeof(int));
-        for (int i = 0; i< fSize; i++)
-            fOffsets[i] = other.fOffsets[i];
-    }
-    if(other.fMatch)
+    if (this != &other)
     {
-        fMatch=new Match(*other.fMatch);
-        fAdoptMatch=true;
+        fStart=other.fStart;
+        fLimit=other.fLimit;
+        fLength=other.fLength;
+        fSize=other.fSize;
+        fStringMaxLen=other.fStringMaxLen;
+        fString=other.fString;
+        fOptions=other.fOptions;
+        if (fOffsets)
+            fMemoryManager->deallocate(fOffsets);//delete [] fOffsets;
+        fOffsets=0;
+        if (fAdoptMatch)
+            delete fMatch;
+        fMatch=0;
+        fAdoptMatch=false;
+
+        fMemoryManager=other.fMemoryManager;
+        if(other.fOffsets)
+        {
+            fOffsets = (int*) fMemoryManager->allocate(fSize* sizeof(int));
+            for (int i = 0; i< fSize; i++)
+                fOffsets[i] = other.fOffsets[i];
+        }
+        if(other.fMatch)
+        {
+            fMatch=new Match(*other.fMatch);
+            fAdoptMatch=true;
+        }
     }
+
     return *this;
 }
 
@@ -753,7 +757,7 @@
     XMLSize_t matchStart = start;
     while(matchStart <= end) {
         XMLSize_t matchEnd = match(&context, fOperations, matchStart, 1);
-        if(matchEnd != -1) {
+        if(matchEnd != (XMLSize_t)-1) {
             context.fMatch->setStartPos(0, matchStart);
             context.fMatch->setEndPos(0, matchEnd);
 

Modified: 
xerces/c/trunk/src/xercesc/validators/datatype/AbstractNumericFacetValidator.hpp
URL: 
http://svn.apache.org/viewvc/xerces/c/trunk/src/xercesc/validators/datatype/AbstractNumericFacetValidator.hpp?rev=673155&r1=673154&r2=673155&view=diff
==============================================================================
--- 
xerces/c/trunk/src/xercesc/validators/datatype/AbstractNumericFacetValidator.hpp
 (original)
+++ 
xerces/c/trunk/src/xercesc/validators/datatype/AbstractNumericFacetValidator.hpp
 Tue Jul  1 10:55:39 2008
@@ -105,13 +105,13 @@
 // Getter methods
 // -----------------------------------------------------------------------
 
-    inline XMLNumber* const            getMaxInclusive() const;
+    inline XMLNumber*                  getMaxInclusive() const;
 
-    inline XMLNumber* const            getMaxExclusive() const;
+    inline XMLNumber*                  getMaxExclusive() const;
 
-    inline XMLNumber* const            getMinInclusive() const;
+    inline XMLNumber*                  getMinInclusive() const;
 
-    inline XMLNumber* const            getMinExclusive() const;
+    inline XMLNumber*                  getMinExclusive() const;
 
     inline RefVectorOf<XMLNumber>*     getEnumeration() const;
 
@@ -167,22 +167,22 @@
 // Getter methods
 // -----------------------------------------------------------------------
 
-inline XMLNumber* const AbstractNumericFacetValidator::getMaxInclusive() const
+inline XMLNumber* AbstractNumericFacetValidator::getMaxInclusive() const
 {
     return fMaxInclusive;
 }
 
-inline XMLNumber* const AbstractNumericFacetValidator::getMaxExclusive() const
+inline XMLNumber* AbstractNumericFacetValidator::getMaxExclusive() const
 {
     return fMaxExclusive;
 }
 
-inline XMLNumber* const AbstractNumericFacetValidator::getMinInclusive() const
+inline XMLNumber* AbstractNumericFacetValidator::getMinInclusive() const
 {
     return fMinInclusive;
 }
 
-inline XMLNumber* const AbstractNumericFacetValidator::getMinExclusive() const
+inline XMLNumber* AbstractNumericFacetValidator::getMinExclusive() const
 {
     return fMinExclusive;
 }

Modified: xerces/c/trunk/src/xercesc/validators/schema/GeneralAttributeCheck.cpp
URL: 
http://svn.apache.org/viewvc/xerces/c/trunk/src/xercesc/validators/schema/GeneralAttributeCheck.cpp?rev=673155&r1=673154&r2=673155&view=diff
==============================================================================
--- xerces/c/trunk/src/xercesc/validators/schema/GeneralAttributeCheck.cpp 
(original)
+++ xerces/c/trunk/src/xercesc/validators/schema/GeneralAttributeCheck.cpp Tue 
Jul  1 10:55:39 2008
@@ -189,12 +189,12 @@
     }
 
     DOMNamedNodeMap* eltAttrs = elem->getAttributes();
-    unsigned int     attrCount = eltAttrs->getLength();
+    const XMLSize_t  attrCount = eltAttrs->getLength();
     XMLByte          attList[A_Count];
 
     memset(attList, 0, sizeof(attList));
 
-    for (unsigned int i = 0; i < attrCount; i++) {
+    for (XMLSize_t i = 0; i < attrCount; i++) {
 
         DOMNode*     attribute = eltAttrs->item(i);
         const XMLCh* attName = attribute->getNodeName();

Modified: xerces/c/trunk/src/xercesc/validators/schema/SchemaInfo.hpp
URL: 
http://svn.apache.org/viewvc/xerces/c/trunk/src/xercesc/validators/schema/SchemaInfo.hpp?rev=673155&r1=673154&r2=673155&view=diff
==============================================================================
--- xerces/c/trunk/src/xercesc/validators/schema/SchemaInfo.hpp (original)
+++ xerces/c/trunk/src/xercesc/validators/schema/SchemaInfo.hpp Tue Jul  1 
10:55:39 2008
@@ -92,8 +92,8 @@
     // -----------------------------------------------------------------------
     //  Getter methods
     // -----------------------------------------------------------------------
-    XMLCh* const                      getCurrentSchemaURL() const;
-    const XMLCh* const                getTargetNSURIString() const;
+    XMLCh*                            getCurrentSchemaURL() const;
+    const XMLCh*                      getTargetNSURIString() const;
     const DOMElement*                 getRoot() const;
     bool                              getProcessed() const;
     int                               getBlockDefault() const;
@@ -205,12 +205,12 @@
     return fNamespaceScope;
 }
 
-inline XMLCh* const SchemaInfo::getCurrentSchemaURL() const {
+inline XMLCh* SchemaInfo::getCurrentSchemaURL() const {
 
     return fCurrentSchemaURL;
 }
 
-inline const XMLCh* const SchemaInfo::getTargetNSURIString() const {
+inline const XMLCh* SchemaInfo::getTargetNSURIString() const {
 
     return fTargetNSURIString;
 }

Modified: xerces/c/trunk/src/xercesc/validators/schema/TraverseSchema.cpp
URL: 
http://svn.apache.org/viewvc/xerces/c/trunk/src/xercesc/validators/schema/TraverseSchema.cpp?rev=673155&r1=673154&r2=673155&view=diff
==============================================================================
--- xerces/c/trunk/src/xercesc/validators/schema/TraverseSchema.cpp (original)
+++ xerces/c/trunk/src/xercesc/validators/schema/TraverseSchema.cpp Tue Jul  1 
10:55:39 2008
@@ -1402,7 +1402,7 @@
     // Create a new instance
     // -----------------------------------------------------------------------
     unsigned int previousCircularCheckIndex = fCircularCheckIndex;
-    int previousScope = fCurrentScope;
+    unsigned int previousScope = fCurrentScope;
 
     if (preProcessFlag) {
 
@@ -1629,7 +1629,7 @@
     // ------------------------------------------------------------------
     // Process contents of global groups
     // ------------------------------------------------------------------
-    int saveScope = fCurrentScope;
+    unsigned int saveScope = fCurrentScope;
     Janitor<ContentSpecNode> specNode(0);
     XercesGroupInfo* saveGroupInfo = fCurrentGroupInfo;
 
@@ -4704,9 +4704,9 @@
 
     DOMNamedNodeMap* eltAttrs = elem->getAttributes();
     bool seenNS=false;
-    unsigned int attrCount = eltAttrs->getLength();
+    const XMLSize_t attrCount = eltAttrs->getLength();
 
-    for (unsigned int i = 0; i < attrCount; i++) {
+    for (XMLSize_t i = 0; i < attrCount; i++) {
 
         DOMNode* attribute = eltAttrs->item(i);
 
@@ -5089,7 +5089,7 @@
 
         SchemaInfo::ListType infoType = SchemaInfo::INCLUDE;
         SchemaInfo* saveInfo = fSchemaInfo;
-        int                  saveScope = fCurrentScope;
+        unsigned int         saveScope = fCurrentScope;
 
         if (!XMLString::equals(uri, fTargetNSURIString) && (uri && *uri)) {
 
@@ -5381,7 +5381,7 @@
     DatatypeValidator* dv = 0;
     SchemaInfo::ListType infoType = SchemaInfo::INCLUDE;
     SchemaInfo*          saveInfo = fSchemaInfo;
-    int                  saveScope = fCurrentScope;
+    unsigned int         saveScope = fCurrentScope;
 
     if (otherSchemaURI && *otherSchemaURI) {
 
@@ -5458,7 +5458,7 @@
     DatatypeValidator*   dv = getDatatypeValidator(typeURI, localPart);
     SchemaInfo::ListType infoType = SchemaInfo::INCLUDE;
     SchemaInfo*          saveInfo = fSchemaInfo;
-    int                  saveScope = fCurrentScope;
+    unsigned int         saveScope = fCurrentScope;
 
     if (!XMLString::equals(typeURI, fTargetNSURIString)
         && (typeURI && *typeURI)) {
@@ -5521,7 +5521,7 @@
     ComplexTypeInfo*     typeInfo = 0;
     SchemaInfo*          saveInfo = fSchemaInfo;
     SchemaInfo::ListType infoType = SchemaInfo::INCLUDE;
-    int                  saveScope = fCurrentScope;
+    unsigned int         saveScope = fCurrentScope;
 
     fBuffer.set(typeURI);
     fBuffer.append(chComma);
@@ -5596,7 +5596,7 @@
     SchemaElementDecl*   elemDecl = 0;
     SchemaInfo*          saveInfo = fSchemaInfo;
     SchemaInfo::ListType infoType = SchemaInfo::INCLUDE;
-    int                  saveScope = fCurrentScope;
+    unsigned int         saveScope = fCurrentScope;
     unsigned int         uriId = fURIStringPool->addOrFind(nameURI);
 
     if (fSchemaInfo->getTargetNSURI() != (int) uriId)
@@ -5858,7 +5858,7 @@
     SchemaInfo* saveInfo = fSchemaInfo;
     SchemaInfo::ListType infoType = SchemaInfo::INCLUDE;
     SchemaAttDef* refAttDef = 0;
-    int saveScope = fCurrentScope;
+    unsigned int saveScope = fCurrentScope;
 
     if (!XMLString::equals(uriStr, fTargetNSURIString)) {
 
@@ -6496,7 +6496,7 @@
     ComplexTypeInfo*     baseComplexTypeInfo = 0;
     DatatypeValidator*   baseDTValidator = 0;
     SchemaInfo::ListType infoType = SchemaInfo::INCLUDE;
-    int                  saveScope = fCurrentScope;
+    unsigned int         saveScope = fCurrentScope;
 
     // check if the base type is from another schema
     if (!XMLString::equals(uriStr, fTargetNSURIString)) {
@@ -6941,7 +6941,7 @@
 
 void TraverseSchema::restoreSchemaInfo(SchemaInfo* const toRestore,
                                        SchemaInfo::ListType const aListType,
-                                       const int saveScope) {
+                                       const unsigned int saveScope) {
 
 
     if (aListType == SchemaInfo::IMPORT) { // restore grammar info
@@ -7188,7 +7188,7 @@
     XercesGroupInfo*     groupInfo = 0;
     SchemaInfo*          saveInfo = fSchemaInfo;
     SchemaInfo::ListType infoType = SchemaInfo::INCLUDE;
-    int                  saveScope = fCurrentScope;
+    unsigned int         saveScope = fCurrentScope;
 
     //if from another target namespace
     if (!XMLString::equals(uriStr, fTargetNSURIString)) {
@@ -7288,7 +7288,7 @@
     XercesAttGroupInfo*  attGroupInfo = 0;
     SchemaInfo*          saveInfo = fSchemaInfo;
     SchemaInfo::ListType infoType = SchemaInfo::INCLUDE;
-    int                  saveScope = fCurrentScope;
+    unsigned int         saveScope = fCurrentScope;
 
     if (!XMLString::equals(uriStr, fTargetNSURIString)) {
 
@@ -9096,7 +9096,7 @@
     // next is the namespaces on the elem
     DOMElement* currentElem = (DOMElement*) elem;
     DOMNamedNodeMap* eltAttrs;
-    unsigned int     attrCount;
+    XMLSize_t     attrCount;
     do {
         eltAttrs = currentElem->getAttributes();
         attrCount = eltAttrs->getLength();

Modified: xerces/c/trunk/src/xercesc/validators/schema/TraverseSchema.hpp
URL: 
http://svn.apache.org/viewvc/xerces/c/trunk/src/xercesc/validators/schema/TraverseSchema.hpp?rev=673155&r1=673154&r2=673155&view=diff
==============================================================================
--- xerces/c/trunk/src/xercesc/validators/schema/TraverseSchema.hpp (original)
+++ xerces/c/trunk/src/xercesc/validators/schema/TraverseSchema.hpp Tue Jul  1 
10:55:39 2008
@@ -510,7 +510,7 @@
 
     void restoreSchemaInfo(SchemaInfo* const toRestore,
                            SchemaInfo::ListType const aListType = 
SchemaInfo::INCLUDE,
-                           const int saveScope = Grammar::TOP_LEVEL_SCOPE);
+                           const unsigned int saveScope = 
Grammar::TOP_LEVEL_SCOPE);
     void  popCurrentTypeNameStack();
 
     /**



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to