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 <[email protected]> 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
> [email protected]
> http://lists.blender.org/mailman/listinfo/bf-blender-cvs
>



-- 
- Campbell
_______________________________________________
Bf-committers mailing list
[email protected]
http://lists.blender.org/mailman/listinfo/bf-committers

Reply via email to