Hi, Naruto.

I've tested in elementary_test > entry menu after applying my edje patch
related to client_window_set.
However, I couldn't see the preedit string in the entry widget while I was
typing.

In addition, sometimes segmentation fault was occured.
Here is backtrace of GDB.

(gdb) bt
#0  0x0030fb62 in eina_unicode_strlen (ustr=0x0) at eina_unicode.c:70
#1  0x003105dc in eina_unicode_unicode_to_utf8 (uni=0x0, _len=0xbffff184)
    at eina_unicode.c:331
#2  0x00efd7b3 in _ecore_imf_context_xim_preedit_string_get (ctx=0x8251b28,
    str=0xbffff1d8, cursor_pos=0xbffff1dc) at ecore_imf_xim.c:179
#3  0x004736d6 in ecore_imf_context_preedit_string_get (ctx=0x8251b28,
    str=0xbffff1d8, cursor_pos=0xbffff1dc) at ecore_imf_context.c:388
#4  0x004d4c44 in _edje_entry_imf_event_preedit_changed_cb (data=0x8333628,
    type=79, event=0x838f0b8) at edje_entry.c:2887
#5  0x00428ac6 in _ecore_event_call () at ecore_events.c:693
#6  0x0042f4d2 in _ecore_main_loop_iterate_internal (once_only=0)
    at ecore_main.c:1750
#7  0x0042dc49 in ecore_main_loop_begin () at ecore_main.c:848
#8  0x001d08cc in elm_run () at elm_main.c:1075
#9  0x0805551b in elm_main (argc=1, argv=0xbffff3a4) at test.c:489
#10 0x08055554 in main (argc=1, argv=0xbffff3a4) at test.c:498

2011/7/23 Naruto TAKAHASHI <tnar...@gmail.com>

