From be93eb845d8a6eca64629717a149d3da48ec7a46 Mon Sep 17 00:00:00 2001 From: Glenn Maynard Date: Fri, 26 Nov 2004 20:41:13 +0000 Subject: [PATCH] scope/style cleanup if ReadData returns -1, be sure to not append(-1) --- stepmania/src/ScreenPackages.cpp | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/stepmania/src/ScreenPackages.cpp b/stepmania/src/ScreenPackages.cpp index 8929858958..b7d5c665fc 100644 --- a/stepmania/src/ScreenPackages.cpp +++ b/stepmania/src/ScreenPackages.cpp @@ -580,14 +580,17 @@ void ScreenPackages::HTTPUpdate() //as there may be need to "if" it out some time. /* If you need a conditional for a large block of code, stick it in * a function and return. */ - int Size = 1; - while( Size > 0 ) + while(1) { char Buffer[1024]; - Size = m_wSocket.ReadData( Buffer, 1024 ); - m_sBUFFER.append( Buffer, Size ); - BytesGot += Size; + int iSize = m_wSocket.ReadData( Buffer, 1024 ); + if( iSize <= 0 ) + break; + + m_sBUFFER.append( Buffer, iSize ); + BytesGot += iSize; } + if( !m_bGotHeader ) { m_sStatus = "Waiting for header.";