Re: [E-devel] Edje Text in box with dynamic hints

[The Rasterman]

On Sat, 23 Jul 2011 22:47:31 +0200 Hugo Camboulive 
said:

you basically can't. there isn't a way to force a recalc when using boxes in
edje. as i said to you before - they were not implemented completely - well
tables were not, and i had to fix up the implementation to do the e17 calendar
bit for clock. and groups are only partly useufl as there is a "leaky barrier"
between a child group and its parent. some things pass through., some don't.
some things work as expected, some don't. this is one of those things that
doesn't.

we'd need to add an explicit "recalc" action. that will also only partly help
as if you do trabnsitions and u want the transition to affect the box too u
have to recalc every frame, and thats making things more complex as there u
want not an action but a property that says "from now on every time something
changes recalc min size until this property is removed".

> Well, how can you center two pieces of text "as one" then? It does not sound
> like a very extravagant request to me ^^
> Could you force recalculating on a specific demand?
> 
> On Sat, Jul 23, 2011 at 6:16 AM, Carsten Haitzler wrote:
> 
> > On Thu, 21 Jul 2011 14:58:53 +0200 Hugo Camboulive <
> > hugo.camboul...@gmail.com>
> > said:
> >
> > basically you can't do what you are doing. once you have a group as a
> > child,
> > that child group will never have its min size recalculated after load. so
> > that
> > means programs later on wont change that. it's basically not possible to do
> > as
> > the expense of recalculating every time you do something will bring
> > everything
> > to a crawl.
> >
> > > Hi everyone,
> > >
> > > I encoutered a problem with the text.min / text.max attributes in edje,
> > > which (I think) are supposed to define the hints depending on the length
> > of
> > > the text itself.
> > >
> > > To center a block of 2 texts, I put two text parts in a box.
> > >
> > > It works fine with text defined in the edc (see first 2 lines in
> > screenshot
> > > included, the text is centered), but when I update the text in the
> > > program (third line), the objects keep the same positions.
> > > Maybe the hints are not updated on edje_object_part_text_set, or maybe
> > the
> > > box layout is not refreshed... I don't know :)
> > >
> > > Included is a sample program+edc and a screenshot of how it looks, I've
> > > heard it's the same in trunk (you may need to change the text size to 20
> > or
> > > smaller)
> > >
> > > Regards
> > >
> > > Hugo Camboulive
> >
> >
> > --
> > - Codito, ergo sum - "I code, therefore I am" --
> > The Rasterman (Carsten Haitzler)ras...@rasterman.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
> 


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


--
Magic Quadrant for Content-Aware Data Loss Prevention
Research study explores the data loss prevention market. Includes in-depth
analysis on the changes within the DLP market, and the criteria used to
evaluate the strengths and weaknesses of these DLP solutions.
http://www.accelacomm.com/jaw/sfnl/114/51385063/
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

[E-devel] elm tooltip's big bug

[Mike Blumenkrantz]

ELM tooltip has a rather obvious bug: if the tooltip is larger than the window
it is anchored to, it get clipped to the window. This is because the tooltip
gets affixed to the same evas that its anchor is on, so it cannot freely expand.

I have started working on a fix, involving using a buffer canvas for the
tooltip itself as in e_pointer (thanks to dh for his very reasonably-priced evas
consulting services), but ran into a problem. The content provider callback
passes the tooltip's anchor object to the user, and this anchor is on the wrong
canvas!

Comments/solutions for this bug: go!
-- 
Mike Blumenkrantz
Zentific: Coding in binary since '10.

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

Re: [E-devel] E SVN: andreas IN trunk/BINDINGS/cxx/ecorexx: include/ecorexx src

[Andreas Volz]

Am Sat, 23 Jul 2011 08:14:55 +0200 (CEST) schrieb Vincent Torri:

> 
> 
> On Fri, 22 Jul 2011, Enlightenment SVN wrote:
> 
> > Log:
> > Ecorexx::Timer design changed to have a factory and provide destroy
> > function to wrap C interface better than before also used sigc++
> > slots as callback provider
> 
> what about using shared_ptr for your factory ?

I thought about it. Even at elementaryxx where I use the same design.
I've that CountedPtr class in elmxx and use it at some places. But for
this use cases I decided against it. I don't remember the exact reason
for elementaryxx...

But at the end the C object is killing itself. Also a shared smart
pointer doesn't solve all problems in my eyes. It could solve the
problem that after some point I have an dangerous pointer which points
nowhere as in the C interface. But this comes with some other negative
side effects in the interface.

But if you've ideas how to solve it better please come up with a better
implementation to discuss it. I'm really open for this. :-)

