Browse Source

tier1: fix commandbuffer unitialized boolean

pull/114/head
nillerusr 2 years ago
parent
commit
45d33c6b97
  1. 1
      public/tier1/CommandBuffer.h
  2. 3
      tier1/commandbuffer.cpp
  3. 4
      tier1/utlstring.cpp

1
public/tier1/CommandBuffer.h

@ -119,7 +119,6 @@ private:
bool ParseArgV0( CUtlBuffer &buf, char *pArgv0, int nMaxLen, const char **pArgs ); bool ParseArgV0( CUtlBuffer &buf, char *pArgv0, int nMaxLen, const char **pArgs );
char m_pArgSBuffer[ ARGS_BUFFER_LENGTH ]; char m_pArgSBuffer[ ARGS_BUFFER_LENGTH ];
int m_nLastUsedArgSSize;
int m_nArgSBufferSize; int m_nArgSBufferSize;
CUtlFixedLinkedList< Command_t > m_Commands; CUtlFixedLinkedList< Command_t > m_Commands;
int m_nCurrentTick; int m_nCurrentTick;

3
tier1/commandbuffer.cpp

@ -37,6 +37,7 @@ CCommandBuffer::CCommandBuffer( ) : m_Commands( 32, 32 )
m_nArgSBufferSize = 0; m_nArgSBufferSize = 0;
m_bIsProcessingCommands = false; m_bIsProcessingCommands = false;
m_nMaxArgSBufferLength = ARGS_BUFFER_LENGTH; m_nMaxArgSBufferLength = ARGS_BUFFER_LENGTH;
m_bWaitEnabled = true;
} }
CCommandBuffer::~CCommandBuffer() CCommandBuffer::~CCommandBuffer()
@ -395,10 +396,8 @@ void CCommandBuffer::EndProcessingCommands()
break; break;
AssertMsgOnce( false, "CCommandBuffer::EndProcessingCommands() called before all appropriate commands were dequeued.\n" ); AssertMsgOnce( false, "CCommandBuffer::EndProcessingCommands() called before all appropriate commands were dequeued.\n" );
int nNext = i;
Msg( "Warning: Skipping command %s\n", &m_pArgSBuffer[ m_Commands[i].m_nFirstArgS ] ); Msg( "Warning: Skipping command %s\n", &m_pArgSBuffer[ m_Commands[i].m_nFirstArgS ] );
m_Commands.Remove( i ); m_Commands.Remove( i );
i = nNext;
} }
Compact(); Compact();

4
tier1/utlstring.cpp

@ -426,7 +426,7 @@ void CUtlString::TrimLeft( char cTarget )
// We have some whitespace to remove // We have some whitespace to remove
if ( nIndex > 0 ) if ( nIndex > 0 )
{ {
memcpy( m_pString, &m_pString[nIndex], Length() - nIndex ); memmove( m_pString, &m_pString[nIndex], Length() - nIndex );
SetLength( Length() - nIndex ); SetLength( Length() - nIndex );
} }
} }
@ -463,7 +463,7 @@ void CUtlString::TrimLeft( const char *szTargets )
// We have some whitespace to remove // We have some whitespace to remove
if ( i > 0 ) if ( i > 0 )
{ {
memcpy( m_pString, &m_pString[i], Length() - i ); memmove( m_pString, &m_pString[i], Length() - i );
SetLength( Length() - i ); SetLength( Length() - i );
} }
} }

Loading…
Cancel
Save