From 114d76765ed2012ecb43d76fa2e61467b8dd1052 Mon Sep 17 00:00:00 2001 From: Kim Kulling Date: Fri, 8 Jan 2016 16:46:13 +0100 Subject: [PATCH] OpenDDLParser: latest greatest to get fix for coverity finding ( dereferenfing possible null ptr ). --- contrib/openddlparser/code/OpenDDLCommon.cpp | 10 ++ contrib/openddlparser/code/OpenDDLExport.cpp | 39 ++++-- contrib/openddlparser/code/OpenDDLParser.cpp | 130 +++++++++++++----- contrib/openddlparser/code/Value.cpp | 10 ++ .../include/openddlparser/OpenDDLCommon.h | 45 +++--- .../include/openddlparser/OpenDDLExport.h | 16 ++- .../include/openddlparser/OpenDDLParser.h | 8 +- .../openddlparser/OpenDDLParserUtils.h | 47 ++++--- .../include/openddlparser/Value.h | 108 +++++++++++++-- 9 files changed, 315 insertions(+), 98 deletions(-) diff --git a/contrib/openddlparser/code/OpenDDLCommon.cpp b/contrib/openddlparser/code/OpenDDLCommon.cpp index 00583a101..27a264541 100644 --- a/contrib/openddlparser/code/OpenDDLCommon.cpp +++ b/contrib/openddlparser/code/OpenDDLCommon.cpp @@ -151,6 +151,16 @@ DataArrayList::~DataArrayList() { // empty } +size_t DataArrayList::size() { + size_t result=1; + DataArrayList *n=m_next; + while( n!=ddl_nullptr ) { + result++; + n=n->m_next; + } + return result; +} + Context::Context() : m_root( ddl_nullptr ) { // empty diff --git a/contrib/openddlparser/code/OpenDDLExport.cpp b/contrib/openddlparser/code/OpenDDLExport.cpp index ba0138d0c..212bfa66a 100644 --- a/contrib/openddlparser/code/OpenDDLExport.cpp +++ b/contrib/openddlparser/code/OpenDDLExport.cpp @@ -29,12 +29,30 @@ CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. BEGIN_ODDLPARSER_NS -IOStreamBase::IOStreamBase() -: m_file( ddl_nullptr ) { - // empty +StreamFormatterBase::StreamFormatterBase() { + } + +StreamFormatterBase::~StreamFormatterBase() { + +} + +std::string StreamFormatterBase::format( const std::string &statement ) { + std::string tmp( statement ); + return tmp; +} + +IOStreamBase::IOStreamBase( StreamFormatterBase *formatter ) +: m_formatter( formatter ) +, m_file( ddl_nullptr ) { + if (ddl_nullptr == m_formatter) { + m_formatter = new StreamFormatterBase; + } +} + IOStreamBase::~IOStreamBase() { - // empty + delete m_formatter; + m_formatter = ddl_nullptr; } bool IOStreamBase::open( const std::string &name ) { @@ -57,12 +75,12 @@ bool IOStreamBase::close() { return true; } -void IOStreamBase::write( const std::string &statement ) { +size_t IOStreamBase::write( const std::string &statement ) { if (ddl_nullptr == m_file) { - return; + return 0; } - - ::fwrite( statement.c_str(), sizeof( char ), statement.size(), m_file ); + std::string formatStatement = m_formatter->format( statement ); + return ::fwrite( formatStatement.c_str(), sizeof( char ), formatStatement.size(), m_file ); } struct DDLNodeIterator { @@ -88,6 +106,10 @@ struct DDLNodeIterator { return false; } + +private: + DDLNodeIterator() ddl_no_copy; + DDLNodeIterator &operator = ( const DDLNodeIterator & ) ddl_no_copy; }; static void writeLineEnd( std::string &statement ) { @@ -410,3 +432,4 @@ bool OpenDDLExport::writeValueArray( DataArrayList *al, std::string &statement ) } END_ODDLPARSER_NS + diff --git a/contrib/openddlparser/code/OpenDDLParser.cpp b/contrib/openddlparser/code/OpenDDLParser.cpp index dbc90f990..99e7dfb8c 100644 --- a/contrib/openddlparser/code/OpenDDLParser.cpp +++ b/contrib/openddlparser/code/OpenDDLParser.cpp @@ -33,7 +33,6 @@ CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. # include #endif // _WIN32 -#define DEBUG_HEADER_NAME BEGIN_ODDLPARSER_NS @@ -74,15 +73,26 @@ const char *getTypeToken( Value::ValueType type ) { static void logInvalidTokenError( char *in, const std::string &exp, OpenDDLParser::logCallback callback ) { std::stringstream stream; - stream << "Invalid token " << *in << ", " << exp << " expected." << std::endl; + stream << "Invalid token \"" << *in << "\"" << " expected \"" << exp << "\"" << std::endl; + std::string full(in); + std::string part(full.substr(0,50)); + stream << part; callback( ddl_error_msg, stream.str() ); } static bool isIntegerType( Value::ValueType integerType ) { - if( integerType != Value::ddl_int8 && integerType != Value::ddl_int16 && + if( integerType != Value::ddl_int8 && integerType != Value::ddl_int16 && integerType != Value::ddl_int32 && integerType != Value::ddl_int64 ) { return false; } + return true; +} + +static bool isUnsignedIntegerType( Value::ValueType integerType ) { + if( integerType != Value::ddl_unsigned_int8 && integerType != Value::ddl_unsigned_int16 && + integerType != Value::ddl_unsigned_int32 && integerType != Value::ddl_unsigned_int64 ) { + return false; + } return true; } @@ -193,7 +203,7 @@ bool OpenDDLParser::parse() { if( m_buffer.empty() ) { return false; } - + normalizeBuffer( m_buffer ); m_context = new Context; @@ -206,6 +216,9 @@ bool OpenDDLParser::parse() { size_t pos( current - &m_buffer[ 0 ] ); while( pos < m_buffer.size() ) { current = parseNextNode( current, end ); + if(current==ddl_nullptr) { + return false; + } pos = current - &m_buffer[ 0 ]; } return true; @@ -215,11 +228,9 @@ bool OpenDDLParser::exportContext( Context *ctx, const std::string &filename ) { if( ddl_nullptr == ctx ) { return false; } - + OpenDDLExport myExporter; return myExporter.exportContext( ctx, filename ); - - return false; } char *OpenDDLParser::parseNextNode( char *in, char *end ) { @@ -229,12 +240,15 @@ char *OpenDDLParser::parseNextNode( char *in, char *end ) { return in; } +#ifdef DEBUG_HEADER_NAME static void dumpId( Identifier *id ) { if( ddl_nullptr != id ) { - if ( ddl_nullptr != id->m_text.m_buffer ) { } + if ( ddl_nullptr != id->m_text.m_buffer ) { std::cout << id->m_text.m_buffer << std::endl; + } } } +#endif char *OpenDDLParser::parseHeader( char *in, char *end ) { if( ddl_nullptr == in || in == end ) { @@ -244,7 +258,7 @@ char *OpenDDLParser::parseHeader( char *in, char *end ) { Identifier *id( ddl_nullptr ); in = OpenDDLParser::parseIdentifier( in, end, &id ); -#ifdef DEBUG_HEADER_NAME +#ifdef DEBUG_HEADER_NAME dumpId( id ); #endif // DEBUG_HEADER_NAME @@ -260,9 +274,9 @@ char *OpenDDLParser::parseHeader( char *in, char *end ) { if( *in != Grammar::CommaSeparator[ 0 ] && *in != Grammar::ClosePropertyToken[ 0 ] ) { logInvalidTokenError( in, Grammar::ClosePropertyToken, m_logCallback ); - return in; + return ddl_nullptr; } - + if( ddl_nullptr != prop && *in != Grammar::CommaSeparator[ 0 ] ) { if( ddl_nullptr == first ) { first = prop; @@ -311,16 +325,19 @@ char *OpenDDLParser::parseStructure( char *in, char *end ) { // loop over all children ( data and nodes ) do { in = parseStructureBody( in, end, error ); + if(in == ddl_nullptr){ + return ddl_nullptr; + } } while ( *in != '}' ); in++; } else { in++; logInvalidTokenError( in, std::string( Grammar::OpenBracketToken ), m_logCallback ); error = true; - return in; + return ddl_nullptr; } in = lookForNextToken( in, end ); - + // pop node from stack after successful parsing if( !error ) { popNode(); @@ -375,7 +392,7 @@ char *OpenDDLParser::parseStructureBody( char *in, char *end, bool &error ) { setNodeValues( top(), values ); setNodeReferences( top(), refs ); } else if( arrayLen > 1 ) { - in = parseDataArrayList( in, end, &dtArrayList ); + in = parseDataArrayList( in, end, type, &dtArrayList ); setNodeDataArrayList( top(), dtArrayList ); } else { std::cerr << "0 for array is invalid." << std::endl; @@ -386,6 +403,7 @@ char *OpenDDLParser::parseStructureBody( char *in, char *end, bool &error ) { in = lookForNextToken( in, end ); if( *in != '}' ) { logInvalidTokenError( in, std::string( Grammar::CloseBracketToken ), m_logCallback ); + return ddl_nullptr; } else { //in++; } @@ -420,7 +438,7 @@ DDLNode *OpenDDLParser::top() { if( m_stack.empty() ) { return ddl_nullptr; } - + DDLNode *top( m_stack.back() ); return top; } @@ -479,7 +497,7 @@ char *OpenDDLParser::parseName( char *in, char *end, Name **name ) { if( *in == '%' ) { ntype = LocalName; } - + in++; Name *currentName( ddl_nullptr ); Identifier *id( ddl_nullptr ); in = parseIdentifier( in, end, &id ); @@ -489,7 +507,7 @@ char *OpenDDLParser::parseName( char *in, char *end, Name **name ) { *name = currentName; } } - + return in; } @@ -501,7 +519,7 @@ char *OpenDDLParser::parseIdentifier( char *in, char *end, Identifier **id ) { // ignore blanks in = lookForNextToken( in, end ); - + // staring with a number is forbidden if( isNumeric( *in ) ) { return in; @@ -510,11 +528,11 @@ char *OpenDDLParser::parseIdentifier( char *in, char *end, Identifier **id ) { // get size of id size_t idLen( 0 ); char *start( in ); - while( !isSeparator( *in ) && !isNewLine( *in ) && ( in != end ) && *in != Grammar::OpenPropertyToken[ 0 ] && *in != Grammar::ClosePropertyToken[ 0 ] ) { + while( !isSeparator( *in ) && !isNewLine( *in ) && ( in != end ) && *in != Grammar::OpenPropertyToken[ 0 ] && *in != Grammar::ClosePropertyToken[ 0 ] && *in != '$' ) { ++in; ++idLen; } - + const size_t len( idLen ); Identifier *newId = new Identifier( start, len ); *id = newId; @@ -631,7 +649,7 @@ char *OpenDDLParser::parseIntegerLiteral( char *in, char *end, Value **integer, return in; } - if( !isIntegerType( integerType ) ) { + if( !(isIntegerType( integerType ) || isUnsignedIntegerType(integerType)) ) { return in; } @@ -642,7 +660,13 @@ char *OpenDDLParser::parseIntegerLiteral( char *in, char *end, Value **integer, } if( isNumeric( *start ) ) { - const int value( atoi( start ) ); +#ifdef OPENDDL_NO_USE_CPP11 + const int64 value( atol( start ) ); // maybe not really 64bit as atoll is but exists without c++11 + const uint64 uvalue( strtoul( start,ddl_nullptr,10 ) ); +#else + const int64 value( atoll( start ) ); + const uint64 uvalue( strtoull( start,ddl_nullptr,10 ) ); +#endif *integer = ValueAllocator::allocPrimData( integerType ); switch( integerType ) { case Value::ddl_int8: @@ -657,15 +681,27 @@ char *OpenDDLParser::parseIntegerLiteral( char *in, char *end, Value **integer, case Value::ddl_int64: ( *integer )->setInt64( ( int64 ) value ); break; + case Value::ddl_unsigned_int8: + ( *integer )->setUnsignedInt8( (uint8) uvalue ); + break; + case Value::ddl_unsigned_int16: + ( *integer )->setUnsignedInt16( ( uint16 ) uvalue ); + break; + case Value::ddl_unsigned_int32: + ( *integer )->setUnsignedInt32( ( uint32 ) uvalue ); + break; + case Value::ddl_unsigned_int64: + ( *integer )->setUnsignedInt64( ( uint64 ) uvalue ); + break; default: break; } - } + } return in; } -char *OpenDDLParser::parseFloatingLiteral( char *in, char *end, Value **floating ) { +char *OpenDDLParser::parseFloatingLiteral( char *in, char *end, Value **floating, Value::ValueType floatType) { *floating = ddl_nullptr; if( ddl_nullptr == in || in == end ) { return in; @@ -690,9 +726,16 @@ char *OpenDDLParser::parseFloatingLiteral( char *in, char *end, Value **floating } if( ok ) { - const float value( ( float ) atof( start ) ); - *floating = ValueAllocator::allocPrimData( Value::ddl_float ); - ( *floating )->setFloat( value ); + if(floatType == Value::ddl_double) + { + const double value( atof( start ) ); + *floating = ValueAllocator::allocPrimData( Value::ddl_double ); + ( *floating )->setDouble( value ); + } else { + const float value( ( float ) atof( start ) ); + *floating = ValueAllocator::allocPrimData( Value::ddl_float ); + ( *floating )->setFloat( value ); + } } return in; @@ -813,7 +856,7 @@ char *OpenDDLParser::parseProperty( char *in, char *end, Property **prop ) { ( *prop )->m_ref = ref; } } - } + } } return in; @@ -856,12 +899,27 @@ char *OpenDDLParser::parseDataList( char *in, char *end, Value::ValueType type, } } } else { - if (Value::ddl_int32 == type) { - in = parseIntegerLiteral( in, end, ¤t ); - } else if (Value::ddl_float == type) { - in = parseFloatingLiteral( in, end, ¤t ); - } else if (Value::ddl_string == type) { - in = parseStringLiteral( in, end, ¤t ); + switch(type){ + case Value::ddl_int8: + case Value::ddl_int16: + case Value::ddl_int32: + case Value::ddl_int64: + case Value::ddl_unsigned_int8: + case Value::ddl_unsigned_int16: + case Value::ddl_unsigned_int32: + case Value::ddl_unsigned_int64: + in = parseIntegerLiteral( in, end, ¤t, type); + break; + case Value::ddl_half: + case Value::ddl_float: + case Value::ddl_double: + in = parseFloatingLiteral( in, end, ¤t, type); + break; + case Value::ddl_string: + in = parseStringLiteral( in, end, ¤t ); + break; + default: + break; } } @@ -895,7 +953,7 @@ static DataArrayList *createDataArrayList( Value *currentValue, size_t numValues return dataList; } -char *OpenDDLParser::parseDataArrayList( char *in, char *end, DataArrayList **dataList ) { +char *OpenDDLParser::parseDataArrayList( char *in, char *end,Value::ValueType type, DataArrayList **dataList ) { *dataList = ddl_nullptr; if( ddl_nullptr == in || in == end ) { return in; @@ -910,7 +968,7 @@ char *OpenDDLParser::parseDataArrayList( char *in, char *end, DataArrayList **da do { size_t numRefs( 0 ), numValues( 0 ); currentValue = ddl_nullptr; - Value::ValueType type( Value::ddl_none ); + in = parseDataList( in, end, type, ¤tValue, numValues, &refs, numRefs ); if( ddl_nullptr != currentValue ) { if( ddl_nullptr == prev ) { diff --git a/contrib/openddlparser/code/Value.cpp b/contrib/openddlparser/code/Value.cpp index 067deccb9..8dc27817e 100644 --- a/contrib/openddlparser/code/Value.cpp +++ b/contrib/openddlparser/code/Value.cpp @@ -316,6 +316,16 @@ Value *Value::getNext() const { return m_next; } +size_t Value::size(){ + size_t result=1; + Value *n=m_next; + while( n!=ddl_nullptr) { + result++; + n=n->m_next; + } + return result; +} + Value *ValueAllocator::allocPrimData( Value::ValueType type, size_t len ) { if( type == Value::ddl_none || Value::ddl_types_max == type ) { return ddl_nullptr; diff --git a/contrib/openddlparser/include/openddlparser/OpenDDLCommon.h b/contrib/openddlparser/include/openddlparser/OpenDDLCommon.h index 701eb72d6..04007fc74 100644 --- a/contrib/openddlparser/include/openddlparser/OpenDDLCommon.h +++ b/contrib/openddlparser/include/openddlparser/OpenDDLCommon.h @@ -30,7 +30,7 @@ CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. #include #ifndef _WIN32 # include -#endif +#endif #ifdef _MSC_VER # define TAG_DLL_EXPORT __declspec(dllexport) @@ -56,9 +56,15 @@ BEGIN_ODDLPARSER_NS #ifndef OPENDDL_NO_USE_CPP11 // All C++11 constructs # define ddl_nullptr nullptr +# define ddl_override override +# define ddl_final final +# define ddl_no_copy = delete #else // Fall-back for older compilers # define ddl_nullptr NULL +# define ddl_override +# define ddl_final +# define ddl_no_copy #endif // OPENDDL_NO_USE_CPP11 // Forward declarations @@ -92,7 +98,7 @@ typedef uint64_impl uint64; ///< Unsigned integer, 8 byte /// @brief Stores a text. /// -/// A text is stored in a simple character buffer. Texts buffer can be +/// A text is stored in a simple character buffer. Texts buffer can be /// greater than the number of stored characters in them. struct DLL_ODDLPARSER_EXPORT Text { size_t m_capacity; ///< The capacity of the text. @@ -122,8 +128,8 @@ struct DLL_ODDLPARSER_EXPORT Text { bool operator == ( const Text &rhs ) const; private: - Text( const Text & ); - Text &operator = ( const Text & ); + Text( const Text & ) ddl_no_copy; + Text &operator = ( const Text & ) ddl_no_copy; }; /// @brief Stores an OpenDDL-specific identifier type. @@ -142,13 +148,13 @@ struct DLL_ODDLPARSER_EXPORT Identifier { /// @brief The destructor. ~Identifier(); - + /// @brief The compare operator. bool operator == ( const Identifier &rhs ) const; private: - Identifier( const Identifier & ); - Identifier &operator = ( const Identifier & ); + Identifier( const Identifier & ) ddl_no_copy; + Identifier &operator = ( const Identifier & ) ddl_no_copy; }; /// @brief Description of the type of a name. @@ -171,8 +177,8 @@ struct DLL_ODDLPARSER_EXPORT Name { ~Name(); private: - Name( const Name & ); - Name &operator = ( const Name& ); + Name( const Name & ) ddl_no_copy; + Name &operator = ( const Name& ) ddl_no_copy; }; /// @brief Stores a bundle of references. @@ -182,7 +188,7 @@ struct DLL_ODDLPARSER_EXPORT Reference { /// @brief The default constructor. Reference(); - + /// @brief The constructor with an array of ref names. /// @param numrefs [in] The number of ref names. /// @param names [in] The ref names. @@ -192,8 +198,8 @@ struct DLL_ODDLPARSER_EXPORT Reference { ~Reference(); private: - Reference( const Reference & ); - Reference &operator = ( const Reference & ); + Reference( const Reference & ) ddl_no_copy; + Reference &operator = ( const Reference & ) ddl_no_copy; }; /// @brief Stores a property list. @@ -214,8 +220,8 @@ struct DLL_ODDLPARSER_EXPORT Property { ~Property(); private: - Property( const Property & ); - Property &operator = ( const Property & ); + Property( const Property & ) ddl_no_copy; + Property &operator = ( const Property & ) ddl_no_copy; }; /// @brief Stores a data array list. @@ -230,9 +236,12 @@ struct DLL_ODDLPARSER_EXPORT DataArrayList { /// @brief The destructor. ~DataArrayList(); + /// @brief Gets the length of the array + size_t size(); + private: - DataArrayList( const DataArrayList & ); - DataArrayList &operator = ( const DataArrayList & ); + DataArrayList( const DataArrayList & ) ddl_no_copy; + DataArrayList &operator = ( const DataArrayList & ) ddl_no_copy; }; /// @brief Stores the context of a parsed OpenDDL declaration. @@ -249,8 +258,8 @@ struct DLL_ODDLPARSER_EXPORT Context { void clear(); private: - Context( const Context & ); - Context &operator = ( const Context & ); + Context( const Context & ) ddl_no_copy; + Context &operator = ( const Context & ) ddl_no_copy; }; END_ODDLPARSER_NS diff --git a/contrib/openddlparser/include/openddlparser/OpenDDLExport.h b/contrib/openddlparser/include/openddlparser/OpenDDLExport.h index 170da5bfa..4fa347140 100644 --- a/contrib/openddlparser/include/openddlparser/OpenDDLExport.h +++ b/contrib/openddlparser/include/openddlparser/OpenDDLExport.h @@ -27,19 +27,31 @@ CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. BEGIN_ODDLPARSER_NS +//------------------------------------------------------------------------------------------------- +/// @ingroup IOStreamBase +/// @brief This class represents the stream to write out. +//------------------------------------------------------------------------------------------------- +class DLL_ODDLPARSER_EXPORT StreamFormatterBase { +public: + StreamFormatterBase(); + virtual ~StreamFormatterBase(); + virtual std::string format( const std::string &statement ); +}; + //------------------------------------------------------------------------------------------------- /// @ingroup IOStreamBase /// @brief This class represents the stream to write out. //------------------------------------------------------------------------------------------------- class DLL_ODDLPARSER_EXPORT IOStreamBase { public: - IOStreamBase(); + IOStreamBase( StreamFormatterBase *formatter = ddl_nullptr ); virtual ~IOStreamBase(); virtual bool open( const std::string &anme ); virtual bool close(); - virtual void write( const std::string &statement ); + virtual size_t write( const std::string &statement ); private: + StreamFormatterBase *m_formatter; FILE *m_file; }; diff --git a/contrib/openddlparser/include/openddlparser/OpenDDLParser.h b/contrib/openddlparser/include/openddlparser/OpenDDLParser.h index 22b1f7fbb..e3fed893b 100644 --- a/contrib/openddlparser/include/openddlparser/OpenDDLParser.h +++ b/contrib/openddlparser/include/openddlparser/OpenDDLParser.h @@ -128,11 +128,11 @@ public: /// @brief Returns the buffer pointer. /// @return The buffer pointer. const char *getBuffer() const; - + /// @brief Returns the size of the buffer. /// @return The buffer size. size_t getBufferSize() const; - + /// @brief Clears all parser data, including buffer and active context. void clear(); @@ -166,12 +166,12 @@ public: // parser helpers static char *parseReference( char *in, char *end, std::vector &names ); static char *parseBooleanLiteral( char *in, char *end, Value **boolean ); static char *parseIntegerLiteral( char *in, char *end, Value **integer, Value::ValueType integerType = Value::ddl_int32 ); - static char *parseFloatingLiteral( char *in, char *end, Value **floating ); + static char *parseFloatingLiteral( char *in, char *end, Value **floating, Value::ValueType floatType= Value::ddl_float ); static char *parseStringLiteral( char *in, char *end, Value **stringData ); static char *parseHexaLiteral( char *in, char *end, Value **data ); static char *parseProperty( char *in, char *end, Property **prop ); static char *parseDataList( char *in, char *end, Value::ValueType type, Value **data, size_t &numValues, Reference **refs, size_t &numRefs ); - static char *parseDataArrayList( char *in, char *end, DataArrayList **dataList ); + static char *parseDataArrayList( char *in, char *end, Value::ValueType type, DataArrayList **dataList ); static const char *getVersion(); private: diff --git a/contrib/openddlparser/include/openddlparser/OpenDDLParserUtils.h b/contrib/openddlparser/include/openddlparser/OpenDDLParserUtils.h index 8572a37ed..635dfd8f7 100644 --- a/contrib/openddlparser/include/openddlparser/OpenDDLParserUtils.h +++ b/contrib/openddlparser/include/openddlparser/OpenDDLParserUtils.h @@ -26,19 +26,6 @@ CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. BEGIN_ODDLPARSER_NS -template -inline -bool isComment( T *in, T *end ) { - if( *in == '/' ) { - if( in + 1 != end ) { - if( *( in + 1 ) == '/' ) { - return true; - } - } - } - return false; -} - template inline bool isUpperCase( T in ) { @@ -51,8 +38,8 @@ bool isLowerCase( T in ) { return ( in >= 'a' && in <= 'z' ); } -template -inline +template +inline bool isSpace( const T in ) { return ( ' ' == in || '\t' == in ); } @@ -66,7 +53,7 @@ bool isNewLine( const T in ) { template inline bool isSeparator( T in ) { - if( isSpace( in ) || ',' == in || '{' == in || '}' == in || '[' == in ) { + if( isSpace( in ) || ',' == in || '{' == in || '}' == in || '[' == in || '(' == in || ')' == in ) { return true; } return false; @@ -83,7 +70,7 @@ static const unsigned char chartype_table[ 256 ] = { 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, // 96-111 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, // 112-127 - 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, // > 127 + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, // > 127 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, @@ -98,7 +85,7 @@ template inline bool isNumeric( const T in ) { return ( in >= '0' && in <= '9' ); - //return ( chartype_table[in] ); + //return ( chartype_table[in] ); /*if (in >= '0' && in <= '9' ) return true; @@ -108,7 +95,7 @@ bool isNumeric( const T in ) { template inline bool isNotEndOfToken( T *in, T *end ) { - return ( '}' != *in && ',' != *in && !isSpace( *in ) && in != end ); + return ( '}' != *in && ',' != *in && !isSpace( *in ) && ')' != *in && in != end ); } template @@ -238,7 +225,7 @@ int hex2Decimal( char in ) { if( isNumeric( in ) ) { return ( in - 48 ); } - + char hexCodeLower( 'a' ), hexCodeUpper( 'A' ); for( int i = 0; i<16; i++ ) { if( in == hexCodeLower + i || in == hexCodeUpper + i ) { @@ -249,4 +236,24 @@ int hex2Decimal( char in ) { return ErrorHex2Decimal; } +template +inline +bool isComment( T *in, T *end ) { + if ( *in=='/' ) { + if ( in+1!=end ) { + if ( *( in+1 )=='/' ) { + char *drive( ( in+2 ) ); + if ( isUpperCase( *drive )||isLowerCase( *drive )&&*( drive+1 )=='/' ) { + return false; + } else { + return true; + } + } + } + } + + return false; +} + END_ODDLPARSER_NS + diff --git a/contrib/openddlparser/include/openddlparser/Value.h b/contrib/openddlparser/include/openddlparser/Value.h index af5ebe7bf..3a6191391 100644 --- a/contrib/openddlparser/include/openddlparser/Value.h +++ b/contrib/openddlparser/include/openddlparser/Value.h @@ -42,8 +42,8 @@ class DLL_ODDLPARSER_EXPORT Value { public: /// @brief This class implements an iterator through a Value list. - /// - /// When getting a new value you need to know how to iterate through it. The Value::Iterator + /// + /// When getting a new value you need to know how to iterate through it. The Value::Iterator /// will help you here: /// @code /// Value *val = node->getValue(); @@ -73,10 +73,10 @@ public: /// @brief Returns the next item and moves the iterator to it. /// @return The next value, is ddl_nullptr in case of being the last item. Value *getNext(); - + /// @brief The post-increment operator. const Iterator operator++( int ); - + /// @brief The pre-increment operator. Iterator &operator++( ); @@ -109,44 +109,132 @@ public: ddl_unsigned_int16, ///< Unsigned integer type, 16 bytes ddl_unsigned_int32, ///< Unsigned integer type, 32 bytes ddl_unsigned_int64, ///< Unsigned integer type, 64 bytes - ddl_half, - ddl_float, - ddl_double, - ddl_string, - ddl_ref, - ddl_types_max + ddl_half, ///< Half data type. + ddl_float, ///< float data type + ddl_double, ///< Double data type. + ddl_string, ///< String data type. + ddl_ref, ///< Reference, used to define references to other data definitions. + ddl_types_max ///< Upper limit. }; + /// @brief The class constructor. + /// @param type [in] The value type. Value( ValueType type ); + + /// @brief The class destructor. ~Value(); + + /// @brief Assigns a boolean to the value. + /// @param value [in9 The value. void setBool( bool value ); + + /// @brief Returns the boolean value. + /// @return The boolean value. bool getBool(); + + /// @brief Assigns a int8 to the value. + /// @param value [in] The value. void setInt8( int8 value ); + + /// @brief Returns the int8 value. + /// @return The int8 value. int8 getInt8(); + + /// @brief Assigns a int16 to the value. + /// @param value [in] The value. void setInt16( int16 value ); + + /// @brief Returns the int16 value. + /// @return The int16 value. int16 getInt16(); + + /// @brief Assigns a int32 to the value. + /// @param value [in] The value. void setInt32( int32 value ); + + /// @brief Returns the int16 value. + /// @return The int32 value. int32 getInt32(); + + /// @brief Assigns a int64 to the value. + /// @param value [in] The value. void setInt64( int64 value ); + + /// @brief Returns the int16 value. + /// @return The int64 value. int64 getInt64(); + + /// @brief Assigns a unsigned int8 to the value. + /// @param value [in] The value. void setUnsignedInt8( uint8 value ); + + /// @brief Returns the unsigned int8 value. + /// @return The unsigned int8 value. uint8 getUnsignedInt8() const; + + /// @brief Assigns a unsigned int16 to the value. + /// @param value [in] The value. void setUnsignedInt16( uint16 value ); + + /// @brief Returns the unsigned int16 value. + /// @return The unsigned int16 value. uint16 getUnsignedInt16() const; + + /// @brief Assigns a unsigned int32 to the value. + /// @param value [in] The value. void setUnsignedInt32( uint32 value ); + + /// @brief Returns the unsigned int8 value. + /// @return The unsigned int32 value. uint32 getUnsignedInt32() const; + + /// @brief Assigns a unsigned int64 to the value. + /// @param value [in] The value. void setUnsignedInt64( uint64 value ); + + /// @brief Returns the unsigned int64 value. + /// @return The unsigned int64 value. uint64 getUnsignedInt64() const; + + /// @brief Assigns a float to the value. + /// @param value [in] The value. void setFloat( float value ); + + /// @brief Returns the float value. + /// @return The float value. float getFloat() const; + + /// @brief Assigns a double to the value. + /// @param value [in] The value. void setDouble( double value ); + + /// @brief Returns the double value. + /// @return The double value. double getDouble() const; + + /// @brief Assigns a std::string to the value. + /// @param value [in] The value. void setString( const std::string &str ); + + /// @brief Returns the std::string value. + /// @return The std::string value. const char *getString() const; + + /// @brief Dumps the value. void dump(); + + /// @brief Assigns the next value. + /// @param next [n] The next value. void setNext( Value *next ); + + /// @brief Returns the next value. + /// @return The next value.s Value *getNext() const; + /// @brief Gets the length of the array. + /// @return The number of items in the array. + size_t size(); + ValueType m_type; size_t m_size; unsigned char *m_data;