You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@xerces.apache.org by am...@apache.org on 2006/09/26 03:36:44 UTC

svn commit: r449880 - in /xerces/c/trunk/src/xercesc: dom/impl/DOMLSSerializerImpl.hpp util/Transcoders/IconvGNU/IconvGNUTransService.cpp util/XMLNumber.cpp

Author: amassari
Date: Mon Sep 25 18:36:43 2006
New Revision: 449880

URL: http://svn.apache.org/viewvc?view=rev&rev=449880
Log:
Removed warnings (jira#1619)

Modified:
    xerces/c/trunk/src/xercesc/dom/impl/DOMLSSerializerImpl.hpp
    xerces/c/trunk/src/xercesc/util/Transcoders/IconvGNU/IconvGNUTransService.cpp
    xerces/c/trunk/src/xercesc/util/XMLNumber.cpp

Modified: xerces/c/trunk/src/xercesc/dom/impl/DOMLSSerializerImpl.hpp
URL: http://svn.apache.org/viewvc/xerces/c/trunk/src/xercesc/dom/impl/DOMLSSerializerImpl.hpp?view=diff&rev=449880&r1=449879&r2=449880
==============================================================================
--- xerces/c/trunk/src/xercesc/dom/impl/DOMLSSerializerImpl.hpp (original)
+++ xerces/c/trunk/src/xercesc/dom/impl/DOMLSSerializerImpl.hpp Mon Sep 25 18:36:43 2006
@@ -214,12 +214,12 @@
                                     , bool      val)
 {
     (val)? fFeatures |= (1<<featureId) : fFeatures &= ~(1<<featureId);
-};
+}
 
 inline bool DOMLSSerializerImpl::getFeature(const int featureId) const
 {
     return ((fFeatures & ( 1<<featureId )) != 0) ? true : false;
-};
+}
 
 inline void DOMLSSerializerImpl::setURCharRef()
 {

Modified: xerces/c/trunk/src/xercesc/util/Transcoders/IconvGNU/IconvGNUTransService.cpp
URL: http://svn.apache.org/viewvc/xerces/c/trunk/src/xercesc/util/Transcoders/IconvGNU/IconvGNUTransService.cpp?view=diff&rev=449880&r1=449879&r2=449880
==============================================================================
--- xerces/c/trunk/src/xercesc/util/Transcoders/IconvGNU/IconvGNUTransService.cpp (original)
+++ xerces/c/trunk/src/xercesc/util/Transcoders/IconvGNU/IconvGNUTransService.cpp Mon Sep 25 18:36:43 2006
@@ -70,6 +70,8 @@
     { NULL, 0,    0 }
 };
 
+#define MAX_UCHSIZE 4
+
 //--------------------------------------------------
 // Macro-definitions to translate "native unicode"
 // characters <-> XMLCh with different host byte order
@@ -239,7 +241,7 @@
     if (ch <= 0x7F)
         return toupper(ch);
 
-    char    wcbuf[fUChSize * 2];
+    char    wcbuf[MAX_UCHSIZE * 2];
     xmlChToMbc (ch, wcbuf);
 
     char    tmpArr[4];
@@ -269,7 +271,7 @@
     if (ch <= 0x7F)
         return tolower(ch);
 
-    char    wcbuf[fUChSize * 2];
+    char    wcbuf[MAX_UCHSIZE * 2];
     xmlChToMbc (ch, wcbuf);
 
     char    tmpArr[4];
@@ -1059,7 +1061,7 @@
     //  If the passed value is really a surrogate embedded together, then
     //  we need to break it out into its two chars. Else just one.
     //
-    char        srcBuf[2 * uChSize()];
+    char        srcBuf[MAX_UCHSIZE * 2];
     unsigned int    srcCount = 1;
     if (toCheck & 0xFFFF0000) {
         XMLCh    ch1 = (toCheck >> 10) + 0xD800;

Modified: xerces/c/trunk/src/xercesc/util/XMLNumber.cpp
URL: http://svn.apache.org/viewvc/xerces/c/trunk/src/xercesc/util/XMLNumber.cpp?view=diff&rev=449880&r1=449879&r2=449880
==============================================================================
--- xerces/c/trunk/src/xercesc/util/XMLNumber.cpp (original)
+++ xerces/c/trunk/src/xercesc/util/XMLNumber.cpp Mon Sep 25 18:36:43 2006
@@ -84,11 +84,9 @@
     case XMLNumber::UnKnown:
         return 0;
         break;
-    default: //we treat this same as UnKnown
-        return 0;
-        break;
     }
-
+    //we treat this same as UnKnown
+    return 0;
 }
 
 XERCES_CPP_NAMESPACE_END



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