regards
Andreas

-- 
Technical Blog 

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

Re: [E-devel] Edje Text in box with dynamic hints

[Hugo Camboulive]

Well, how can you center two pieces of text "as one" then? It does not sound
like a very extravagant request to me ^^
Could you force recalculating on a specific demand?

On Sat, Jul 23, 2011 at 6:16 AM, Carsten Haitzler wrote:

> On Thu, 21 Jul 2011 14:58:53 +0200 Hugo Camboulive <
> hugo.camboul...@gmail.com>
> said:
>
> basically you can't do what you are doing. once you have a group as a
> child,
> that child group will never have its min size recalculated after load. so
> that
> means programs later on wont change that. it's basically not possible to do
> as
> the expense of recalculating every time you do something will bring
> everything
> to a crawl.
>
> > Hi everyone,
> >
> > I encoutered a problem with the text.min / text.max attributes in edje,
> > which (I think) are supposed to define the hints depending on the length
> of
> > the text itself.
> >
> > To center a block of 2 texts, I put two text parts in a box.
> >
> > It works fine with text defined in the edc (see first 2 lines in
> screenshot
> > included, the text is centered), but when I update the text in the
> > program (third line), the objects keep the same positions.
> > Maybe the hints are not updated on edje_object_part_text_set, or maybe
> the
> > box layout is not refreshed... I don't know :)
> >
> > Included is a sample program+edc and a screenshot of how it looks, I've
> > heard it's the same in trunk (you may need to change the text size to 20
> or
> > smaller)
> >
> > Regards
> >
> > Hugo Camboulive
>
>
> --
> - Codito, ergo sum - "I code, therefore I am" --
> The Rasterman (Carsten Haitzler)ras...@rasterman.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

Re: [E-devel] E SVN: hermet trunk/evas/src/lib/canvas

[The Rasterman]

On Sat, 23 Jul 2011 13:30:57 +0900 ChunEon Park said:

we can't... because we'd break a stable api. :)

> Thank you for replying me. 
> But I think, the usage look doesn't better to users while using the evas_map
> APIs still. For example, 
> Even a user wrote a function to rotate any objects by using evas map, 
> it might does not work properly for one of the two cases - image objects and
> normal objects. I guess we can convert the uv value internally by keeping the
> evas_map_uv setting concept for users. 
> 
> Let's run together for the best moment!
>  -Regards, Hermet-
>  
> -Original Message-
> From: "Carsten Haitzler" 
> To: "ChunEon Park"
> Cc: enlightenment-devel@lists.sourceforge.net;
> enlightenment-...@lists.sourceforge.net Sent: 11-07-23(토) 12:58:51
> Subject: Re: [E-devel] E SVN: hermet trunk/evas/src/lib/canvasOn Sat, 23 Jul
> 2011 12:47:29 +0900 ChunEon Park said: because the
> source image3 IS the image. it isnt pre-rendered at a different size. for
> other objects evas has to pre-render at the object geometry. for images it
> can go straight to the image.
> > I reverted below changes because of the pageflip test case. 
> > But need to confirm that why evas_object_image is...
> > Is there any reason?
> > Otherwise, I prefer to change the concept of the evas_object_image map uv
> > set. Please somebody describe to me. ( --> raster :) ) 
> > 
> > Let's run together for the best moment!
> > -Regards, Hermet-
> > 
> > -Original Message-
> > From: "Enlightenment SVN" 
> > To: enlightenment-...@lists.sourceforge.net
> > Cc: 
> > Sent: 11-07-22(금) 14:24:16
> > Subject: E SVN: hermet trunk/evas/src/lib/canvasLog:
> > evas/evas_object_image - modified to keep the uv consistency with normal
> > evas_object. 
> > Although the uv range of normal object is be localized to the it's
> > geometry, 
> > 
> > but the uv range of the image object is be localized it's image size but not
> > geometry size. 
> > In that case, user should find out the objects type and set the uv value
> > differently between normal and image 
> > why they do that?
> > 
> > 
> > Author: hermet
> > Date: 2011-07-21 22:24:16 -0700 (Thu, 21 Jul 2011)
> > New Revision: 61577
> > Trac: http://trac.enlightenment.org/e/changeset/61577
> > Modified:
> > trunk/evas/src/lib/canvas/evas_object_image.c 
> > Modified: trunk/evas/src/lib/canvas/evas_object_image.c
> > ===
> > --- trunk/evas/src/lib/canvas/evas_object_image.c 2011-07-22 01:48:20 UTC
> > (rev 61576) +++ trunk/evas/src/lib/canvas/evas_object_image.c 2011-07-22
> > 05:24:16 UTC (rev 61577) @@ -2657,8 +2657,8 @@
> > pt->fx = p->px;
> > pt->fy = p->py;
> > pt->fz = p->z;
> > - pt->u = ((p->u * imagew) / uvw) * FP1;
> > - pt->v = ((p->v * imageh) / uvh) * FP1;
> > + pt->u = ((p->u / obj->cur.geometry.w) * imagew) * FP1;
> > + pt->v = ((p->v / obj->cur.geometry.h) * imageh) * FP1;
> > if (pt->u < 0) pt->u = 0;
> > else if (pt->u > (imagew * FP1)) pt->u = (imagew * FP1);
> > if (pt->v < 0) pt->v = 0;
> > --
> > 10 Tips for Better Web Security
> > Learn 10 ways to better secure your business today. Topics covered include:
> > Web security, SSL, hacker attacks & Denial of Service (DoS), private keys,
> > security Microsoft Exchange, secure Instant Messaging, and much more.
> > http://www.accelacomm.com/jaw/sfnl/114/51426210/
> > ___
> > enlightenment-svn mailing list
> > enlightenment-...@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/enlightenment-svn
> > --
> > 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
> -- 
> - Codito, ergo sum - "I code, therefore I am" --
> The Rasterman (Carsten Haitzler) ras...@rasterman.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/1

