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

_______________________________________________
sword-devel mailing list: sword-devel@crosswire.org
http://www.crosswire.org/mailman/listinfo/sword-devel
Instructions to unsubscribe/change your settings at above page

Reply via email to