[Bf-committers] [Bf-blender-cvs] SVN commit: /data/svn/bf-blender [34852] trunk/blender/source/blender: This commit will switch blender to use tangent space generated within

[Daniel Stokes]

There are already some files in blenkernel that use their own math
functions, or at least implicit.c does. I am pretty sure many of the math
functions in that file can be handled by BLI_math.

-- Daniel Stokes

On Mon, Feb 14, 2011 at 10:12 PM, Dan Eicher <dan at trollwerks.org> wrote:

> On Mon, Feb 14, 2011 at 10:30 PM, Campbell Barton <ideasman42 at gmail.com>
> wrote:
> > I can see how its nice to have standalone C files which can be
> > re-used, and in this case its not a big problem on its own but I don't
> > think its a good precedent either.
> > If we had 3+ files in blenlib/blenkernel which defined their own
> > vector math functions I think it could become a maintenance problem
> > and reduce readability across blenders own code.
> >
> > In this case its not worth having mikktspace.c be its own library in
> > extern so how about have a version maintained externally which
> > includes blenders BLI_math functions?, the blender version can include
> > a URL to this file so others can use it in their own projects without
> > blender.
> >
> Or the initial check-in could be used as the reference implementation
> and all blender specific changes are made on top of that... kind of
> what SCM was designed for methinks.
>
> Dan
> _______________________________________________
> Bf-committers mailing list
> Bf-committers at blender.org
> http://lists.blender.org/mailman/listinfo/bf-committers
>