Code cleanup: remove redundant NULL check
authorCampbell Barton <ideasman42@gmail.com>
Thu, 29 May 2014 02:33:13 +0000 (12:33 +1000)
committerCampbell Barton <ideasman42@gmail.com>
Fri, 13 Jun 2014 14:47:11 +0000 (00:47 +1000)
source/blender/editors/interface/interface_templates.c

index dd951e48784e197087d4260b61eb3f5957191614..70791eaebc4c0207b2b010583324cb497360f0df 100644 (file)
@@ -418,11 +418,9 @@ static void template_ID(bContext *C, uiLayout *layout, TemplateID *template, Str
 
                but = uiDefBlockButN(block, id_search_menu, MEM_dupallocN(template), "", 0, 0, UI_UNIT_X * 6, UI_UNIT_Y * 6,
                                     TIP_(template_id_browse_tip(type)));
-               if (type) {
-                       but->icon = RNA_struct_ui_icon(type);
-                       if (id) but->icon = ui_id_icon_get(C, id, true);
-                       uiButSetFlag(but, UI_HAS_ICON | UI_ICON_PREVIEW);
-               }
+               but->icon = id ? ui_id_icon_get(C, id, true) : RNA_struct_ui_icon(type);
+               uiButSetFlag(but, UI_HAS_ICON | UI_ICON_PREVIEW);
+
                if ((idfrom && idfrom->lib) || !editable)
                        uiButSetFlag(but, UI_BUT_DISABLED);
                
@@ -431,14 +429,11 @@ static void template_ID(bContext *C, uiLayout *layout, TemplateID *template, Str
        else if (flag & UI_ID_BROWSE) {
                but = uiDefBlockButN(block, id_search_menu, MEM_dupallocN(template), "", 0, 0, UI_UNIT_X * 1.6, UI_UNIT_Y,
                                     TIP_(template_id_browse_tip(type)));
-
-               if (type) {
-                       but->icon = RNA_struct_ui_icon(type);
-                       /* default dragging of icon for id browse buttons */
-                       uiButSetDragID(but, id);
-                       uiButSetFlag(but, UI_HAS_ICON);
-                       uiButSetDrawFlag(but, UI_BUT_ICON_LEFT);
-               }
+               but->icon = RNA_struct_ui_icon(type);
+               /* default dragging of icon for id browse buttons */
+               uiButSetDragID(but, id);
+               uiButSetFlag(but, UI_HAS_ICON);
+               uiButSetDrawFlag(but, UI_BUT_ICON_LEFT);
 
                if ((idfrom && idfrom->lib) || !editable)
                        uiButSetFlag(but, UI_BUT_DISABLED);