[MOAB-dev] [Itaps-users] [PATCH 1/2] iMesh_MOAB: do not allocate new memory for count=0 items

Jed Brown jed at 59A2.org
Fri Jun 10 14:08:30 CDT 2011


On Fri, Jun 10, 2011 at 21:06, Mark Miller <miller86 at llnl.gov> wrote:

> // Macro to free an iMesh 'array'
> #define IMESH_AFREE(ON)                 \
>    if (ON) free(ON);                   \
>

This if statement is not necessary.


Anyone who uses your macros or any similar system is unaffected by my
proposed change.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mcs.anl.gov/pipermail/moab-dev/attachments/20110610/ca4c2aa8/attachment-0001.htm>


More information about the moab-dev mailing list