[E-devel] elm : efl_thread example is using directly pthread

2012-03-21 Thread Vincent Torri
Hey

why isn't efl_thread.c elm example using pthread and not ecore_thread ?

Vincent

--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] elm : efl_thread example is using directly pthread

2012-03-21 Thread The Rasterman
On Wed, 21 Mar 2012 07:03:05 +0100 Vincent Torri vincent.to...@gmail.com said:

 Hey
 
 why isn't efl_thread.c elm example using pthread and not ecore_thread ?

there isnt' such a thing... unless u mean the 6 threading examples in elm docs?
and yes - some use pthread. it's an example of how you can use normal threading
apis WITH ecore together - eg if you have existing libraries or app code, or if
you simply prefer the manual control over your threads via pthread (or
whatever).

-- 
- Codito, ergo sum - I code, therefore I am --
The Rasterman (Carsten Haitzler)ras...@rasterman.com


--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] [Patch][elm_map] Map overlay color grouping route Fix

2012-03-21 Thread Daniel Juyung Seo
Hello,

On Wed, Mar 21, 2012 at 12:12 PM, Bluezery ohpo...@gmail.com wrote:
 I am not idle while waiting review :D
 There are many remain jobs to do in elm_map.  So, wait longer, patch bigger...
 Other patch is also waiting for you. Separating these are too harsh job. :(

Actually slow review is a bad thing happening in EFL. I need more reviewers.
But big patch makes review even slower and you won't get correct
review for your big patch.

Just separate your job.
Do not split the patch after you created a big patch.
While you work, consider separating patches and commits.
Thank you.

Daniel Juyung Seo (SeoZ)



 Anyway, I separated it!!
 Please review these again.


 2012/3/21 Daniel Juyung Seo seojuyu...@gmail.com:
 bluezery again!!
 SEPARATE THEM
 On Mar 20, 2012 7:11 PM, Sanjeev BA eflel...@gmail.com wrote:

 Do you want all of these to go in as one patch ?

 On Tue, Mar 20, 2012 at 4:58 PM, Bluezery ohpo...@gmail.com wrote:

  Hello,
 
  This is patch for elm_map
 
  1. Default and Class overlay can be set color by
   elm_map_overlay_color_set().
  2. Fix test application  for map API testing more.
  3. Fix grouping for enabling group callbacks. If overlay group is
  clicked, callback will be called.
  And elm_map_overlay_group_members_set() API will be added later to get
  list of overlays in the group.
  4. Simplify route overlay: Previously, many polygon objects are needed
  for one route overlay.
  But now  just one is needed.
 
  Please review this patch.
  Thanks
 
  --
  BRs,
  Kim.
 
 
 
 --
  This SF email is sponsosred by:
  Try Windows Azure free for 90 days Click Here
  http://p.sf.net/sfu/sfd2d-msazure
  ___
  enlightenment-devel mailing list
  enlightenment-devel@lists.sourceforge.net
  https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
 
 

 --
 This SF email is sponsosred by:
 Try Windows Azure free for 90 days Click Here
 http://p.sf.net/sfu/sfd2d-msazure
 ___
 enlightenment-devel mailing list
 enlightenment-devel@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

 --
 This SF email is sponsosred by:
 Try Windows Azure free for 90 days Click Here
 http://p.sf.net/sfu/sfd2d-msazure
 ___
 enlightenment-devel mailing list
 enlightenment-devel@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/enlightenment-devel



 --
 BRs,
 Kim.

 --
 This SF email is sponsosred by:
 Try Windows Azure free for 90 days Click Here
 http://p.sf.net/sfu/sfd2d-msazure
 ___
 enlightenment-devel mailing list
 enlightenment-devel@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


[E-devel] [Patch][elm_list] bug fix: crash occurs when container which contains the elm_list is deleted on the elm_list item clicked callback

2012-03-21 Thread cnook
Dear All, Hello.

I have resolved the elm_list issue(goto ELM_LIST_CRASH_ISSUE_MAIL below, I
sent mail but I cannot find the previous mail -_-;).
It would be related with double free issue. In the sub-obejct-del
callback which name is _sub_del() sets item-icon to NULL.
And in the EVAS_CALLBACK_MOUSE_UP callback funtion which name is
_mouse_up() calls evas_object_unref();
It would try to free the item-icon but the item-icon is already set to
NULL. So.. the crash would occur..

Anyhow my solutiion is the following.

static void
_del_pre_hook(Evas_Object *obj)
{
  Widget_Data *wd = elm_widget_data_get(obj);
  const Eina_List *l;
  Elm_List_Item *it;

  evas_object_smart_callback_del(obj, sub-object-del, _sub_del);

  if (!wd) return;

  EINA_LIST_FOREACH(wd-items, l, it)
{
   if (it-icon)
  evas_object_event_callback_del(it-icon,
 EVAS_CALLBACK_CHANGED_SIZE_HINTS,
 _changed_size_hints);
   if (it-end)
  evas_object_event_callback_del(it-icon,
 EVAS_CALLBACK_CHANGED_SIZE_HINTS,
 _changed_size_hints);
}

  evas_object_event_callback_del(wd-scr,
 EVAS_CALLBACK_CHANGED_SIZE_HINTS,
 _changed_size_hints);
  evas_object_event_callback_del(wd-box,
 EVAS_CALLBACK_CHANGED_SIZE_HINTS,
 _changed_size_hints);
}

I'm not sure this is the best way to resolve the issue. Anyhow the crash
does not occur.
I have checked this issue on the popup, ctxpopup and box already.
Please let me know, if you have any other proper ideas to reslove below
issue.. Thanks.




ELM_LIST_CRASH_ISSUE_MAIL:

Dear All, Hello.

If the elm_list is contained in the container - can be whatever containers
such as box, popup, ctxpopup and so on -
which has item with icon object such as a elm_radio and evas_object_del()
is called in the item select callback, the application is crashed.
If there is no icon on the elm_list item, the crash does not occur. The
example to occur the crash as the following.


{{{
static void _filter_all_cb(void *data, Evas_Object *obj,
  void *event_info)
{
   Evas_Object *popup = (Evas_Object *)data;
   evas_object_del(popup);
}

static void
_popup_center_text_cb(void *data, Evas_Object *obj __UNUSED__,
 void *event_info __UNUSED__)
{
  Evas_Object *popup, *btn1,*list, *radio;

  popup = elm_popup_add(data);
  evas_object_size_hint_weight_set(popup, EVAS_HINT_EXPAND,
EVAS_HINT_EXPAND);

   btn1 = elm_button_add(popup);
   elm_object_text_set(btn1, OK);
   elm_object_part_content_set(popup, button1, btn1);
   evas_object_smart_callback_add(btn1, clicked, NULL, NULL);

   list = elm_list_add(popup);
   evas_object_size_hint_weight_set(list, EVAS_HINT_EXPAND,
   EVAS_HINT_EXPAND);
   evas_object_size_hint_align_set(list, EVAS_HINT_FILL, EVAS_HINT_FILL);
   elm_list_mode_set(list, ELM_LIST_EXPAND);
   evas_object_show(list);

   Evas_Object *group;
   group = radio = elm_radio_add(list);
   evas_object_size_hint_weight_set(radio, EVAS_HINT_EXPAND,
   EVAS_HINT_EXPAND);
   elm_list_item_append(list, Radio, radio, NULL, _filter_all_cb, popup);
   evas_object_show(radio);

   elm_object_content_set(popup, list);
  evas_object_show(popup);
}

}}}

I will check this issue also, but your help will be pretty good to  me.
Thanks.
Index: src/lib/elm_list.c
===
--- src/lib/elm_list.c	(revision 69534)
+++ src/lib/elm_list.c	(working copy)
@@ -444,8 +444,25 @@ static void
 _del_pre_hook(Evas_Object *obj)
 {
Widget_Data *wd = elm_widget_data_get(obj);
+   const Eina_List *l;
+   Elm_List_Item *it;
+
+   evas_object_smart_callback_del(obj, sub-object-del, _sub_del);
+
if (!wd) return;
 
+   EINA_LIST_FOREACH(wd-items, l, it)
+ {
+if (it-icon)
+   evas_object_event_callback_del(it-icon,
+  EVAS_CALLBACK_CHANGED_SIZE_HINTS,
+  _changed_size_hints);
+if (it-end)
+   evas_object_event_callback_del(it-icon,
+  EVAS_CALLBACK_CHANGED_SIZE_HINTS,
+  _changed_size_hints);
+ }
+
evas_object_event_callback_del(wd-scr,
   EVAS_CALLBACK_CHANGED_SIZE_HINTS,
   _changed_size_hints);
--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


[E-devel] [Patch][elementary] elc_ctxpopup using elm_list internally.

2012-03-21 Thread cnook
Dear All, Hello.

The patch is for using elm_list as a item controller of elc_ctxpopup.
I have learned that there was an opinion to change the internal structure
of ctxpopup using the list.
It would be able to match up to your expectation. but it is just a draft
version of using list in ctxpopup.
So, It would be pretty good to me if you give any feedback on this. Thanks.

Sincerely,
Shinwoo Kim.


elc_ctxpopup.c.using.list.internally
Description: Binary data
--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] [PATCH] ecore/ecore_fb: Tweaks in _ecore_fb_size_get

2012-03-21 Thread The Rasterman
On Sat, 17 Mar 2012 15:16:49 +0530 Vikram Narayanan vikram...@gmail.com said:

 As the width and height are defined as static, no need to initialize it 
 again to zero. Also removed checks for 'valid pointers' as it is redundant.

question... why did you make these changes? did you get warnings or something
from a compiler (that i've never seen), or is this just being pedantic or
what? curious. :)

-- 
- Codito, ergo sum - I code, therefore I am --
The Rasterman (Carsten Haitzler)ras...@rasterman.com


--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] [Patch][elementary] elm_list.c - item_content_set() related.

