new [] / delete mismatch

cgp1024

30-11-2009 17:44:40

looks to me like ~TheoraVideoFrame should use "delete []", not "delete"

Kreso

24-12-2009 11:00:14

whoah! good point, thanks :)

Cougar

07-08-2010 15:51:47

There's a similar mismatch in the demos, in ~OpenAL_AudioInterface().


if (mTempBuffer) delete mTempBuffer;


should be


if (mTempBuffer) delete [] mTempBuffer;


I know it's just a sample, but I believe many people will end up using this particular sample class in their projects :)

Kreso

07-08-2010 23:21:18

heh, good point :)