Hi, Raster.
I am sorry. The previous patch has simple typo.
Please apply the attached patch for fixing it.
Thanks in advance.

-----Original Message-----
From: Carsten Haitzler (The Rasterman) [mailto:ras...@rasterman.com] 
Sent: Thursday, July 07, 2011 3:24 PM
To: Jihoon Kim
Cc: '이지훈'; enlightenment-devel@lists.sourceforge.net
Subject: Re: [E-devel] [PATCH] add ecore_imf_context_cursor_location_set API

On Thu, 07 Jul 2011 09:55:04 +0900 Jihoon Kim <jihoon48....@samsung.com> said:

yay! in svn! :) tnx!

> Hi, Raster.
> 
> I send again after fixing this patch according to your comment.
> Would you please review again?
> 
> -----Original Message-----
> From: Carsten Haitzler (The Rasterman) [mailto:ras...@rasterman.com] 
> Sent: Wednesday, July 06, 2011 6:23 PM
> To: Jihoon Kim
> Cc: enlightenment-devel@lists.sourceforge.net; 이지훈
> Subject: Re: [E-devel] [PATCH] add ecore_imf_context_cursor_location_set API
> 
> On Tue, 05 Jul 2011 16:17:12 +0900 Jihoon Kim <jihoon48....@samsung.com>
> said:
> 
> > Hi, EFL developers.
> > 
> > In this patch, it will add ecore_imf_context_cursor_location_set API in
> > Ecore_IMF.
> > 
> > The cursor location can be used to determine the position of candidate
> word
> > window in immodule.
> > 
> > Would you please apply this patch in svn?
> 
> reject! :) actually.. you broke ABI. you added cursor_location_set in the
> middle of the struct.
> 
> 
> -- 
> ------------- Codito, ergo sum - "I code, therefore I am" --------------
> The Rasterman (Carsten Haitzler)    ras...@rasterman.com


-- 
------------- Codito, ergo sum - "I code, therefore I am" --------------
The Rasterman (Carsten Haitzler)    ras...@rasterman.com
Index: src/lib/ecore_imf/ecore_imf_context.c
===================================================================
--- src/lib/ecore_imf/ecore_imf_context.c       (revision 61106)
+++ src/lib/ecore_imf/ecore_imf_context.c       (working copy)
@@ -520,7 +520,7 @@ ecore_imf_context_cursor_location_set(Ecore_IMF_Co
    if (!ECORE_MAGIC_CHECK(ctx, ECORE_MAGIC_CONTEXT))
      {
         ECORE_MAGIC_FAIL(ctx, ECORE_MAGIC_CONTEXT,
-                         "ecore_imf_context_cursor_position_set");
+                         "ecore_imf_context_cursor_location_set");
         return;
      }
    if (ctx->klass->cursor_location_set) ctx->klass->cursor_location_set(ctx, 
x, y, w, h);

------------------------------------------------------------------------------
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

Reply via email to