2012-03-21 Thread The Rasterman
On Tue, 20 Mar 2012 14:27:55 +0900 cnook kimci...@gmail.com said:

yeah - you're right. the boolean was reversed. patch in svn! tnx! :)

 Dear All, Hello.
 
 The Eina_Bool item-dummy_icon and item-dummy_end of Elm_List_Item
 indicate whether the content has REAL object or not.
 - not means.. if elm_object_item_part_content_set(); is called with
 NULL content, the Elm_List set the swallow part with
 rectangle which has alpha value 0 and the item-dummy_icon or
 item-dummy_end is set to EINA_TRUE. -
 But in the item_content_set(); it works the opposite way. So when
 content set with NULL value the flag should be EINA_TRUE.
 The attached patch is for this. Please review the patch and give any
 feedbacks. Thanks.
 
 Sincerely,
 Shinwoo Kim.


-- 
- Codito, ergo sum - I code, therefore I am --
The Rasterman (Carsten Haitzler)ras...@rasterman.com


--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


[E-devel] [Question][Windows] Shift/Alt/Ctrl key press operation

2012-03-21 Thread cnook
Dear All, Hello.

I have a question about shift/alt/ctrl key press operation.
On the X environment, shift/alt/ctrl key press
event(EVAS_CALLBACK_KEY_DOWN) does not occur repeatedly .
But on the Windows, shift/alt/ctrl key press event occurs repeatedly.

Now let's suppose you have an application which takes care of
shift/alt/ctrl key press event.
The application would work improperly if there is not enough handling for
those key events.

So how about your opinion on this.
Should I change ecore_win32_things to occur EVAS_CALLBACK_KEY_DOWN event
once (not repeatedly)
Or should I change the Application?

Would you ENLIGHTEN me on this question?

Sincerely,
Shinwoo Kim.
--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


[E-devel] [patch] evas doc typos

2012-03-21 Thread Jérôme Pinot
3 typos in evas doc.


-- 
Jérôme Pinot
http://ngc891.blogdns.net/
Index: evas/src/lib/Evas.h
===
--- evas/src/lib/Evas.h (revision 69535)
+++ evas/src/lib/Evas.h (working copy)
@@ -2424,7 +2424,7 @@
  * of this kind during the freeze will get @b discarded. Every freeze
  * call must be matched by a thaw call in order to completely thaw out
  * a canvas (i.e. these calls may be nested). The most common use is
- * when you don't want the user to interect with your user interface
+ * when you don't want the user to interact with your user interface
  * when you're populating a view or changing the layout.
  *
  * Example:
@@ -9716,7 +9716,7 @@
  * used on the icon object.
  *
  * Besides that, generally smart objects implement a bspecific
- * API/b, so that users interect with its own custom features. The
+ * API/b, so that users interact with its own custom features. The
  * API takes form of explicit exported functions one may call and
  * bsmart callbacks/b.
  *
@@ -9730,7 +9730,7 @@
  * definition for the callback functions: #Evas_Smart_Cb.
  *
  * When defining an #Evas_Smart_Class, smart object implementors are
- * strongly encorauged to properly set the Evas_Smart_Class::callbacks
+ * strongly encouraged to properly set the Evas_Smart_Class::callbacks
  * callbacks description array, so that the users of the smart object
  * can have introspection on its events API bat run time/b.
  *


signature.asc
Description: Digital signature
--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] [patch] elm_genlist - bugfix at flip_realize

2012-03-21 Thread The Rasterman
On Tue, 20 Mar 2012 16:53:37 +0900 Hyoyoung Chang hyoyo...@gmail.com said:

this one seems good - in svn it is! tnx! :)

 oops sorry.
 in diff, i did a mistakes
 it's refined version.
 
 On Tue, Mar 20, 2012 at 4:43 PM, Sanjeev BA eflel...@gmail.com wrote:
  how does EINA_LIST_FOREACH(*source, l, key) work without initializing
  *source ?
 
  On Tue, Mar 20, 2012 at 4:31 PM, Hyoyoung Chang hyoyo...@gmail.com wrote:
 
  Dear all,
 
  I found a bug in _item_flip_realize.
  It replaces previous string list without free.
  this patch fixed for it.
 
  Thanks
 
 
  --
  This SF email is sponsosred by:
  Try Windows Azure free for 90 days Click Here
  http://p.sf.net/sfu/sfd2d-msazure
  ___
  enlightenment-devel mailing list
  enlightenment-devel@lists.sourceforge.net
  https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
 
 
  --
  This SF email is sponsosred by:
  Try Windows Azure free for 90 days Click Here
  http://p.sf.net/sfu/sfd2d-msazure
  ___
  enlightenment-devel mailing list
  enlightenment-devel@lists.sourceforge.net
  https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


-- 
- Codito, ergo sum - I code, therefore I am --
The Rasterman (Carsten Haitzler)ras...@rasterman.com


--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] [Question][Windows] Shift/Alt/Ctrl key press operation

2012-03-21 Thread The Rasterman
On Wed, 21 Mar 2012 19:53:30 +0900 cnook kimci...@gmail.com said:

 Dear All, Hello.
 
 I have a question about shift/alt/ctrl key press operation.
 On the X environment, shift/alt/ctrl key press
 event(EVAS_CALLBACK_KEY_DOWN) does not occur repeatedly .
 But on the Windows, shift/alt/ctrl key press event occurs repeatedly.
 
 Now let's suppose you have an application which takes care of
 shift/alt/ctrl key press event.
 The application would work improperly if there is not enough handling for
 those key events.

correct.

 So how about your opinion on this.
 Should I change ecore_win32_things to occur EVAS_CALLBACK_KEY_DOWN event
 once (not repeatedly)
 Or should I change the Application?

for modifiers (and locks - like capslock, numlock etc.) - yes, ecore_win32
should be3 changed to weed out key repeats of these. :(

 Would you ENLIGHTEN me on this question?
 
 Sincerely,
 Shinwoo Kim.
 --
 This SF email is sponsosred by:
 Try Windows Azure free for 90 days Click Here 
 http://p.sf.net/sfu/sfd2d-msazure
 ___
 enlightenment-devel mailing list
 enlightenment-devel@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
 


-- 
- Codito, ergo sum - I code, therefore I am --
The Rasterman (Carsten Haitzler)ras...@rasterman.com


--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] [Patch][Genlist] Add the moved.after/before signal instead of moved

2012-03-21 Thread The Rasterman
On Tue, 20 Mar 2012 16:40:56 +0900 chanwook jung jchanw...@gmail.com said:

ok - in svn, but you forgot to change the docs to match the . - , change.
fixed for you :)

 I make a new patch.
 I did not remove moved signal just add moved,after/before and change to ,.
 
 Thanks
 Joey
 
 2012/3/20 Hyoyoung Chang hyoyo...@gmail.com
 
  It seems nice except signal separator is .
  How about change to , from .? other genlist signals uses , as a
  naming separator.
  Also I think it's better gengrid also changed like this patch in next time
  :D
 
  On Tue, Mar 20, 2012 at 2:46 PM, chanwook jung jchanw...@gmail.com
  wrote:
   Dear all,
  
   Some applications want to know moved.after or before and relative item
   because of updating their own list.
   So I separated moved into moved.after and move.before.
  
   Thanks,
   Joey
  
  
  --
   This SF email is sponsosred by:
   Try Windows Azure free for 90 days Click Here
   http://p.sf.net/sfu/sfd2d-msazure
   ___
   enlightenment-devel mailing list
   enlightenment-devel@lists.sourceforge.net
   https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
  
 
 
  --
  This SF email is sponsosred by:
  Try Windows Azure free for 90 days Click Here
  http://p.sf.net/sfu/sfd2d-msazure
  ___
  enlightenment-devel mailing list
  enlightenment-devel@lists.sourceforge.net
  https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
 


-- 
- Codito, ergo sum - I code, therefore I am --
The Rasterman (Carsten Haitzler)ras...@rasterman.com


--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] [patch] elm_genlist - bugfix at item_state_update

2012-03-21 Thread The Rasterman
On Mon, 19 Mar 2012 20:03:58 +0900 Hyoyoung Chang hyoyo...@gmail.com said:

looked at patch. seems fine to me. in svn! tnx! :)

 Dear all,
 
 I fixed minor error in _elm_genlist_item_state_update.
 If item is selected, it sends signal to item and edit_obj.
 But edit_obj is not checked by if statement.
 I added a parenthesis for it.
 
 Thanks


-- 
- Codito, ergo sum - I code, therefore I am --
The Rasterman (Carsten Haitzler)ras...@rasterman.com


--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] [patch] evas doc typos

2012-03-21 Thread The Rasterman
On Wed, 21 Mar 2012 20:00:18 +0900 Jérôme Pinot ngc...@gmail.com said:

 3 typos in evas doc.

hehehe. that typo is funny. :) in svn! tnx muchly! :)

-- 
- Codito, ergo sum - I code, therefore I am --
The Rasterman (Carsten Haitzler)ras...@rasterman.com


--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] DNS in Ubuntu 12.04.

2012-03-21 Thread The Rasterman
On Wed, 21 Mar 2012 09:45:39 +0900 Sanjeev BA eflel...@gmail.com said:

 Hi,
 
 I found the following info useful to fix my DNS issue in Ubuntu 12.04,
 today morning.
 
 http://www.stgraber.org/2012/02/24/dns-in-ubuntu-12-04/

ummm... aaarrrh you had a dns issue? what was your issue?

-- 
- Codito, ergo sum - I code, therefore I am --
The Rasterman (Carsten Haitzler)ras...@rasterman.com


--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] [Patch][elm_list] bug fix: crash occurs when container which contains the elm_list is deleted on the elm_list item clicked callback

2012-03-21 Thread Hyoyoung Chang
I think previous your mail is in another your mail account :P
btw, i have a question.

+if (it-icon)
+   evas_object_event_callback_del(it-icon,
+  EVAS_CALLBACK_CHANGED_SIZE_HINTS,
+  _changed_size_hints);
+if (it-end)
+   evas_object_event_callback_del(it-icon,
+  EVAS_CALLBACK_CHANGED_SIZE_HINTS,
+  _changed_size_hints);

in second callback_del, it-icon should be it-end?

On Wed, Mar 21, 2012 at 5:53 PM, cnook kimci...@gmail.com wrote:
 Dear All, Hello.

 I have resolved the elm_list issue(goto ELM_LIST_CRASH_ISSUE_MAIL below, I
 sent mail but I cannot find the previous mail -_-;).
 It would be related with double free issue. In the sub-obejct-del
 callback which name is _sub_del() sets item-icon to NULL.
 And in the EVAS_CALLBACK_MOUSE_UP callback funtion which name is
 _mouse_up() calls evas_object_unref();
 It would try to free the item-icon but the item-icon is already set to
 NULL. So.. the crash would occur..

 Anyhow my solutiion is the following.

 static void
 _del_pre_hook(Evas_Object *obj)
 {
  Widget_Data *wd = elm_widget_data_get(obj);
  const Eina_List *l;
  Elm_List_Item *it;

  evas_object_smart_callback_del(obj, sub-object-del, _sub_del);

  if (!wd) return;

  EINA_LIST_FOREACH(wd-items, l, it)
    {
       if (it-icon)
          evas_object_event_callback_del(it-icon,
                                         EVAS_CALLBACK_CHANGED_SIZE_HINTS,
                                         _changed_size_hints);
       if (it-end)
          evas_object_event_callback_del(it-icon,
                                         EVAS_CALLBACK_CHANGED_SIZE_HINTS,
                                         _changed_size_hints);
    }

  evas_object_event_callback_del(wd-scr,
                                 EVAS_CALLBACK_CHANGED_SIZE_HINTS,
                                 _changed_size_hints);
  evas_object_event_callback_del(wd-box,
                                 EVAS_CALLBACK_CHANGED_SIZE_HINTS,
                                 _changed_size_hints);
 }

 I'm not sure this is the best way to resolve the issue. Anyhow the crash
 does not occur.
 I have checked this issue on the popup, ctxpopup and box already.
 Please let me know, if you have any other proper ideas to reslove below
 issue.. Thanks.




 ELM_LIST_CRASH_ISSUE_MAIL:

 Dear All, Hello.

 If the elm_list is contained in the container - can be whatever containers
 such as box, popup, ctxpopup and so on -
 which has item with icon object such as a elm_radio and evas_object_del()
 is called in the item select callback, the application is crashed.
 If there is no icon on the elm_list item, the crash does not occur. The
 example to occur the crash as the following.


 {{{
 static void _filter_all_cb(void *data, Evas_Object *obj,
              void *event_info)
 {
   Evas_Object *popup = (Evas_Object *)data;
   evas_object_del(popup);
 }

 static void
 _popup_center_text_cb(void *data, Evas_Object *obj __UNUSED__,
                     void *event_info __UNUSED__)
 {
  Evas_Object *popup, *btn1,*list, *radio;

  popup = elm_popup_add(data);
  evas_object_size_hint_weight_set(popup, EVAS_HINT_EXPAND,
 EVAS_HINT_EXPAND);

   btn1 = elm_button_add(popup);
   elm_object_text_set(btn1, OK);
   elm_object_part_content_set(popup, button1, btn1);
   evas_object_smart_callback_add(btn1, clicked, NULL, NULL);

   list = elm_list_add(popup);
   evas_object_size_hint_weight_set(list, EVAS_HINT_EXPAND,
           EVAS_HINT_EXPAND);
   evas_object_size_hint_align_set(list, EVAS_HINT_FILL, EVAS_HINT_FILL);
   elm_list_mode_set(list, ELM_LIST_EXPAND);
   evas_object_show(list);

   Evas_Object *group;
   group = radio = elm_radio_add(list);
   evas_object_size_hint_weight_set(radio, EVAS_HINT_EXPAND,
           EVAS_HINT_EXPAND);
   elm_list_item_append(list, Radio, radio, NULL, _filter_all_cb, popup);
   evas_object_show(radio);

   elm_object_content_set(popup, list);
  evas_object_show(popup);
 }

 }}}

 I will check this issue also, but your help will be pretty good to  me.
 Thanks.

 --
 This SF email is sponsosred by:
 Try Windows Azure free for 90 days Click Here
 http://p.sf.net/sfu/sfd2d-msazure
 ___
 enlightenment-devel mailing list
 enlightenment-devel@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] [Patch][elementary] elc_ctxpopup using elm_list internally.

2012-03-21 Thread Hyoyoung Chang
yeap, it seems nice.
btw, i have a question.

-   if (eina_list_count(wd-items)  1)
+   list = elm_object_item_widget_get(ctxpopup_it-list_item);
+
+   if (eina_list_count(elm_list_items_get(list))  2)
  {
+elm_object_item_del(ctxpopup_it-list_item);
 evas_object_hide(WIDGET(ctxpopup_it));
 return EINA_TRUE;
  }

is there any special reason to check '2' case?

On Wed, Mar 21, 2012 at 6:19 PM, cnook kimci...@gmail.com wrote:
 Dear All, Hello.

 The patch is for using elm_list as a item controller of elc_ctxpopup.
 I have learned that there was an opinion to change the internal structure
 of ctxpopup using the list.
 It would be able to match up to your expectation. but it is just a draft
 version of using list in ctxpopup.
 So, It would be pretty good to me if you give any feedback on this. Thanks.

 Sincerely,
 Shinwoo Kim.

 --
 This SF email is sponsosred by:
 Try Windows Azure free for 90 days Click Here
 http://p.sf.net/sfu/sfd2d-msazure
 ___
 enlightenment-devel mailing list
 enlightenment-devel@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