> Hi JihoonKim, and EFL developers.
>
> I attach a patch for fixing some XIM module bugs.
>
>  - fix showing previous preedit string bug.
>  - delete compile warning(thanks JihoonKim)
>  -  fix some sequence issue to send preedit changed event and commit
> event.(thanks JihoonKim)
>
> Please review this patch?
>
> Regards.
>
> 2011年7月22日23:52 Naruto TAKAHASHI <tnar...@gmail.com>:
> > Hi, JihoonKim.
> >
> > I talked to you how fix "preedit draw callback " direction on IRC.
> > I repeat to talk this stuff for didn't see this talk.
> >
> > Your patch behavior is no problem.
> > But Preedit Draw Callback needs insert, delete, and replace by
> > referencing Preedit Draw Callback argument.
> > (detail:
> http://static.cray-cyber.org/Documentation/NEC_SX_R10_1/G1AE02E/CHAP13.HTML#13.18.6
> .
> > Preedit Draw Callback)
> >
> > I'm trying to fix too easy routine by using Eina_UStrBuf, now.
> >
> > So, please wait this.
> > Off course, I merge deleting comple warning of your patch with thanks. :)
> >
> > Thanks.
> >
> > 2011/7/17 Jihoon Kim <imfin...@gmail.com>:
> >> Hi, Naruto.
> >> As I told you last Friday on IRC, I guess there are some bugs in your
> xim
> >> immodule.
> >> For example, in case that I'd like to input '私の' (watasino), the preedit
> >> string was got from your immodule like below
> >> '私のしの'.
> >> In addition, there are some sequence issue to send preedit changed event
> and
> >> commit event.
> >> I've tried to fix this problem and send you the patch.
> >> (I've tested on elementary_test > entry)
> >> If you don't mind uploading this patch to svn, I'll request Maintainers
> to
> >> upload this patch.
> >> Thanks.
> >> On Sun, Jul 10, 2011 at 2:51 AM, Naruto TAKAHASHI <tnar...@gmail.com>
> wrote:
> >>>
> >>> Hi Mike.
> >>>
> >>> Thanks, feedback. I merged it to xim/Makefile.am.
> >>>
> >>> I attach a source code for using XIM module debug.
> >>> This program can check a below behaviors.
> >>>
> >>>  - toggle enable and disable XIM
> >>>  - commit string from XIM
> >>>
> >>> Another test, by using Desktop Entry Editor's text field.
> >>> (Enlightenment Main->Settings->Settings Panel->New Application)
> >>>
> >>> When executing test program, set ECORE_IMF_MODULE=xim.
> >>>
> >>> Thanks.
> >>>
> >>> 2011/7/8 Mike McCormack <mj.mccorm...@samsung.com>:
> >>> > On 07/08/2011 03:15 PM, Naruto TAKAHASHI wrote:
> >>> >> Hi, All.
> >>> >>
> >>> >> I attached some patches of XIM module of ecore_imf.
> >>> >>
> >>> >> As far as I know, EFL has not having official ecore_imf module in E
> >>> >> repository. And ecore_x has XIM code but is unavailable condition.
> >>> >> So I moved XIM code of ecore_x as ecore_imf module.
> >>> >>
> >>> >> please review this patches.
> >>> >
> >>> > Hello Naruto,
> >>> >
> >>> > Looks like nice work.
> >>> >
> >>> > I applied your patches to my ecore and built, but there was a build
> >>> > error (fix below):
> >>> >
> >>> > make[5]: Entering directory
> >>> > `/home/mike/git/e/ecore/src/modules/immodules/xim'
> >>> >  CC     xim_la-ecore_imf_xim.lo
> >>> > ecore_imf_xim.c:5:18: error: Evas.h: No such file or directory
> >>> > ecore_imf_xim.c: In function ‘_ecore_x_event_reverse_locks’:
> >>> > ecore_imf_xim.c:359: warning: suggest braces around empty body in an
> >>> > ‘if’ statement
> >>> > ecore_imf_xim.c: In function ‘preedit_start_callback’:
> >>> > ecore_imf_xim.c:662: warning: unused parameter ‘xic’
> >>> > ecore_imf_xim.c:664: warning: unused parameter ‘call_data’
> >>> >
> >>> > How can I test it?
> >>> >
> >>> > thanks,
> >>> >
> >>> > Mike
> >>> >
> >>> >
> >>> >
> >>> > diff --git a/ecore/src/modules/immodules/xim/Makefile.am
> >>> > b/ecore/src/modules/immodules/xim/Makefile.am
> >>> > index 006035c..cc0682d 100644
> >>> > --- a/ecore/src/modules/immodules/xim/Makefile.am
> >>> > +++ b/ecore/src/modules/immodules/xim/Makefile.am
> >>> > @@ -12,6 +12,7 @@ AM_CPPFLAGS = \
> >>> >  -I$(top_builddir)/src/lib/ecore_imf \
> >>> >  -DPACKAGE_LIB_DIR=\"$(libdir)\" \
> >>> >  -DPACKAGE_DATA_DIR=\"$(datadir)/$(PACKAGE)\" \
> >>> > +@EVAS_CFLAGS@ \
> >>> >  @EINA_CFLAGS@
> >>> >
> >>> >  pkgdir = $(libdir)/ecore/immodules
> >>> >
> >>> >
> >>> >
> >>> >
> ------------------------------------------------------------------------------
> >>> > All of the data generated in your IT infrastructure is seriously
> >>> > valuable.
> >>> > Why? It contains a definitive record of application performance,
> >>> > security
> >>> > threats, fraudulent activity, and more. Splunk takes this data and
> makes
> >>> > sense of it. IT sense. And common sense.
> >>> > http://p.sf.net/sfu/splunk-d2d-c2
> >>> > _______________________________________________
> >>> > enlightenment-devel mailing list
> >>> > enlightenment-devel@lists.sourceforge.net
> >>> > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
> >>> >
> >>>
> >>>
> >>>
> >>> --
> >>> Naruto TAKAHASHI
> >>> tnar...@gmail.com
> >>>
> >>>
> >>>
> ------------------------------------------------------------------------------
> >>> All of the data generated in your IT infrastructure is seriously
> valuable.
> >>> Why? It contains a definitive record of application performance,
> security
> >>> threats, fraudulent activity, and more. Splunk takes this data and
> makes
> >>> sense of it. IT sense. And common sense.
> >>> http://p.sf.net/sfu/splunk-d2d-c2
> >>> _______________________________________________
> >>> enlightenment-devel mailing list
> >>> enlightenment-devel@lists.sourceforge.net
> >>> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
> >>>
> >>
> >>
> >
> >
> >
> > --
> > Naruto TAKAHASHI
> > tnar...@gmail.com
> >
>
>
>
> --
> Naruto TAKAHASHI
> tnar...@gmail.com
>
------------------------------------------------------------------------------
Storage Efficiency Calculator
This modeling tool is based on patent-pending intellectual property that
has been used successfully in hundreds of IBM storage optimization engage-
ments, worldwide.  Store less, Store more with what you own, Move data to 
the right place. Try It Now! http://www.accelacomm.com/jaw/sfnl/114/51427378/
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to