rimmed pushed a commit to branch master.

http://git.enlightenment.org/tools/eflete.git/commit/?id=2c762cc57ed6e4ec278f4baa42f496ec0b34f2f5

commit 2c762cc57ed6e4ec278f4baa42f496ec0b34f2f5
Author: Vyacheslav Reutskiy <[email protected]>
Date:   Mon May 30 17:58:51 2016 +0300

    project_close: don't allocate title string if project not changed
    
    Change-Id: I3893237a9bf8bd0a9828f0bd7cebc5b414796bb1
---
 src/bin/ui/project_close.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/bin/ui/project_close.c b/src/bin/ui/project_close.c
index 5af5589..ebb60b2 100644
--- a/src/bin/ui/project_close.c
+++ b/src/bin/ui/project_close.c
@@ -152,9 +152,9 @@ project_close(void)
 
    assert(ap.project != NULL);
 
-   title = eina_stringshare_printf(_("Close project %s"), ap.project->name);
    if (ap.project->changed)
      {
+        title = eina_stringshare_printf(_("Close project %s"), 
ap.project->name);
         btn_res = popup_want_action(title, _("Do you want to save changes?"), 
NULL,
                                     NULL, BTN_OK|BTN_DONT_SAVE|BTN_CANCEL,
                                     NULL, NULL);
@@ -171,8 +171,8 @@ project_close(void)
               ERR("Popup return wrong value. Go to fix it!");
               abort(); /* it's wrong value need to fix popup code or popup 
call */
           }
+        eina_stringshare_del(title);
      }
-   eina_stringshare_del(title);
 
    ui_menu_items_list_disable_set(ap.menu, MENU_ITEMS_LIST_BASE, true);
    ui_menu_items_list_disable_set(ap.menu, MENU_ITEMS_LIST_STYLE_ONLY, true);

-- 


Reply via email to