[E-devel] elm_genlist scrollto bottom

2012-03-21 Thread Leif Middelschulte
Hello there,

thanks to mike, I switched from elm_list to elm_genlist for my little
project, for which I need to scroll to the bottom most element of a
list, once it's realized.

Turns out that genlist can't do this, while elm_list can _if_ you
iterate the main loop once.

Code that demonstrates it is attached.


-- 
Leif
#include Elementary.h

#define EXPAND(o) evas_object_size_hint_weight_set(o, EVAS_HINT_EXPAND, EVAS_HINT_EXPAND)
#define FILL(o) evas_object_size_hint_align_set(o, EVAS_HINT_FILL, EVAS_HINT_FILL)

static Evas_Object *_item_content_get(void *data, Evas_Object *obj, const char *part);
static void _on_done(void *data, Evas_Object *obj, void *event_info);

static Elm_Genlist_Item_Class genlist_class = { .item_style = default, .func.content_get = _item_content_get, .func.del = NULL};

static Evas_Object *list;

static void _btn_add_cb(void *data, Evas_Object *obj, void *event_info)
{
   elm_genlist_item_append(list, genlist_class, NULL, NULL, ELM_GENLIST_ITEM_NONE, NULL, NULL);
}


static Evas_Object *_item_content_get(void *data, Evas_Object *obj, const char *part)
{
   Evas_Object *label;
   char timebuf[256];
   double ts;

   if (strcmp(part, elm.swallow.icon))
 return NULL;

   printf(called for part %s\n, part);

   label = elm_label_add(obj);
   evas_object_show(label);
   EXPAND(label);

   ts = ecore_time_get();
   strftime(timebuf, sizeof(timebuf), [%H:%M:%S],
 localtime((time_t[]){ ts }));
   elm_object_text_set(label, timebuf);

   return label;
}


static void _item_realized_cb(void *data, Evas_Object *obj, void *event_info)
{
   Elm_Object_Item *li = (Elm_Object_Item*)event_info;

   elm_genlist_item_bring_in(li, ELM_GENLIST_ITEM_SCROLLTO_IN);
}

int main(int argc, char *argv[])
{
   Evas_Object *win, *box;
   Evas_Object *btn;
   unsigned int j;

   elm_init(argc, argv);

   win = elm_win_util_standard_add(Elm Playground, Elm Playground - List append test);
   evas_object_show(win);
   evas_object_smart_callback_add(win, delete,request, _on_done, NULL);

   box = elm_box_add(win);
   evas_object_show(box);
   EXPAND(box);
   FILL(box);

   btn = elm_button_add(box);
   evas_object_show(btn);
   elm_object_text_set(btn, Add item);
   evas_object_size_hint_weight_set(btn, EVAS_HINT_EXPAND, 0.0);
   evas_object_size_hint_align_set(btn, EVAS_HINT_FILL, 0.0);
   evas_object_smart_callback_add(btn, clicked, _btn_add_cb, NULL);
   elm_box_pack_end(box, btn);

   list = elm_genlist_add(box);
   evas_object_show(list);
   EXPAND(list);
   FILL(list);
   evas_object_smart_callback_add(list, realized, _item_realized_cb, NULL);
   for (j = 0; j  20; j++)
 {
elm_genlist_item_append(list, genlist_class, NULL, NULL, ELM_GENLIST_ITEM_NONE, NULL, NULL);
 }
   elm_box_pack_end(box, list);

   elm_win_resize_object_add(win, box);
   evas_object_resize(win, 200, 400);

   elm_run();

   elm_shutdown();

   return 0;
}

   static void
_on_done(void *data, Evas_Object *obj, void *event_info)
{
   elm_exit();
}
--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] elm_genlist scrollto bottom

2012-03-21 Thread Daniel Juyung Seo
Oh.. Genlist is designed to handle this kind of situation without trick.
It's a genlist bug. In case you have more than 34 items, genlist will
handle this correctly.
I will fix this soon. (very busy at the moment.)
So you don't even need to use realized callback. Genlist will do all
the things for you.
I attached a sample code.
Thank you.

Daniel Juyung Seo (SeoZ)

On Wed, Mar 21, 2012 at 11:32 PM, Leif Middelschulte
leif.middelschu...@gmail.com wrote:
 Hello there,

 thanks to mike, I switched from elm_list to elm_genlist for my little
 project, for which I need to scroll to the bottom most element of a
 list, once it's realized.

 Turns out that genlist can't do this, while elm_list can _if_ you
 iterate the main loop once.

 Code that demonstrates it is attached.


 --
 Leif

 --
 This SF email is sponsosred by:
 Try Windows Azure free for 90 days Click Here
 http://p.sf.net/sfu/sfd2d-msazure
 ___
 enlightenment-devel mailing list
 enlightenment-devel@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

#include Elementary.h

#define EXPAND(o) evas_object_size_hint_weight_set(o, EVAS_HINT_EXPAND, EVAS_HINT_EXPAND)
#define FILL(o) evas_object_size_hint_align_set(o, EVAS_HINT_FILL, EVAS_HINT_FILL)

static Evas_Object *_item_content_get(void *data, Evas_Object *obj, const char *part);
static void _on_done(void *data, Evas_Object *obj, void *event_info);

static Elm_Genlist_Item_Class genlist_class = { .item_style = default, .func.content_get = _item_content_get, .func.del = NULL};

static Evas_Object *list;

static void _btn_add_cb(void *data, Evas_Object *obj, void *event_info)
{
   Elm_Object_Item *li;
   li = elm_genlist_item_append(list, genlist_class, NULL, NULL, ELM_GENLIST_ITEM_NONE, NULL, NULL);
   elm_genlist_item_bring_in(li, ELM_GENLIST_ITEM_SCROLLTO_IN);
}


static Evas_Object *_item_content_get(void *data, Evas_Object *obj, const char *part)
{
   Evas_Object *label;
   char timebuf[256];
   double ts;

   if (strcmp(part, elm.swallow.icon))
 return NULL;

   printf(called for part %s\n, part);

   label = elm_label_add(obj);
   evas_object_show(label);
   EXPAND(label);

   ts = ecore_time_get();
   strftime(timebuf, sizeof(timebuf), [%H:%M:%S],
 localtime((time_t[]){ ts }));
   elm_object_text_set(label, timebuf);

   return label;
}

int main(int argc, char *argv[])
{
   Evas_Object *win, *box;
   Evas_Object *btn;
   unsigned int j;
   Elm_Object_Item *li;

   elm_init(argc, argv);

   win = elm_win_util_standard_add(Elm Playground, Elm Playground - List append test);
   evas_object_show(win);
   evas_object_smart_callback_add(win, delete,request, _on_done, NULL);

   box = elm_box_add(win);
   evas_object_show(box);
   EXPAND(box);
   FILL(box);

   btn = elm_button_add(box);
   evas_object_show(btn);
   elm_object_text_set(btn, Add item);
   evas_object_size_hint_weight_set(btn, EVAS_HINT_EXPAND, 0.0);
   evas_object_size_hint_align_set(btn, EVAS_HINT_FILL, 0.0);
   evas_object_smart_callback_add(btn, clicked, _btn_add_cb, NULL);
   elm_box_pack_end(box, btn);

   list = elm_genlist_add(box);
   evas_object_show(list);
   EXPAND(list);
   FILL(list);
   for (j = 0; j  34; j++)
 {
li = elm_genlist_item_append(list, genlist_class, NULL, NULL, ELM_GENLIST_ITEM_NONE, NULL, NULL);
 }
   elm_genlist_item_bring_in(li, ELM_GENLIST_ITEM_SCROLLTO_IN);
   elm_box_pack_end(box, list);

   elm_win_resize_object_add(win, box);
   evas_object_resize(win, 200, 400);

   elm_run();

   elm_shutdown();

   return 0;
}

   static void
_on_done(void *data, Evas_Object *obj, void *event_info)
{
   elm_exit();
}
--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] elm_genlist scrollto bottom

2012-03-21 Thread Leif Middelschulte
2012/3/21 Daniel Juyung Seo seojuyu...@gmail.com:
 Oh.. Genlist is designed to handle this kind of situation without trick.
 It's a genlist bug. In case you have more than 34 items, genlist will
 handle this correctly.
