Singularity-Viewer/SingularityViewer
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Aleric is right. This makes no sense. Don't bring this from v2, cuz it's wrong.

Browse Source
This commit is contained in:
Shyotl
2011-07-12 15:35:35 -05:00
parent f93bcae65b
commit 4f8f24921f
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
indra/llmessage/lldatapacker.h
View File

@@ -179,10 +179,12 @@ public:
void freeBuffer() { delete [] mBufferp; mBufferp = mCurBufferp = NULL; mBufferSize = 0; mWriteEnabled = FALSE; } void freeBuffer() { delete [] mBufferp; mBufferp = mCurBufferp = NULL; mBufferSize = 0; mWriteEnabled = FALSE; }
void assignBuffer(U8 *bufferp, S32 size) void assignBuffer(U8 *bufferp, S32 size)
{ {
if(mBufferp && mBufferp != bufferp) //No no no no no! This breaks the paradigm of callers handling buffer allocation/deallocation
//Also, buffers can be on stack! Calling delete[] on such a buffer would be VERY bad.
/*if(mBufferp && mBufferp != bufferp)
{ {
freeBuffer() ; freeBuffer() ;
} }*/
mBufferp = bufferp; mBufferp = bufferp;
mCurBufferp = bufferp; mCurBufferp = bufferp;
mBufferSize = size; mBufferSize = size;