delete of NULL pointer is noop, so no need to check
Closed, InvalidPublic


In C++, it is unnecessary to check for non-NULL before invoking delete:

if (foo) delete foo;

can be more simply written as

delete foo;

(Ref: section 5.3.5 of the C++98 spec, paragraph 2.)

This patch just fixes some initial cases I came across; no doubt there are others waiting to be found.


Lawrence D'Oliveiro (ldo) updated the task description. (Show Details)
Lawrence D'Oliveiro (ldo) raised the priority of this task from to Needs Triage.
Lawrence D'Oliveiro (ldo) set Type to Bug.

Sorry, I should have marked this as a patch, not a bug.

Bastien Montagne (mont29) claimed this task.

Thanks for the patch, be we for sure won’t touch to extern library's code - not until it’s a question of live or death for Blender! :P

Suggest you report the issue to carve team.

I see the header files refer to, which no longer exists.

So where is this “carve team” to be found?