Developer forums (C::B DEVELOPMENT STRICTLY!) > Development

HUGE MEMORY LEAK pinpointed!

<< < (2/21) > >>

rickg22:
Everybody update their SVN's!

Fix posted at revision 4120!

jpaterso:
Should it be delete [] instead of delete?

TDragon:

--- Quote from: jpaterso on June 20, 2007, 03:48:53 pm ---Should it be delete [] instead of delete?

--- End quote ---
YES.

Jan van den Borst:
Typing in delete by hand is almost always bad practice. Use f.e. auto_ptr.

byo:

--- Quote from: TDragon on June 20, 2007, 03:54:34 pm ---
--- Quote from: jpaterso on June 20, 2007, 03:48:53 pm ---Should it be delete [] instead of delete?

--- End quote ---
YES.

--- End quote ---

Yup, should be called. Fortunately in this particular case it will work with normal delete since there's no destructor called (binary code will probably look the same after switching to delete[]). So there's no need to panic ;).

Ok, now it's time to build the new c::B. Thanks rickg22 for the fix :)

BYO

Navigation

[0] Message Index

[#] Next page

[*] Previous page