Re: [E-devel] [Patch] Add XIM module for ecore_imf

[Jihoon Kim]

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=0xb184)
at eina_unicode.c:331
#2  0x00efd7b3 in _ecore_imf_context_xim_preedit_string_get (ctx=0x8251b28,
str=0xb1d8, cursor_pos=0xb1dc) at ecore_imf_xim.c:179
#3  0x004736d6 in ecore_imf_context_preedit_string_get (ctx=0x8251b28,
str=0xb1d8, cursor_pos=0xb1dc) 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=0xb3a4) at test.c:489
#10 0x0804 in main (argc=1, argv=0xb3a4) at test.c:498

2011/7/23 Naruto TAKAHASHI 

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

Re: [E-devel] [Patch] Add XIM module for ecore_imf

[Naruto TAKAHASHI]

Hi, Vencent.

(I forgeted adding e-devel ML address to Cc. So send again.)

> split your patch in 3 patches (the 3 things above). Or at least 2 patches
> (one for the warnings, the other for the fixes).

I split my patch to 2 paches. one for warnings, the other for fixes.

Thanks.

2011/7/23 Vincent Torri :
>
> hey
>
> On Sat, 23 Jul 2011, Naruto TAKAHASHI wrote:
>
>> 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?
>
> split your patch in 3 patches (the 3 things above). Or at least 2 patches
> (one for the warnings, the other for the fixes).
>
> Vincent
>
>> Regards.
>>
>> 2011年7月22日23:52 Naruto TAKAHASHI :
>>>
>>> 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 :

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

Re: [E-devel] segfault of edje_cc during compilation of elementary

[Tomas Cech]

Hi,

On Sat, Jul 23, 2011 at 03:10:15AM +0200, Cedric BAIL wrote:
>Hi,
>
>On Sat, Jul 23, 2011 at 12:55 AM, Tomas Cech  wrote:
>> Hi,
>>
>> On Fri, Jul 22, 2011 at 11:44:53AM +0200, Tomas Cech wrote:
>>>
>>> On Fri, Jul 22, 2011 at 01:20:32AM +0200, Cedric BAIL wrote:

 Hi,

 On Thu, Jul 21, 2011 at 11:47 PM, Tomas Cech  wrote:
>
> I'm trying to create packages for openSUSE once again. I noticed that
> build of elementary fails on segfault of edje_cc.
>
> Log of the build can be found here:
> http://goo.gl/xTTuj
>
> I tried to analyze coredump of this segfault and it looks like:
>
> Program received signal SIGSEGV, Segmentation fault.
> 0x7724f236 in evas_object_image_add (e=)
> at evas_object_image.c:147
> 147        o->cur.cspace =
>
> obj->layer->evas->engine.func->image_colorspace_get(obj->layer->evas->engine.data.output,
> (gdb) bt
> #0  0x7724f236 in evas_object_image_add (e= out>) at evas_object_image.c:147
> #1  0x0040441a in data_write_images
> (ef=0xa00580,image_num=0x7fffbc04, input_bytes=0x7fffbc0c,
>     input_raw_bytes=0x7fffbc08) at edje_cc_out.c:519
> #2  0x004059ad in data_write () at edje_cc_out.c:1084
> #3  0x0040343f in main (argc=8, argv=) at
> edje_cc.c:217
> (gdb) p obj->layer->evas->engine
> $5 = {module = 0x0, func = 0x0, data = {output = 0x0, context = 0x0},
> info = 0x0, info_magic = 0}

 Sounds like you don't have buffer engine installed for evas.
 What would be interesting is evas build log.
>>
>> Buffer engine is installed and successfully loaded. I'm unable to
>> find, where EcoreEvas->Evas->engine gets initialized.
>
>How do you know that the engine is successfully loaded ? Did you try
>running edje_cc on top of strace ? I would be interested to know what
>it tell.

I walked through the code and got here:

Breakpoint 4, eina_module_load (m=0x6902d0) at
eina_amalgamation.c:15511
15511  DBG("successfully loaded %s", m->file);
(gdb) p m->file
$64 = 0x6902dc
"/usr/lib64/evas/modules/engines/buffer/linux-gnu-x86_64-1.0.1/module.so"

>
>> I got to 'ecore_evas_buffer_allocfunc_new', where is
>>  557    ee->evas = evas_new();
>>
>> but this evas object doesn't have engine initialized:
>> 557        ee->evas = evas_new();
>> (gdb) 558        evas_data_attach_set(ee->evas, ee);
>> (gdb) p ee->evas
>> $60 = (Evas *) 0x660610
>> (gdb) p ee->evas->engine
>> $61 = {module = 0x0, func = 0x0, data = {output = 0x0, context = 0x0},
>> info = 0x0, info_magic = 0}
>>
>> I'd appreciate if someone could tell me where to look.
>
>The function call that will bind the engine to the evas cancas is the
>following one :
> evas_output_method_set(ee->evas, rmethod);
>
>After reading your build log, I don't see anything obviously wrong, so
>try strace that the best advice I can give you and post the output
>when done. Maybe that will us figure out what's going on.

Strace really shown the culprit:

stat("/usr/lib64/evas/modules/engines/buffer/linux-gnu-x86_64-1.0.1/module.so", 
{st_mode=S_IFREG|0755, st_size=18904, ...}) = 0
futex(0x7fb2f25b30e0, FUTEX_WAKE_PRIVATE, 2147483647) = 0
open("/usr/lib64/evas/modules/engines/buffer/linux-gnu-x86_64-1.0.1/module.so", 
O_RDONLY) = 7
read(7, "\177ELF\2\1\1\0\0\0\0\0\0\0\0\0\3\0>\0\1\0\0\0\0\30\0\0\0\0\0\0"..., 
832) = 832
fstat(7, {st_mode=S_IFREG|0755, st_size=18904, ...}) = 0
mmap(NULL, 2115568, PROT_READ|PROT_EXEC, MAP_PRIVATE|MAP_DENYWRITE, 7, 0) = 
0x7fb2ed84c000
fadvise64(7, 0, 2115568, POSIX_FADV_WILLNEED) = 0
mprotect(0x7fb2ed84f000, 2097152, PROT_NONE) = 0
mmap(0x7fb2eda4f000, 8192, PROT_READ|PROT_WRITE, 
MAP_PRIVATE|MAP_FIXED|MAP_DENYWRITE, 7, 0x3000) = 0x7fb2eda4f000
close(7)= 0
mprotect(0x7fb2eda4f000, 4096, PROT_READ) = 0
stat("/usr/lib64/evas/modules/engines/software_generic/linux-gnu-x86_64-1.0.1/module.so",
 0x7fff235b05f0) = -1 ENOENT (No such file or directory)

It also explains where 'image_colorspace_get' is defined when not
redefined in engine.

After adding 'evas-module_engine_software_generic' to Requires field
in the spec file problem vanished. I'll recheck which engines need
this dependency and fix it too.

I'd suggest to print some error message when such condition happen and
add check for not accessing NULL pointer there.

Thanks for your help!

Best regards,

Tomas Cech
Sleep_Walker

--
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://li

Re: [E-devel] [Patch] Add XIM module for ecore_imf

[Vincent Torri]

hey

On Sat, 23 Jul 2011, Naruto TAKAHASHI wrote:


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?


split your patch in 3 patches (the 3 things above). Or at least 2 patches 
(one for the warnings, the other for the fixes).


Vincent


Regards.

2011ǯ7·î22Æü23:52 Naruto TAKAHASHI :

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 :

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

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

Re: [E-devel] [Patch] Add XIM module for ecore_imf

[Naruto TAKAHASHI]

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 :
> 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 :
>> 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  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 :
>>> > 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
>>> ___
>>> enl