What an arbitraty number, isn't it?!
 I will fix this soon. (very busy at the moment.)
Even in your example, it seems like it would use 'show' insted of
'bring in' (no animation).
 So you don't even need to use realized callback. Genlist will do all
 the things for you.
What will it do for me? I mean, I don't want to to keep scrolling to
the bottom element. I want to be able to scroll the list, but make it
scroll down, when a new element is inserted.
 I attached a sample code.
 Thank you.
Thank you, for your quick reply :-)

 Daniel Juyung Seo (SeoZ)

 On Wed, Mar 21, 2012 at 11:32 PM, Leif Middelschulte
 leif.middelschu...@gmail.com wrote:
 Hello there,

 thanks to mike, I switched from elm_list to elm_genlist for my little
 project, for which I need to scroll to the bottom most element of a
 list, once it's realized.

 Turns out that genlist can't do this, while elm_list can _if_ you
 iterate the main loop once.

 Code that demonstrates it is attached.


 --
 Leif

 --
 This SF email is sponsosred by:
 Try Windows Azure free for 90 days Click Here
 http://p.sf.net/sfu/sfd2d-msazure
 ___
 enlightenment-devel mailing list
 enlightenment-devel@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


 --
 This SF email is sponsosred by:
 Try Windows Azure free for 90 days Click Here
 http://p.sf.net/sfu/sfd2d-msazure
 ___
 enlightenment-devel mailing list
 enlightenment-devel@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/enlightenment-devel




-- 
Leif

--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] [PATCH] ecore/ecore_fb: Tweaks in _ecore_fb_size_get

2012-03-21 Thread Vikram Narayanan
On 3/21/2012 3:28 PM, Carsten Haitzler (The Rasterman) wrote:
 On Sat, 17 Mar 2012 15:16:49 +0530 Vikram Narayananvikram...@gmail.com  
 said:

 As the width and height are defined as static, no need to initialize it
 again to zero. Also removed checks for 'valid pointers' as it is redundant.

 question... why did you make these changes? did you get warnings or something
 from a compiler (that i've never seen), or is this just being pedantic or
 what? curious. :)


The closest answer is pedantic. :)

Was just compiling the e17 from sources and just noticed this one. I 
leave it to the community to decide whether or not they are interested 
in these type of patches :)

~Vikram

--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


[E-devel] Screen Resolution - Restore on Login

2012-03-21 Thread Jeff Hoogland
Can we please get the restore on login option in the screen resolution
tool to actually work or else remove it until it is functional?

It causes confusion among users every now and then with it being there, but
not functioning.

-- 
~Jeff Hoogland http://jeffhoogland.com/
Thoughts on Technology http://jeffhoogland.blogspot.com/, Tech Blog
Bodhi Linux http://bodhilinux.com/, Enlightenment for your Desktop
--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


[E-devel] create custom elementary widget

2012-03-21 Thread Andreas Volz
Hi,

I've some special widgets which I couldn't assemble from other existing
widgets (rotary). So I like to create a new one. Could you tell me the
way how to do this?

Should I create a new one in elementary/src/lib/ (I won't commit it
without permission!) or is there some elementary-extras library in
PROTO? Or should I create a private lib on top of elementary for my
special widgets?

Could someone explain me a difference between elm_*.c, elc_*.c, elu_*.c
and els_*.c. Is this documented somewhere? I'm not sure which one to
take as template for my new widget.

regards
Andreas

-- 
Technical Blog http://andreasvolz.wordpress.com/

--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] create custom elementary widget

2012-03-21 Thread Gustavo Chaves
On Wed, Mar 21, 2012 at 6:30 PM, Andreas Volz li...@brachttal.net wrote:

 Hi,


Hi.


 I've some special widgets which I couldn't assemble from other existing
 widgets (rotary). So I like to create a new one. Could you tell me the
 way how to do this?

 Should I create a new one in elementary/src/lib/ (I won't commit it
 without permission!) or is there some elementary-extras library in
 PROTO? Or should I create a private lib on top of elementary for my
 special widgets?


No. Just create it inside your application's code. Just include
elm_widget.h, define ELM_INTERNAL_API_ARGESFSDFEFC so it won't bitch at
you, and use the elm_widget_* api to assemble it, just like the other ones.

After my work gets merged, that will change, be warned.


 Could someone explain me a difference between elm_*.c, elc_*.c, elu_*.c
 and els_*.c. Is this documented somewhere? I'm not sure which one to
 take as template for my new widget.


elm_* are, say, base widgets. elc_* are composed widgets (ones aggregating
others). Don't bother with elu_* stuff. Right now you won't be able to
extend widgets behavior (except for composing them, like the elc_* stuff),
but after my work gets in this will be straighforward and the way to go.


 regards
 Andreas

 --
 Technical Blog http://andreasvolz.wordpress.com/


 --
 This SF email is sponsosred by:
 Try Windows Azure free for 90 days Click Here
 http://p.sf.net/sfu/sfd2d-msazure
 ___
 enlightenment-devel mailing list
 enlightenment-devel@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/enlightenment-devel




-- 
Gustavo Lima Chaves
Senior Developer

ProFUSION embedded systems

http://profusion.mobi
--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] Screen Resolution - Restore on Login

2012-03-21 Thread Leif Middelschulte
2012/3/21 Jeff Hoogland jeffhoogl...@linux.com:
 Can we please get the restore on login option in the screen resolution
 tool to actually work or else remove it until it is functional?

 It causes confusion among users every now and then with it being there, but
 not functioning.
Fixed in r69547.


 --
 ~Jeff Hoogland http://jeffhoogland.com/
 Thoughts on Technology http://jeffhoogland.blogspot.com/, Tech Blog
 Bodhi Linux http://bodhilinux.com/, Enlightenment for your Desktop
 --
 This SF email is sponsosred by:
 Try Windows Azure free for 90 days Click Here
 http://p.sf.net/sfu/sfd2d-msazure
 ___
 enlightenment-devel mailing list
 enlightenment-devel@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/enlightenment-devel



-- 
Leif

--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] Screen Resolution - Restore on Login

2012-03-21 Thread The Rasterman
On Wed, 21 Mar 2012 14:10:38 -0500 Jeff Hoogland jeffhoogl...@linux.com said:

 Can we please get the restore on login option in the screen resolution
 tool to actually work or else remove it until it is functional?
 
 It causes confusion among users every now and then with it being there, but
 not functioning.

the more fundamental issue is us having now a screen res and a screen setup
config... both of which actually conflict/overlap.

-- 
- Codito, ergo sum - I code, therefore I am --
The Rasterman (Carsten Haitzler)ras...@rasterman.com


--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] [Patch][elm_list] bug fix: crash occurs when container which contains the elm_list is deleted on the elm_list item clicked callback

2012-03-21 Thread Kim Shinwoo
OMG. Thanks for your response. You're definitely right.
I have attached revised patch. Please review the patch.

Anyhow.. Are you alive?!!


