[Bf-committers] Re: [Bf-blender-cvs] CVS commit: blender/source/blender/blenkernel/intern font.c softbody.c blender/source/blender/python/api2_2x Mathutils.c NMesh.c blender/source/blender/src editscreen.c

Stephen Swaney sswaney at centurytel.net
Fri Oct 28 18:15:28 CEST 2005


On Fri, Oct 28, 2005 at 07:30:38AM -0700, Ken Hughes wrote:
> 
> EXPP_ReturnPyObjError() returns a NULL, so
> 
>     return EXPP_ReturnPyObjError(...);
> 
> is the same as
> 
>     EXPP_ReturnPyObjError(...);
>     return NULL;

The net result for both is the same but IMHO  

  return EXPP_ReturnPyObjError(...);

is a little clearer.

Since this form is used all over the Bpy code, it
should be the preferred idiom.

Note: in the best of all possible worlds,
this would be a macro that hides the return() like 
the new  Py_RETURN_NONE.
Just one more example of old legacy stuff!

-- 
Stephen Swaney			
sswaney at centurytel.net



More information about the Bf-committers mailing list