- In ewl_entry_text_insert(), cursor position should be set _after_ applying 
the insert operation?

Uninitialised e->text fix included as well, in case my previous patch doesn't 
get applied due to just being comment corrections.

Nick
Index: ewl_entry.c
===================================================================
RCS file: /cvsroot/enlightenment/e17/libs/ewl/src/lib/ewl_entry.c,v
retrieving revision 1.7
diff -u -r1.7 ewl_entry.c
--- ewl_entry.c 22 Mar 2005 16:36:35 -0000      1.7
+++ ewl_entry.c 28 Mar 2005 00:09:08 -0000
@@ -146,6 +146,8 @@
 
        if (text)
                ewl_entry_text_set(e, text);
+    else
+        e->text=NULL;
 
@@ -364,14 +369,14 @@
 
        op = ewl_entry_op_text_insert_new(e, text, index);
        len = strlen(text);
-       ewl_entry_cursor_position_set(EWL_ENTRY_CURSOR(e->cursor),
-                                      index + len);
        e->length += len;
 
        ecore_dlist_prepend(e->ops, op);
        if (REALIZED(e))
                ewl_entry_ops_apply(e);
 
+       ewl_entry_cursor_position_set(EWL_ENTRY_CURSOR(e->cursor),
+                                      index + len);
        ewl_widget_configure(EWL_WIDGET(e));
 
        DLEAVE_FUNCTION(DLEVEL_STABLE);

Reply via email to