[Bf-blender-cvs] CVS commit: blender/source/blender/blenlib BLI_arithb.h blender/source/blender/blenlib/intern arithb.c

John K. Walton walton at mrnutty.com
Wed Jul 2 22:41:22 CEST 2003


I just want to make a quick editorial note here:
I really appreciate the way Ton does his commit
log. He includes a brief/concise explanation of
what he did, followed by a reason or rationalization
for the change. So, even without looking at the change
i can get an appreciation for what was done, and why.

thanks Ton
John

On Wed, 2 Jul 2003, Ton Roosendaal wrote:

> ton (Ton Roosendaal) 2003/07/02 15:37:04 CEST
> 
>   Modified files:
>     blender/source/blender/blenlib BLI_arithb.h 
>     blender/source/blender/blenlib/intern arithb.c 
>   
>   Log:
>   - removed the 'const' from arithb function declarations. it caused
>     the myriad of warnings people complained about so long.
>   
>     after careful consulting & reading I could not find a good reason
>     for this const stuff, apart from a sort-of comment to indicate
>     this array (matrix) pointer cannot be changed. Well, doh! you
>     should not do that with a float mat[][3] anyway!
>   
>   Revision  Changes    Path
>   1.6       +128 -128  blender/source/blender/blenlib/BLI_arithb.h
>   1.5       +89 -90    blender/source/blender/blenlib/intern/arithb.c
> _______________________________________________
> Bf-blender-cvs mailing list
> Bf-blender-cvs at blender.org
> http://www.blender.org/mailman/listinfo/bf-blender-cvs
> 




More information about the Bf-blender-cvs mailing list