diff options
author | Dalai Felinto <dfelinto@gmail.com> | 2018-10-26 19:45:35 +0300 |
---|---|---|
committer | Dalai Felinto <dfelinto@gmail.com> | 2018-10-26 19:48:14 +0300 |
commit | b6a795f28586f436cf01e8a348f998d6b3dfaf4d (patch) | |
tree | 2129af13703bdf69513482a843cc9f75350020f3 /source/blender/blenkernel/intern/object_deform.c | |
parent | f1974bb34036b846da989efb870d7ccfa1ada209 (diff) |
Small cleanup: Remove uneeded/wrong check
If ob was NULL it would crash in the else part of the if statement.
If we really think we may run into that (which we should not) we can just assert
or add a if (ob == NULL) return; in the top of the function.
Diffstat (limited to 'source/blender/blenkernel/intern/object_deform.c')
-rw-r--r-- | source/blender/blenkernel/intern/object_deform.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/source/blender/blenkernel/intern/object_deform.c b/source/blender/blenkernel/intern/object_deform.c index e051c57ac31..70569bfd51b 100644 --- a/source/blender/blenkernel/intern/object_deform.c +++ b/source/blender/blenkernel/intern/object_deform.c @@ -403,7 +403,7 @@ static void object_defgroup_remove_edit_mode(Object *ob, bDeformGroup *dg) */ void BKE_object_defgroup_remove(Object *ob, bDeformGroup *defgroup) { - if ((ob) && (ob->type == OB_GPENCIL)) { + if (ob->type == OB_GPENCIL) { BKE_gpencil_vgroup_remove(ob, defgroup); } else { |