Revision: 58827 http://sourceforge.net/p/brlcad/code/58827 Author: bob1961 Date: 2013-12-05 15:19:09 +0000 (Thu, 05 Dec 2013) Log Message: ----------- This reverts r50778. rt_memfree() needs to continue in order to initialize *pp, otherwise curp and prevp will always be NULL. The short of it is that if we return early when (\!prevp || \!curp), the free memory nodes will never coalesce which results in the file size growing unnecessarily because it's being filled with contiguous free regions that are always seen seperately.
Revision Links: -------------- http://sourceforge.net/p/brlcad/code/50778 Modified Paths: -------------- brlcad/trunk/src/librt/memalloc.c Modified: brlcad/trunk/src/librt/memalloc.c =================================================================== --- brlcad/trunk/src/librt/memalloc.c 2013-12-05 14:16:20 UTC (rev 58826) +++ brlcad/trunk/src/librt/memalloc.c 2013-12-05 15:19:09 UTC (rev 58827) @@ -271,10 +271,6 @@ return; } - /* nothing to do */ - if (!prevp || !curp) - return; - /* * Now we do the surgery: * If there are no matches on boundaries we allocate a buffer This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. ------------------------------------------------------------------------------ Sponsored by Intel(R) XDK Develop, test and display web and hybrid apps with a single code base. Download it for free now! http://pubads.g.doubleclick.net/gampad/clk?id=111408631&iu=/4140/ostg.clktrk _______________________________________________ BRL-CAD Source Commits mailing list brlcad-commits@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/brlcad-commits