From b6760f5612ff438bfd97bf0616b938f1089ada77 Mon Sep 17 00:00:00 2001
From: discomfitor <discomfitor@7cbeb6ba-43b4-40fd-8cce-4c39aea84d33>
Date: Mon, 3 Oct 2011 02:25:30 +0000
Subject: [PATCH] fix fixme in elm_toolbar_item_menu_get: no longer modifies a
 toolbar item

git-svn-id: svn+ssh://svn.enlightenment.org/var/svn/e/trunk/elementary@63756 7cbeb6ba-43b4-40fd-8cce-4c39aea84d33
---
 src/lib/Elementary.h.in | 2 +-
 src/lib/elm_toolbar.c   | 7 ++-----
 2 files changed, 3 insertions(+), 6 deletions(-)

diff --git a/src/lib/Elementary.h.in b/src/lib/Elementary.h.in
index 2653a9e..d37dc4d 100644
--- a/src/lib/Elementary.h.in
+++ b/src/lib/Elementary.h.in
@@ -14847,7 +14847,7 @@ extern "C" {
     *
     * @ingroup Toolbar
     */
-   EAPI Evas_Object            *elm_toolbar_item_menu_get(Elm_Toolbar_Item *item) EINA_ARG_NONNULL(1);
+   EAPI Evas_Object            *elm_toolbar_item_menu_get(const Elm_Toolbar_Item *item) EINA_ARG_NONNULL(1);
 
    /**
     * Add a new state to @p item.
diff --git a/src/lib/elm_toolbar.c b/src/lib/elm_toolbar.c
index 5feae35..8f9e524 100644
--- a/src/lib/elm_toolbar.c
+++ b/src/lib/elm_toolbar.c
@@ -1755,14 +1755,11 @@ elm_toolbar_item_cursor_engine_only_get(const Elm_Toolbar_Item *item)
 }
 
 EAPI Evas_Object *
-elm_toolbar_item_menu_get(Elm_Toolbar_Item *item)
+elm_toolbar_item_menu_get(const Elm_Toolbar_Item *item)
 {
    ELM_WIDGET_ITEM_WIDTYPE_CHECK_OR_RETURN(item, NULL);
    Widget_Data *wd = elm_widget_data_get(item->base.widget);
-   if (!wd) return NULL;
-   /* FIXME: It's not ok. This function needs to be reviewed. And should
-    * receive a const item */
-   elm_toolbar_item_menu_set(item, 1);
+   if ((!wd) || (!item->menu)) return NULL;
    return item->o_menu;
 }
 
-- 
2.7.4