Hi, This patch and the other one you posted contain many formatting changes, please keep those out of patches, it's hard to see what you changed.
Regarding this fix, it would be better to store original roll/head/tail + loc/rot/scale on entering edit mode in the EditBone, and then use that when you convert back from EditBone to Bone, if the roll/head/tail didn't change (maybe need to check for parent changes too). That keeps it nice localized. Better not store duplicate state in files, not because of memory usage, but because it's better to keep such hacks in runtime code. Brecht. On Tue, Feb 1, 2011 at 6:36 PM, Stephen Swaney <sswa...@centurytel.net> wrote: > On Tue, Feb 01, 2011 at 06:04:59PM +0100, Tobias Oelgarte wrote: >> >> The patch: >> * http://pastebin.com/mV5BwHqW > > Blender has a Patch Tracker. Putting patches in our tracker > makes it easier to review and maintain them. > > Making it easier for patch reviewers is making it easier to > get your patch applied. > > -- > Stephen Swaney > sswa...@centurytel.net > > _______________________________________________ > Bf-committers mailing list > Bf-committers@blender.org > http://lists.blender.org/mailman/listinfo/bf-committers > _______________________________________________ Bf-committers mailing list Bf-committers@blender.org http://lists.blender.org/mailman/listinfo/bf-committers
