[Bf-committers] [Bf-blender-cvs] SVN commit: /data/svn/bf-blender [40677] trunk/blender/source/blender/ editors/animation/anim_filter.c: Reverting part of r.40659

Matt Ebb matt at mke3.net
Thu Sep 29 02:09:10 CEST 2011


On Thu, Sep 29, 2011 at 9:43 AM, Joshua Leung <aligorith at gmail.com> wrote:

> Reverting part of r.40659
>
> The output of an automated tool is not a valid excuse for clobbering
> code to increase maintenance headaches later on.
>

It might be a good idea to reiterate for new developers: Please avoid
committing in other peoples' code without the permission or understanding of
the module owner. There have been a couple of occasions in the last week or
so where people have done seemingly innocuous fixes in other code which have
ended up causing problems or bugs.

If you've found something which you think should be fixed and it's not 100%
urgent (like its breaking compilation), just ask the owner of the code
first, or make a patch. Just because you have access to commit everywhere in
blender doesn't make it always a good idea :)

cheers

Matt


More information about the Bf-committers mailing list