2012/3/21 Hyoyoung Chang hyoyo...@gmail.com:
 I think previous your mail is in another your mail account :P
 btw, i have a question.

 +        if (it-icon)
 +           evas_object_event_callback_del(it-icon,
 +                                          EVAS_CALLBACK_CHANGED_SIZE_HINTS,
 +                                          _changed_size_hints);
 +        if (it-end)
 +           evas_object_event_callback_del(it-icon,
 +                                          EVAS_CALLBACK_CHANGED_SIZE_HINTS,
 +                                          _changed_size_hints);

 in second callback_del, it-icon should be it-end?

 On Wed, Mar 21, 2012 at 5:53 PM, cnook kimci...@gmail.com wrote:
 Dear All, Hello.

 I have resolved the elm_list issue(goto ELM_LIST_CRASH_ISSUE_MAIL below, I
 sent mail but I cannot find the previous mail -_-;).
 It would be related with double free issue. In the sub-obejct-del
 callback which name is _sub_del() sets item-icon to NULL.
 And in the EVAS_CALLBACK_MOUSE_UP callback funtion which name is
 _mouse_up() calls evas_object_unref();
 It would try to free the item-icon but the item-icon is already set to
 NULL. So.. the crash would occur..

 Anyhow my solutiion is the following.

 static void
 _del_pre_hook(Evas_Object *obj)
 {
  Widget_Data *wd = elm_widget_data_get(obj);
  const Eina_List *l;
  Elm_List_Item *it;

  evas_object_smart_callback_del(obj, sub-object-del, _sub_del);

  if (!wd) return;

  EINA_LIST_FOREACH(wd-items, l, it)
    {
       if (it-icon)
          evas_object_event_callback_del(it-icon,
                                         EVAS_CALLBACK_CHANGED_SIZE_HINTS,
                                         _changed_size_hints);
       if (it-end)
          evas_object_event_callback_del(it-icon,
                                         EVAS_CALLBACK_CHANGED_SIZE_HINTS,
                                         _changed_size_hints);
    }

  evas_object_event_callback_del(wd-scr,
                                 EVAS_CALLBACK_CHANGED_SIZE_HINTS,
                                 _changed_size_hints);
  evas_object_event_callback_del(wd-box,
                                 EVAS_CALLBACK_CHANGED_SIZE_HINTS,
                                 _changed_size_hints);
 }

 I'm not sure this is the best way to resolve the issue. Anyhow the crash
 does not occur.
 I have checked this issue on the popup, ctxpopup and box already.
 Please let me know, if you have any other proper ideas to reslove below
 issue.. Thanks.




 ELM_LIST_CRASH_ISSUE_MAIL:

 Dear All, Hello.

 If the elm_list is contained in the container - can be whatever containers
 such as box, popup, ctxpopup and so on -
 which has item with icon object such as a elm_radio and evas_object_del()
 is called in the item select callback, the application is crashed.
 If there is no icon on the elm_list item, the crash does not occur. The
 example to occur the crash as the following.


 {{{
 static void _filter_all_cb(void *data, Evas_Object *obj,
              void *event_info)
 {
   Evas_Object *popup = (Evas_Object *)data;
   evas_object_del(popup);
 }

 static void
 _popup_center_text_cb(void *data, Evas_Object *obj __UNUSED__,
                     void *event_info __UNUSED__)
 {
  Evas_Object *popup, *btn1,*list, *radio;

  popup = elm_popup_add(data);
  evas_object_size_hint_weight_set(popup, EVAS_HINT_EXPAND,
 EVAS_HINT_EXPAND);

   btn1 = elm_button_add(popup);
   elm_object_text_set(btn1, OK);
   elm_object_part_content_set(popup, button1, btn1);
   evas_object_smart_callback_add(btn1, clicked, NULL, NULL);

   list = elm_list_add(popup);
   evas_object_size_hint_weight_set(list, EVAS_HINT_EXPAND,
           EVAS_HINT_EXPAND);
   evas_object_size_hint_align_set(list, EVAS_HINT_FILL, EVAS_HINT_FILL);
   elm_list_mode_set(list, ELM_LIST_EXPAND);
   evas_object_show(list);

   Evas_Object *group;
   group = radio = elm_radio_add(list);
   evas_object_size_hint_weight_set(radio, EVAS_HINT_EXPAND,
           EVAS_HINT_EXPAND);
   elm_list_item_append(list, Radio, radio, NULL, _filter_all_cb, popup);
   evas_object_show(radio);

   elm_object_content_set(popup, list);
  evas_object_show(popup);
 }

 }}}

 I will check this issue also, but your help will be pretty good to  me.
 Thanks.

 --
 This SF email is sponsosred by:
 Try Windows Azure free for 90 days Click Here
 http://p.sf.net/sfu/sfd2d-msazure
 ___
 enlightenment-devel mailing list
 enlightenment-devel@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


 --
 This SF email is sponsosred by:
 Try Windows Azure free for 90 days Click Here
 http://p.sf.net/sfu/sfd2d-msazure
 

Re: [E-devel] [Patch][elementary] elc_ctxpopup using elm_list internally.

2012-03-21 Thread Kim Shinwoo
Thanks for your response.
It is for hiding ctxpopup if there is no item..


2012/3/21 Hyoyoung Chang hyoyo...@gmail.com:
 yeap, it seems nice.
 btw, i have a question.

 -   if (eina_list_count(wd-items)  1)
 +   list = elm_object_item_widget_get(ctxpopup_it-list_item);
 +
 +   if (eina_list_count(elm_list_items_get(list))  2)
      {
 +        elm_object_item_del(ctxpopup_it-list_item);
         evas_object_hide(WIDGET(ctxpopup_it));
         return EINA_TRUE;
      }

 is there any special reason to check '2' case?

 On Wed, Mar 21, 2012 at 6:19 PM, cnook kimci...@gmail.com wrote:
 Dear All, Hello.

 The patch is for using elm_list as a item controller of elc_ctxpopup.
 I have learned that there was an opinion to change the internal structure
 of ctxpopup using the list.
 It would be able to match up to your expectation. but it is just a draft
 version of using list in ctxpopup.
 So, It would be pretty good to me if you give any feedback on this. Thanks.

 Sincerely,
 Shinwoo Kim.

 --
 This SF email is sponsosred by:
 Try Windows Azure free for 90 days Click Here
 http://p.sf.net/sfu/sfd2d-msazure
 ___
 enlightenment-devel mailing list
 enlightenment-devel@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


 --
 This SF email is sponsosred by:
 Try Windows Azure free for 90 days Click Here
 http://p.sf.net/sfu/sfd2d-msazure
 ___
 enlightenment-devel mailing list
 enlightenment-devel@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] Screen Resolution - Restore on Login

2012-03-21 Thread Jeff Hoogland
Fixed as in it was removed or it now works? Either way thanks! Building the
update now.

On Wed, Mar 21, 2012 at 6:42 PM, Leif Middelschulte 
leif.middelschu...@gmail.com wrote:

 2012/3/21 Jeff Hoogland jeffhoogl...@linux.com:
  Can we please get the restore on login option in the screen resolution
  tool to actually work or else remove it until it is functional?
 
  It causes confusion among users every now and then with it being there,
 but
  not functioning.
 Fixed in r69547.

 
  --
  ~Jeff Hoogland http://jeffhoogland.com/
  Thoughts on Technology http://jeffhoogland.blogspot.com/, Tech Blog
  Bodhi Linux http://bodhilinux.com/, Enlightenment for your Desktop
 
 --
  This SF email is sponsosred by:
  Try Windows Azure free for 90 days Click Here
  http://p.sf.net/sfu/sfd2d-msazure
  ___
  enlightenment-devel mailing list
  enlightenment-devel@lists.sourceforge.net
  https://lists.sourceforge.net/lists/listinfo/enlightenment-devel



 --
 Leif


 --
 This SF email is sponsosred by:
 Try Windows Azure free for 90 days Click Here
 http://p.sf.net/sfu/sfd2d-msazure
 ___
 enlightenment-devel mailing list
 enlightenment-devel@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/enlightenment-devel




-- 
~Jeff Hoogland http://jeffhoogland.com/
Thoughts on Technology http://jeffhoogland.blogspot.com/, Tech Blog
Bodhi Linux http://bodhilinux.com/, Enlightenment for your Desktop
--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] [patch] elm_map - Bug fix

2012-03-21 Thread The Rasterman
On Tue, 20 Mar 2012 16:11:28 +0900 Bluezery ohpo...@gmail.com said:

seems like this is in svn... :/ it'd be nice if ppl reply TO THE LIST... when
they put things in svn :) saves me re-reviewing this :)

 I resend map patch!
 Thanks for your comments! !
 
 2012/3/20 Bluezery ohpo...@gmail.com:
  2012/3/19 Bluezery ohpo...@gmail.com:
  2012/3/19 Sanjeev BA eflel...@gmail.com:
  I am looking into it.
 
  In removed/deprecated header, some APIs are removed.
  So those are not need to be patched.
  I resend modified patch.
  Thanks.
 
  Dear Sanjeeve,
 
  I am waiting for you.  :)
 
 
 
  On Sun, Mar 18, 2012 at 12:08 PM, Bluezery ohpo...@gmail.com wrote:
 
  2012/3/10 Bluezery ohpo...@gmail.com:
   Hello,
  
   This patch fixes two elm map bugs
   1) Geometry calculation error
   Calculation of canvas geometry is not correct If other evas objects
   take up some space in the canvas.
   This patch fix this by calculating smart pan geometry values.
  
   2) Map moves southeastward when zooming
   When repeating zoom in/out, map moves southeastward.
   ceil() function fixes this bug.
  
   Thanks.
  
   --
   BRs,
   Kim.
 
 
  Could you review this patch, please??  :)
 
  --
  BRs,
  Kim.
 
 
  --
  This SF email is sponsosred by:
  Try Windows Azure free for 90 days Click Here
  http://p.sf.net/sfu/sfd2d-msazure
  ___
  enlightenment-devel mailing list
  enlightenment-devel@lists.sourceforge.net
  https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
 
  --
  This SF email is sponsosred by:
  Try Windows Azure free for 90 days Click Here
  http://p.sf.net/sfu/sfd2d-msazure
  ___
  enlightenment-devel mailing list
  enlightenment-devel@lists.sourceforge.net
  https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
 
 
 
  --
  BRs,
  Kim.
 
 
 
  --
  BRs,
  Kim.
 
 
 
 -- 
 BRs,
 Kim.


-- 
- Codito, ergo sum - I code, therefore I am --
The Rasterman (Carsten Haitzler)ras...@rasterman.com


--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] [Patch][elm_map] Map overlay color grouping route Fix

2012-03-21 Thread The Rasterman
On Wed, 21 Mar 2012 12:12:45 +0900 Bluezery ohpo...@gmail.com said:

ok 1 thing. 

+   ELM_MAP_OVERLAY_TYPE_GROUP,

should be added to the END of the enum list. you just broke abi. i fixed it for
you. :)

in svn now :)

 I am not idle while waiting review :D
 There are many remain jobs to do in elm_map.  So, wait longer, patch bigger...
 Other patch is also waiting for you. Separating these are too harsh job. :(
 
 Anyway, I separated it!!
 Please review these again.
 
 
 2012/3/21 Daniel Juyung Seo seojuyu...@gmail.com:
  bluezery again!!
  SEPARATE THEM
  On Mar 20, 2012 7:11 PM, Sanjeev BA eflel...@gmail.com wrote:
 
  Do you want all of these to go in as one patch ?
 
  On Tue, Mar 20, 2012 at 4:58 PM, Bluezery ohpo...@gmail.com wrote:
 
   Hello,
  
   This is patch for elm_map
  
   1. Default and Class overlay can be set color by
    elm_map_overlay_color_set().
   2. Fix test application  for map API testing more.
   3. Fix grouping for enabling group callbacks. If overlay group is
   clicked, callback will be called.
   And elm_map_overlay_group_members_set() API will be added later to get
   list of overlays in the group.
   4. Simplify route overlay: Previously, many polygon objects are needed
   for one route overlay.
   But now  just one is needed.
  
   Please review this patch.
   Thanks
  
   --
   BRs,
   Kim.
  
  
  
  --
   This SF email is sponsosred by:
   Try Windows Azure free for 90 days Click Here
   http://p.sf.net/sfu/sfd2d-msazure
   ___
   enlightenment-devel mailing list
   enlightenment-devel@lists.sourceforge.net
   https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
  
  
 
  --
  This SF email is sponsosred by:
  Try Windows Azure free for 90 days Click Here
  http://p.sf.net/sfu/sfd2d-msazure
  ___
  enlightenment-devel mailing list
  enlightenment-devel@lists.sourceforge.net
  https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
 
  --
  This SF email is sponsosred by:
  Try Windows Azure free for 90 days Click Here
  http://p.sf.net/sfu/sfd2d-msazure
  ___
  enlightenment-devel mailing list
  enlightenment-devel@lists.sourceforge.net
  https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
 
 
 
 -- 
 BRs,
 Kim.


-- 
- Codito, ergo sum - I code, therefore I am --
The Rasterman (Carsten Haitzler)ras...@rasterman.com


--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] [Patch][elm_map] Map overlay color grouping route Fix

2012-03-21 Thread The Rasterman
On Wed, 21 Mar 2012 15:59:32 +0900 Daniel Juyung Seo seojuyu...@gmail.com
said:

 Hello,
 
 On Wed, Mar 21, 2012 at 12:12 PM, Bluezery ohpo...@gmail.com wrote:
  I am not idle while waiting review :D
  There are many remain jobs to do in elm_map.  So, wait longer, patch
  bigger... Other patch is also waiting for you. Separating these are too
  harsh job. :(
 
 Actually slow review is a bad thing happening in EFL. I need more reviewers.
 But big patch makes review even slower and you won't get correct
 review for your big patch.
 
 Just separate your job.
 Do not split the patch after you created a big patch.
 While you work, consider separating patches and commits.
 Thank you.

actually realistically, review here in efl is pretty fast. you should try lots
of other open source projects where patches get lost/ignored for YEARS :)

 Daniel Juyung Seo (SeoZ)
 
 
 
  Anyway, I separated it!!
  Please review these again.
 
 
  2012/3/21 Daniel Juyung Seo seojuyu...@gmail.com:
  bluezery again!!
  SEPARATE THEM
  On Mar 20, 2012 7:11 PM, Sanjeev BA eflel...@gmail.com wrote:
 
  Do you want all of these to go in as one patch ?
 
  On Tue, Mar 20, 2012 at 4:58 PM, Bluezery ohpo...@gmail.com wrote:
 
   Hello,
  
   This is patch for elm_map
  
   1. Default and Class overlay can be set color by
    elm_map_overlay_color_set().
   2. Fix test application  for map API testing more.
   3. Fix grouping for enabling group callbacks. If overlay group is
   clicked, callback will be called.
   And elm_map_overlay_group_members_set() API will be added later to get
   list of overlays in the group.
   4. Simplify route overlay: Previously, many polygon objects are needed
   for one route overlay.
   But now  just one is needed.
  
   Please review this patch.
   Thanks
  
   --
   BRs,
   Kim.
  
  
  
  --
   This SF email is sponsosred by:
   Try Windows Azure free for 90 days Click Here
   http://p.sf.net/sfu/sfd2d-msazure
   ___
   enlightenment-devel mailing list
   enlightenment-devel@lists.sourceforge.net
   https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
  
  
 
  --
  This SF email is sponsosred by:
  Try Windows Azure free for 90 days Click Here
  http://p.sf.net/sfu/sfd2d-msazure
  ___
  enlightenment-devel mailing list
  enlightenment-devel@lists.sourceforge.net
  https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
 
  --
  This SF email is sponsosred by:
  Try Windows Azure free for 90 days Click Here
  http://p.sf.net/sfu/sfd2d-msazure
  ___
  enlightenment-devel mailing list
  enlightenment-devel@lists.sourceforge.net
  https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
 
 
 
  --
  BRs,
  Kim.
 
  --
  This SF email is sponsosred by:
  Try Windows Azure free for 90 days Click Here
  http://p.sf.net/sfu/sfd2d-msazure
  ___
  enlightenment-devel mailing list
  enlightenment-devel@lists.sourceforge.net
  https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
 
 
 --
 This SF email is sponsosred by:
 Try Windows Azure free for 90 days Click Here 
 http://p.sf.net/sfu/sfd2d-msazure
 ___
 enlightenment-devel mailing list
 enlightenment-devel@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
 


-- 
- Codito, ergo sum - I code, therefore I am --
The Rasterman (Carsten Haitzler)ras...@rasterman.com


--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] [Bug Report] when elm_list deletes its container.

2012-03-21 Thread The Rasterman
On Wed, 21 Mar 2012 10:53:47 +0900 Kim Shinwoo kimcinoo@gmail.com said:

do you have a more complete test case with code? that'd be awesome!

 Dear All, Hello.
 
 If the elm_list is contained in the container - can be popup, ctxpopup
 and so on - which has item with icon object such as a elm_radio
 and evas_object_del() is called in the item select callback, the
 application is crashed.
 If there is no icon on the elm_list item, the crash does not occur.
 The example to occur the crash as the following.
 
 
 static void _filter_all_cb(void *data, Evas_Object *obj,
void *event_info)
 {
 Evas_Object *popup = (Evas_Object *)data;
 evas_object_del(popup);
 }
 
 static void
 _popup_center_text_cb(void *data, Evas_Object *obj __UNUSED__,
   void *event_info __UNUSED__)
 {
Evas_Object *popup, *btn1,*list, *radio;
 
popup = elm_popup_add(data);
evas_object_size_hint_weight_set(popup, EVAS_HINT_EXPAND,
 EVAS_HINT_EXPAND);
 
 btn1 = elm_button_add(popup);
 elm_object_text_set(btn1, OK);
 elm_object_part_content_set(popup, button1, btn1);
 evas_object_smart_callback_add(btn1, clicked, NULL,
 NULL);//_response_cb, popup);
 
 list = elm_list_add(popup);
 evas_object_size_hint_weight_set(list, EVAS_HINT_EXPAND,
 EVAS_HINT_EXPAND);
 evas_object_size_hint_align_set(list, EVAS_HINT_FILL, EVAS_HINT_FILL);
 elm_list_mode_set(list, ELM_LIST_EXPAND);
 evas_object_show(list);
 
 Evas_Object *group;
 group = radio = elm_radio_add(list);
 evas_object_size_hint_weight_set(radio, EVAS_HINT_EXPAND,
 EVAS_HINT_EXPAND);
 elm_list_item_append(list, Radio, radio, NULL, _filter_all_cb, popup);
 evas_object_show(radio);
 
 elm_object_content_set(popup, list);
evas_object_show(popup);
 }
 
 I will check this issue also, but your help will be pretty good to  me.
 Thanks. Anyhow, I cannot create a ticket because my content is considered as
 a spam.. :(
 
 Sincerely,
 Shinwoo Kim.
 
 --
 This SF email is sponsosred by:
 Try Windows Azure free for 90 days Click Here 
 http://p.sf.net/sfu/sfd2d-msazure
 ___
 enlightenment-devel mailing list
 enlightenment-devel@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
 


-- 
- Codito, ergo sum - I code, therefore I am --
The Rasterman (Carsten Haitzler)ras...@rasterman.com


--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] [Patch][elm_map] Map overlay color grouping route Fix

