forked from enlightenment/efl
elm_gengrid/genlist.c: removed logically dead code. NULL is already checked.
This fixes 'Logically dead code' warning which was spotted by coverity. CID 1040038, 1040040, 1040036, 1040037.
This commit is contained in:
parent
3be4bf95ce
commit
14c7bf8039
|
@ -3993,9 +3993,7 @@ elm_gengrid_item_select_mode_set(Elm_Object_Item *item,
|
||||||
Elm_Object_Select_Mode mode)
|
Elm_Object_Select_Mode mode)
|
||||||
{
|
{
|
||||||
Elm_Gen_Item *it = (Elm_Gen_Item *)item;
|
Elm_Gen_Item *it = (Elm_Gen_Item *)item;
|
||||||
|
|
||||||
ELM_GENGRID_ITEM_CHECK_OR_RETURN(it);
|
ELM_GENGRID_ITEM_CHECK_OR_RETURN(it);
|
||||||
if (!it) return;
|
|
||||||
|
|
||||||
if (it->generation < GG_IT(it)->wsd->generation) return;
|
if (it->generation < GG_IT(it)->wsd->generation) return;
|
||||||
if (mode >= ELM_OBJECT_SELECT_MODE_MAX)
|
if (mode >= ELM_OBJECT_SELECT_MODE_MAX)
|
||||||
|
@ -4012,10 +4010,8 @@ EAPI Elm_Object_Select_Mode
|
||||||
elm_gengrid_item_select_mode_get(const Elm_Object_Item *item)
|
elm_gengrid_item_select_mode_get(const Elm_Object_Item *item)
|
||||||
{
|
{
|
||||||
Elm_Gen_Item *it = (Elm_Gen_Item *)item;
|
Elm_Gen_Item *it = (Elm_Gen_Item *)item;
|
||||||
|
|
||||||
ELM_GENGRID_ITEM_CHECK_OR_RETURN(it, ELM_OBJECT_SELECT_MODE_MAX);
|
ELM_GENGRID_ITEM_CHECK_OR_RETURN(it, ELM_OBJECT_SELECT_MODE_MAX);
|
||||||
|
|
||||||
if (!it) return ELM_OBJECT_SELECT_MODE_MAX;
|
|
||||||
return it->select_mode;
|
return it->select_mode;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -7308,7 +7308,6 @@ elm_genlist_item_select_mode_set(Elm_Object_Item *item,
|
||||||
Elm_Gen_Item *it = (Elm_Gen_Item *)item;
|
Elm_Gen_Item *it = (Elm_Gen_Item *)item;
|
||||||
|
|
||||||
ELM_GENLIST_ITEM_CHECK_OR_RETURN(item);
|
ELM_GENLIST_ITEM_CHECK_OR_RETURN(item);
|
||||||
if (!it) return;
|
|
||||||
ELM_GENLIST_DATA_GET_FROM_ITEM(it, sd);
|
ELM_GENLIST_DATA_GET_FROM_ITEM(it, sd);
|
||||||
|
|
||||||
if (it->generation < sd->generation) return;
|
if (it->generation < sd->generation) return;
|
||||||
|
@ -7340,11 +7339,8 @@ EAPI Elm_Object_Select_Mode
|
||||||
elm_genlist_item_select_mode_get(const Elm_Object_Item *item)
|
elm_genlist_item_select_mode_get(const Elm_Object_Item *item)
|
||||||
{
|
{
|
||||||
Elm_Gen_Item *it = (Elm_Gen_Item *)item;
|
Elm_Gen_Item *it = (Elm_Gen_Item *)item;
|
||||||
|
|
||||||
ELM_GENLIST_ITEM_CHECK_OR_RETURN(item, ELM_OBJECT_SELECT_MODE_MAX);
|
ELM_GENLIST_ITEM_CHECK_OR_RETURN(item, ELM_OBJECT_SELECT_MODE_MAX);
|
||||||
|
|
||||||
if (!it) return ELM_OBJECT_SELECT_MODE_MAX;
|
|
||||||
|
|
||||||
return it->select_mode;
|
return it->select_mode;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue