[Bf-committers] [Bf-blender-cvs] SVN commit: /data/svn/bf-blender [56936] trunk/blender: remove return value from MEM_freeN, it wasn' t used anywhere and was cast to a different function signature.

[Campbell Barton]

Yep, compiled and checked it didn't crash or cause problems for basic usage.
Unfortunately I had WITH_GUARDEDALLOC off (from testing errors with
address sanitizer), so it didn't error out here,
I see Brecht's resolved the issue.

On Tue, May 21, 2013 at 7:03 PM, Sergey Sharybin <sergey.vfx at gmail.com> wrote:
> No idea how it was harmful, but did you ever try compile blender after such
> a cleanup? ;) There're lots of "error: ‘return’ with a value, in function
> returning void" errors in MEM_freeN.
>
>
> On Tue, May 21, 2013 at 1:37 PM, Campbell Barton <ideasman42 at gmail.com>wrote:
>
>> Revision: 56936
>>
>> http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=56936
>> Author:   campbellbarton
>> Date:     2013-05-21 07:37:59 +0000 (Tue, 21 May 2013)
>> Log Message:
>> -----------
>> remove return value from MEM_freeN, it wasn't used anywhere and was cast
>> to a different function signature. (which evidently works but error prone).
>>
>> Modified Paths: