[E-devel] Evas box layout doesn't guarantee items' min size.

[Conrad Um]

I am trying to bring edje box and table to elm.box and elm.table to provide
more features.

Unlike table, each box can have its own layout function, so there are
several issues to adopt edje box at once. So, if there are not significant
differences between evas box layouts (builtin edje box uses them) and elm
box layout, using evas box layouts seems a good choice. By doing that, box
behaviors will be consistent along with evas, edje and elementary. (at this
moment, evas = edje != elm)

BTW, even though we don't make them same, I found that something weird in
evas box behavior. Look at this picture. http://imgur.com/zYk0HjF

Let's assume that there is an evas box which has three items.
The layout function of box is "evas_object_box_layout_horizontal", and
three items are evas rectangles having size hint min width 50px. (Items can
be something different, for example, elm.button) The second and third
item's horizontal weight is EVAS_HINT_EXPAND(1.0). The first one's is 0.

I expected that size hint min width of box would be 150px. (3 * 50px)
However, evas box layout ignores expanded item's size hint min, so expanded
items will become smaller because box doesn't have min size including them.

I think that boxes' behaviors can be different, but size hint min value of
items should be guaranteed except that box is resized forcibly smaller than
its own min size.
What do you think of this? Is this a normal behavior for evas box?

Regards,
conr2d
--
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=272487151=/4140
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Re: [E-devel] EFL interface change - Animator

[Tom Hacohen]

On 24/02/16 10:35, Tom Hacohen wrote:
> On 04/02/16 15:58, marcel-hollerb...@t-online.de wrote:
>> Hi,
>>
>> On Thu, Feb 04, 2016 at 03:48:17PM +, Tom Hacohen wrote:
>>> On 04/02/16 15:18, Hermet Park wrote:
 my point is not  EINA_UNUSED nor animator.

 As you mentioned, event_info is used for sometimes.

 How many scenarios will use those event_info and desc in the future?
 Im worring about our code is getting more long and dirty because of
 this.
 See our evas_object_smart_callback and evas_object_event_callback
 function prototypes.

 I'm curious if we could provide simpler version.
>>>
>>> event_info: a lot.
>>> desc: debatable, though maybe bindings will make good use of it to ease
>>> attaching to callbacks. We are not entirely sure, but we wanted it
>>> because we had some cases in mind (that I don't remember at the moment)
>>> where we thought this could prove useful.
>>>
>>> smart callbacks have event_info, and it's useful, so I don't see your
>>> point.
>>
>> Why not having something like
>>
>> _event_cb(void *data, Event *ev) {...}
>>
>> with
>>
>> typedef struct {
>> Eo *obj
>> Eo_Event_Description2 *desc
>> void *event_info
>> } Event;
>>
>> I wouldnt put *data in this structure since its more often used than
>> obj/desc/event_info.
>>
>> Greetings bu5hm4n
>
> Sorry for the long delay.
>
> Since there has been no objections, and I managed to convince Carsten,
> we are going to go forward with this suggestion almost the way you
> suggested it. The only different is that data is going to be the second
> parameter. :)
>
> I plan on working on it when I'm back from Korea (next week).
>
> Thanks again for the good idea. :P

OK, following comments by bu5hm4n data is going to be first.

--
Tom.

--
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=272487151=/4140
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Re: [E-devel] embryo define checking

[Mike Blumenkrantz]

I am trying to avoid using the efl version defines because they don't
indicate if it's the prerelease from git, meaning that the features aren't
usable until the release occurs; not very helpful if I want a feature to be
active now when it's supported.

It seems like this should be something available; perhaps we need to
#define EFL_VERSION_MICRO for it?

On Wed, Feb 24, 2016 at 5:49 PM Carsten Haitzler 
wrote:

> On Wed, 24 Feb 2016 17:44:42 + Mike Blumenkrantz
>  said:
>
> > It's possible to #define constants in embryo, such as ones used for
> version
> > checking. I haven't found any reference for how to detect them in a
> script,
> > however. Any #if directive gets handled by epp during edje compilation,
> and
> > using other statements involving the constant will result in compile
> > failure due to undefined references.
> >
> > Is there any sample code for this in existence?
> yeah... that's a problem. epp gets to the #... before embryo_cc ... so
> there is
> no way to do #if you CAN use defines from default.inc as they expand
> later
> but you cant #if etc. ... the best way to do this is use EFL_VERSION_MAJOR
> and
> EFL_VERSION_MINOR that are defined and know that efl is distributed  and
> required to be "in step" so
>
> #if (EFL_VERSION_MAJOR >= 1) && (EFL_VERSION_MINOR >= 18)
> ...
> #endif
>
> for example :)
>
> --
> - Codito, ergo sum - "I code, therefore I am" --
> The Rasterman (Carsten Haitzler)ras...@rasterman.com
>
>
--
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=272487151=/4140
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Re: [E-devel] embryo define checking

[The Rasterman]

On Wed, 24 Feb 2016 17:44:42 + Mike Blumenkrantz
 said:

> It's possible to #define constants in embryo, such as ones used for version
> checking. I haven't found any reference for how to detect them in a script,
> however. Any #if directive gets handled by epp during edje compilation, and
> using other statements involving the constant will result in compile
> failure due to undefined references.
> 
> Is there any sample code for this in existence?
yeah... that's a problem. epp gets to the #... before embryo_cc ... so there is
no way to do #if you CAN use defines from default.inc as they expand later
but you cant #if etc. ... the best way to do this is use EFL_VERSION_MAJOR and
EFL_VERSION_MINOR that are defined and know that efl is distributed  and
required to be "in step" so

#if (EFL_VERSION_MAJOR >= 1) && (EFL_VERSION_MINOR >= 18)
...
#endif

for example :)

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


--
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=272487151=/4140
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

[E-devel] embryo define checking

[Mike Blumenkrantz]

It's possible to #define constants in embryo, such as ones used for version
checking. I haven't found any reference for how to detect them in a script,
however. Any #if directive gets handled by epp during edje compilation, and
using other statements involving the constant will result in compile
failure due to undefined references.

Is there any sample code for this in existence?
--
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=272487151=/4140
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Re: [E-devel] EFL interface change - Animator

[Tom Hacohen]

On 04/02/16 15:58, marcel-hollerb...@t-online.de wrote:
> Hi,
>
> On Thu, Feb 04, 2016 at 03:48:17PM +, Tom Hacohen wrote:
>> On 04/02/16 15:18, Hermet Park wrote:
>>> my point is not  EINA_UNUSED nor animator.
>>>
>>> As you mentioned, event_info is used for sometimes.
>>>
>>> How many scenarios will use those event_info and desc in the future?
>>> Im worring about our code is getting more long and dirty because of this.
>>> See our evas_object_smart_callback and evas_object_event_callback function 
>>> prototypes.
>>>
>>> I'm curious if we could provide simpler version.
>>
>> event_info: a lot.
>> desc: debatable, though maybe bindings will make good use of it to ease
>> attaching to callbacks. We are not entirely sure, but we wanted it
>> because we had some cases in mind (that I don't remember at the moment)
>> where we thought this could prove useful.
>>
>> smart callbacks have event_info, and it's useful, so I don't see your point.
>
> Why not having something like
>
> _event_cb(void *data, Event *ev) {...}
>
> with
>
> typedef struct {
> Eo *obj
> Eo_Event_Description2 *desc
> void *event_info
> } Event;
>
> I wouldnt put *data in this structure since its more often used than
> obj/desc/event_info.
>
> Greetings bu5hm4n

Sorry for the long delay.

Since there has been no objections, and I managed to convince Carsten, 
we are going to go forward with this suggestion almost the way you 
suggested it. The only different is that data is going to be the second 
parameter. :)

I plan on working on it when I'm back from Korea (next week).

Thanks again for the good idea. :P

--
Tom.


--
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=272487151=/4140
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Re: [E-devel] [EGIT] [core/efl] master 01/01: edje_cc: abort compile when unnamed part exists

[The Rasterman]

On Wed, 24 Feb 2016 13:53:48 +0900 (KST) Hermet Park  said:

> I think this changes edc grammar.
> only one part may not need a part name.
> and App edc will be broken after this patch.
> 
> Do you really like it?

i wrote edje_cc - i never intended or especterd people to NOt name parts. ihave
NEVER written edc without naming a part. it's pretty much expected to be named
because name is then used in signals - eg mouse in.out,down,up ans the source.

the fact edje_cc didn't CHECK and tell you/abort/warn is indeed an oversight,
but it was never INTENDED to be able to not name parts. you need the names to
do inheritance or a swallows to as anything that is a src of a signal.

on the other hand i do get hermet's point. maybe we can do a sleep(30); there
with the warning - their stuff won't break but their compiles will slow down
with a very noticeable warning there in the logs. later it can become a fatal
error.

> -Original Message-
> From: "Viacheslav Reutskiy" 
> To: "Enlightenment developer
> list"; Cc: 
> Sent: 2016-02-23 (화) 18:13:30
> Subject: Re: [E-devel] [EGIT] [core/efl] master 01/01: edje_cc: abort compile
> when unnamed part exists 
> Crap, I made a mistake. I made 'arc land' instead 'arc close-revision'.
> How to fix it? Revert and push correctly?
> 
> 
> On 02/23/2016 11:06 AM, Vyacheslav Reutskiy wrote:
> > rimmed pushed a commit to branch master.
> >
> > http://git.enlightenment.org/core/efl.git/commit/?id=81f8e8683d90435c6321e643d40047de643d5a58
> >
> > commit 81f8e8683d90435c6321e643d40047de643d5a58
> > Author: Vyacheslav Reutskiy 
> > Date:   Tue Feb 23 11:06:55 2016 +0200
> >
> >  edje_cc: abort compile when unnamed part exists
> >  
> >  Summary: abort compile when unnamed part exists.
> >  
> >  Reviewers: raster, reutskiy.v.v
> >  
> >  Reviewed By: reutskiy.v.v
> >  
> >  Subscribers: cedric, jpeg
> >  
> >  Differential Revision: https://phab.enlightenment.org/D3723
> > ---
> >   src/bin/edje/edje_cc_handlers.c  5 +
> >   1 file changed, 5 insertions(+)
> >
> > diff --git a/src/bin/edje/edje_cc_handlers.c
> > b/src/bin/edje/edje_cc_handlers.c index 4ce5a86..fcaad2e 100644
> > --- a/src/bin/edje/edje_cc_handlers.c
> > +++ b/src/bin/edje/edje_cc_handlers.c
> > @@ -14647,6 +14647,11 @@ edje_cc_handlers_hierarchy_pop(void)
> >{
> >   unsigned int i;
> >   
> > +if (!current_part->name)
> > +  {
> > + error_and_abort(NULL, "parse error near %s:%i. Unnamed part
> > exists in Group \"%s\".", file_in, line - 1, current_de->entry);
> > +  }
> > +
> >   for (i = 0; i < current_part->other.desc_count; i++)
> > {
> >if (!current_part->other.desc[i]->state.name)
> >
> 
> -- 
> Viacheslav Reutskiy (rimmed)
> 
> 
> --
> Site24x7 APM Insight: Get Deep Visibility into Application Performance
> APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
> Monitor end-to-end web transactions and take corrective actions now
> Troubleshoot faster and improve end-user experience. Signup Now!
> http://pubads.g.doubleclick.net/gampad/clk?id=272487151=/4140
> ___
> enlightenment-devel mailing list
> enlightenment-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
> --
> Site24x7 APM Insight: Get Deep Visibility into Application Performance
> APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
> Monitor end-to-end web transactions and take corrective actions now
> Troubleshoot faster and improve end-user experience. Signup Now!
> http://pubads.g.doubleclick.net/gampad/clk?id=272487151=/4140
> ___
> 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


--
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=272487151=/4140
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel