[Bf-committers] [Bf-blender-cvs] SVN commit: /data/svn/bf-blender [27951] trunk/blender/source/blender/ editors/object/object_edit.c: Simple crasher fix: TAB to change mode crashed after moving all the scene' s objects to a hidden layer.

[Campbell Barton]

a better fix would be to use ED_object_active_context() rather then
CTX_data_active_object(), since the poll function now uses
ED_object_active_context.

Matt, could you check that all ops that use
ED_operator_object_active_editable also get the object using
ED_object_active_context, otherwise we're likely to get similar
problems elsewhere.

On Fri, Apr 2, 2010 at 12:07 PM, Joshua Leung <aligorith at gmail.com> wrote:
> Revision: 27951
>          http://projects.blender.org/plugins/scmsvn/viewcvs.php?view=rev&root=bf-blender&revision=27951
> Author:   aligorith
> Date:     2010-04-02 12:07:31 +0200 (Fri, 02 Apr 2010)
>
> Log Message:
> -----------
> Simple crasher fix: TAB to change mode crashed after moving all the scene's objects to a hidden layer.
>
> Modified Paths:
> --------------
>    trunk/blender/source/blender/editors/object/object_edit.c
>
> Modified: trunk/blender/source/blender/editors/object/object_edit.c
> ===================================================================
> --- trunk/blender/source/blender/editors/object/object_edit.c   2010-04-02 09:23:54 UTC (rev 27950)
> +++ trunk/blender/source/blender/editors/object/object_edit.c   2010-04-02 10:07:31 UTC (rev 27951)
> @@ -1987,7 +1987,7 @@
>  {
>        Object *ob= CTX_data_active_object(C);
>        ObjectMode mode = RNA_enum_get(op->ptr, "mode");
> -       ObjectMode restore_mode = ob->mode;
> +       ObjectMode restore_mode = (ob) ? ob->mode : OB_MODE_OBJECT;
>        int toggle = RNA_boolean_get(op->ptr, "toggle");
>
>        if(!ob || !object_mode_set_compat(C, op, ob))
>
>
> _______________________________________________
> Bf-blender-cvs mailing list
> Bf-blender-cvs at blender.org
> http://lists.blender.org/mailman/listinfo/bf-blender-cvs
>



-- 
- Campbell