[sword-devel] swld.cpp at line 111 - is this a bug?

[Jaak Ristioja]

On 12.10.2016 09:37, Manfred Bergmann wrote:
>> On 12.10.2016 01:08, TS wrote:
>>> Xcode was warning me that in PocketSword's Sword framework, swld.cpp at
>>> line 111 with
>>>
>>> "delete buf;"
>>> may be wrong and it should perhaps be
>>>
>>> "delete[] buf;"
>>> instead. Is this correct? I looked at the code and I think Xcode is
>>> correct in that the variable is an array and needs the brackets to
>>> delete properly.
>>
>> Yes, Xcode has it correct. It should be `delete[] buf;`. This was also
>> fixed in Sword++ in March.
> 
> Despite the warning, would but not be deleted completely?

It is undefined behavior, as stated by §3.7.4.2.3 in C++11. However, the
C++ draft at https://github.com/cplusplus/draft doesn't have this
wording since e51a2152 [1] for some reason.

J


[1]:
https://github.com/cplusplus/draft/commit/e51a2152599bce93d72b371590d03960f99f85e3