2012-03-21 Thread Bluezery
 On Wed, 21 Mar 2012 15:59:32 +0900 Daniel Juyung Seo seojuyu...@gmail.com
 said:
 Actually slow review is a bad thing happening in EFL. I need more reviewers.
 But big patch makes review even slower and you won't get correct
 review for your big patch.

 Just separate your job.
 Do not split the patch after you created a big patch.
 While you work, consider separating patches and commits.
 Thank you.

Yes, I should get consideration for reviewers. :D

2012/3/22 Carsten Haitzler ras...@rasterman.com:
 actually realistically, review here in efl is pretty fast. you should try lots
 of other open source projects where patches get lost/ignored for YEARS :)

Right, I need to practice mind control. :)

--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] [Patch][elementary] elc_ctxpopup using elm_list internally.

2012-03-21 Thread The Rasterman
On Wed, 21 Mar 2012 18:19:26 +0900 cnook kimci...@gmail.com said:

 Dear All, Hello.
 
 The patch is for using elm_list as a item controller of elc_ctxpopup.
 I have learned that there was an opinion to change the internal structure
 of ctxpopup using the list.
 It would be able to match up to your expectation. but it is just a draft
 version of using list in ctxpopup.
 So, It would be pretty good to me if you give any feedback on this. Thanks.
 
 Sincerely,
 Shinwoo Kim.

change:
  (wd-list  !wd-list_visible)
to
  ((wd-list)  (!wd-list_visible))

change:
  if(list_size.x = rect.w || list_size.y = rect.h)
to
  if ((list_size.x = rect.w) || (list_size.y = rect.h))

-- 
- Codito, ergo sum - I code, therefore I am --
The Rasterman (Carsten Haitzler)ras...@rasterman.com


--
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] [Patch][elm_list] bug fix: crash occurs when container which contains the elm_list is deleted on the elm_list item clicked callback

2012-03-21 Thread The Rasterman
On Thu, 22 Mar 2012 09:51:49 +0900 Kim Shinwoo kimcinoo@gmail.com said:

in svn it is! tnx! :)

 OMG. Thanks for your response. You're definitely right.
 I have attached revised patch. Please review the patch.
 
 Anyhow.. Are you alive?!!
 
 
 2012/3/21 Hyoyoung Chang hyoyo...@gmail.com:
  I think previous your mail is in another your mail account :P
  btw, i have a question.
 
  +        if (it-icon)
  +           evas_object_event_callback_del(it-icon,
  +                                          EVAS_CALLBACK_CHANGED_SIZE_HINTS,
  +                                          _changed_size_hints);
  +        if (it-end)
  +           evas_object_event_callback_del(it-icon,
  +                                          EVAS_CALLBACK_CHANGED_SIZE_HINTS,
  +                                          _changed_size_hints);
 
  in second callback_del, it-icon should be it-end?
 
  On Wed, Mar 21, 2012 at 5:53 PM, cnook kimci...@gmail.com wrote:
  Dear All, Hello.
 
  I have resolved the elm_list issue(goto ELM_LIST_CRASH_ISSUE_MAIL below, I
  sent mail but I cannot find the previous mail -_-;).
  It would be related with double free issue. In the sub-obejct-del
  callback which name is _sub_del() sets item-icon to NULL.
  And in the EVAS_CALLBACK_MOUSE_UP callback funtion which name is
  _mouse_up() calls evas_object_unref();
  It would try to free the item-icon but the item-icon is already set to
  NULL. So.. the crash would occur..
 
  Anyhow my solutiion is the following.
 
  static void
  _del_pre_hook(Evas_Object *obj)
  {
   Widget_Data *wd = elm_widget_data_get(obj);
   const Eina_List *l;
   Elm_List_Item *it;
 
   evas_object_smart_callback_del(obj, sub-object-del, _sub_del);
 
   if (!wd) return;
 
   EINA_LIST_FOREACH(wd-items, l, it)
     {
        if (it-icon)
           evas_object_event_callback_del(it-icon,
                                          EVAS_CALLBACK_CHANGED_SIZE_HINTS,
                                          _changed_size_hints);
        if (it-end)
           evas_object_event_callback_del(it-icon,
                                          EVAS_CALLBACK_CHANGED_SIZE_HINTS,
                                          _changed_size_hints);
     }
 
   evas_object_event_callback_del(wd-scr,
                                  EVAS_CALLBACK_CHANGED_SIZE_HINTS,
                                  _changed_size_hints);
   evas_object_event_callback_del(wd-box,
                                  EVAS_CALLBACK_CHANGED_SIZE_HINTS,
                                  _changed_size_hints);
  }
 
  I'm not sure this is the best way to resolve the issue. Anyhow the crash
  does not occur.
  I have checked this issue on the popup, ctxpopup and box already.
  Please let me know, if you have any other proper ideas to reslove below
  issue.. Thanks.
 
 
 
 
  ELM_LIST_CRASH_ISSUE_MAIL:
 
  Dear All, Hello.
 
  If the elm_list is contained in the container - can be whatever containers
  such as box, popup, ctxpopup and so on -
  which has item with icon object such as a elm_radio and evas_object_del()
  is called in the item select callback, the application is crashed.
  If there is no icon on the elm_list item, the crash does not occur. The
  example to occur the crash as the following.
 
 
  {{{
  static void _filter_all_cb(void *data, Evas_Object *obj,
               void *event_info)
  {
    Evas_Object *popup = (Evas_Object *)data;
    evas_object_del(popup);
  }
 
  static void
  _popup_center_text_cb(void *data, Evas_Object *obj __UNUSED__,
                      void *event_info __UNUSED__)
  {
   Evas_Object *popup, *btn1,*list, *radio;
 
   popup = elm_popup_add(data);
   evas_object_size_hint_weight_set(popup, EVAS_HINT_EXPAND,
  EVAS_HINT_EXPAND);
 
    btn1 = elm_button_add(popup);
    elm_object_text_set(btn1, OK);
    elm_object_part_content_set(popup, button1, btn1);
    evas_object_smart_callback_add(btn1, clicked, NULL, NULL);
 
    list = elm_list_add(popup);
    evas_object_size_hint_weight_set(list, EVAS_HINT_EXPAND,
            EVAS_HINT_EXPAND);
    evas_object_size_hint_align_set(list, EVAS_HINT_FILL, EVAS_HINT_FILL);
    elm_list_mode_set(list, ELM_LIST_EXPAND);
    evas_object_show(list);
 
    Evas_Object *group;
    group = radio = elm_radio_add(list);
    evas_object_size_hint_weight_set(radio, EVAS_HINT_EXPAND,
            EVAS_HINT_EXPAND);
    elm_list_item_append(list, Radio, radio, NULL, _filter_all_cb, popup);
    evas_object_show(radio);
 
    elm_object_content_set(popup, list);
   evas_object_show(popup);
  }
 
  }}}
 
  I will check this issue also, but your help will be pretty good to  me.
  Thanks.
 
  --
  This SF email is sponsosred by:
  Try Windows Azure free for 90 days Click Here
  http://p.sf.net/sfu/sfd2d-msazure
  ___
  enlightenment-devel mailing list
  enlightenment-devel@lists.sourceforge.net