From 4ee49f169093c71b9e964ae3b19709c8330848e0 Mon Sep 17 00:00:00 2001 From: MortenMacFly Date: Mon, 14 Jan 2013 20:03:14 +0100 Subject: [PATCH] Eliminate warnings with GCC/MinGW --- tinyxml2.cpp | 22 ++++++++++++---------- tinyxml2.h | 38 +++++++++++++++++++------------------- 2 files changed, 31 insertions(+), 29 deletions(-) diff --git a/tinyxml2.cpp b/tinyxml2.cpp index 61e9b58..647901b 100755 --- a/tinyxml2.cpp +++ b/tinyxml2.cpp @@ -326,6 +326,8 @@ void XMLUtil::ConvertUTF32ToUTF8( unsigned long input, char* output, int* length case 1: --output; *output = (char)(input | FIRST_BYTE_MARK[*length]); + default: + break; } } @@ -1229,11 +1231,11 @@ const char* XMLElement::GetText() const } -XMLError XMLElement::QueryIntText( int* _value ) const +XMLError XMLElement::QueryIntText( int* ival ) const { if ( FirstChild() && FirstChild()->ToText() ) { const char* t = FirstChild()->ToText()->Value(); - if ( XMLUtil::ToInt( t, _value ) ) { + if ( XMLUtil::ToInt( t, ival ) ) { return XML_SUCCESS; } return XML_CAN_NOT_CONVERT_TEXT; @@ -1242,11 +1244,11 @@ XMLError XMLElement::QueryIntText( int* _value ) const } -XMLError XMLElement::QueryUnsignedText( unsigned* _value ) const +XMLError XMLElement::QueryUnsignedText( unsigned* uval ) const { if ( FirstChild() && FirstChild()->ToText() ) { const char* t = FirstChild()->ToText()->Value(); - if ( XMLUtil::ToUnsigned( t, _value ) ) { + if ( XMLUtil::ToUnsigned( t, uval ) ) { return XML_SUCCESS; } return XML_CAN_NOT_CONVERT_TEXT; @@ -1255,11 +1257,11 @@ XMLError XMLElement::QueryUnsignedText( unsigned* _value ) const } -XMLError XMLElement::QueryBoolText( bool* _value ) const +XMLError XMLElement::QueryBoolText( bool* bval ) const { if ( FirstChild() && FirstChild()->ToText() ) { const char* t = FirstChild()->ToText()->Value(); - if ( XMLUtil::ToBool( t, _value ) ) { + if ( XMLUtil::ToBool( t, bval ) ) { return XML_SUCCESS; } return XML_CAN_NOT_CONVERT_TEXT; @@ -1268,11 +1270,11 @@ XMLError XMLElement::QueryBoolText( bool* _value ) const } -XMLError XMLElement::QueryDoubleText( double* _value ) const +XMLError XMLElement::QueryDoubleText( double* dval ) const { if ( FirstChild() && FirstChild()->ToText() ) { const char* t = FirstChild()->ToText()->Value(); - if ( XMLUtil::ToDouble( t, _value ) ) { + if ( XMLUtil::ToDouble( t, dval ) ) { return XML_SUCCESS; } return XML_CAN_NOT_CONVERT_TEXT; @@ -1281,11 +1283,11 @@ XMLError XMLElement::QueryDoubleText( double* _value ) const } -XMLError XMLElement::QueryFloatText( float* _value ) const +XMLError XMLElement::QueryFloatText( float* fval ) const { if ( FirstChild() && FirstChild()->ToText() ) { const char* t = FirstChild()->ToText()->Value(); - if ( XMLUtil::ToFloat( t, _value ) ) { + if ( XMLUtil::ToFloat( t, fval ) ) { return XML_SUCCESS; } return XML_CAN_NOT_CONVERT_TEXT; diff --git a/tinyxml2.h b/tinyxml2.h index 55f57a3..0f8e9ad 100755 --- a/tinyxml2.h +++ b/tinyxml2.h @@ -46,7 +46,7 @@ distribution. /* gcc: g++ -Wall -DDEBUG tinyxml2.cpp xmltest.cpp -o gccxmltest.exe - + Formatting, Artistic Style: AStyle.exe --style=1tbs --indent-switches --break-closing-brackets --indent-preprocessor tinyxml2.cpp tinyxml2.h */ @@ -554,27 +554,27 @@ public: /// Safely cast to an Element, or null. virtual XMLElement* ToElement() { - return 0; + return 0; } /// Safely cast to Text, or null. virtual XMLText* ToText() { - return 0; + return 0; } /// Safely cast to a Comment, or null. virtual XMLComment* ToComment() { - return 0; + return 0; } /// Safely cast to a Document, or null. virtual XMLDocument* ToDocument() { - return 0; + return 0; } /// Safely cast to a Declaration, or null. virtual XMLDeclaration* ToDeclaration() { - return 0; + return 0; } /// Safely cast to an Unknown, or null. virtual XMLUnknown* ToUnknown() { - return 0; + return 0; } virtual const XMLElement* ToElement() const { @@ -608,7 +608,7 @@ public: const char* Value() const { return _value.GetStr(); } - + /** Set the Value of an XML node. @sa Value() */ @@ -618,7 +618,7 @@ public: const XMLNode* Parent() const { return _parent; } - + XMLNode* Parent() { return _parent; } @@ -632,11 +632,11 @@ public: const XMLNode* FirstChild() const { return _firstChild; } - + XMLNode* FirstChild() { return _firstChild; } - + /** Get the first child element, or optionally the first child element with the specified name. */ @@ -977,15 +977,15 @@ class XMLAttribute public: /// The name of the attribute. const char* Name() const { - return _name.GetStr(); + return _name.GetStr(); } /// The value of the attribute. const char* Value() const { - return _value.GetStr(); + return _value.GetStr(); } /// The next attribute in the list. const XMLAttribute* Next() const { - return _next; + return _next; } /** IntAttribute interprets the attribute as an integer, and returns the value. @@ -1301,15 +1301,15 @@ public: to the requested type, and XML_NO_TEXT_NODE if there is no child text to query. */ - XMLError QueryIntText( int* _value ) const; + XMLError QueryIntText( int* ival ) const; /// See QueryIntText() - XMLError QueryUnsignedText( unsigned* _value ) const; + XMLError QueryUnsignedText( unsigned* uval ) const; /// See QueryIntText() - XMLError QueryBoolText( bool* _value ) const; + XMLError QueryBoolText( bool* bval ) const; /// See QueryIntText() - XMLError QueryDoubleText( double* _value ) const; + XMLError QueryDoubleText( double* dval ) const; /// See QueryIntText() - XMLError QueryFloatText( float* _value ) const; + XMLError QueryFloatText( float* fval ) const; // internal: enum {