Hello,

Sorry for delay.

On 06/09/16 18:29, Cedric BAIL wrote:
> On Jun 9, 2016 5:02 AM, "Viacheslav Reutskiy" <reutskiy....@gmail.com>
> wrote:
>> Hello,
>>
>> Cedric, you broke the edje_edit. Edje_edit cann't work with unnamed parts.
> Ah, dang ! Problem is that this was legit in the past and application are
> using it (which is why I removed it).it also means that edje edit is not
> robust on valid older input.Would it be possible to patch edje edit, so
> that on open, it patches the edj and generate name as needed ? This way old
> flee could be opened by edje edit property.
>
> Cedric

I can set generated name for unnamed parts, but it's wrong. As you said me:
"It's broke ABI".
Maybe pospone this changes to EFL 2.0? For EFL 2.0 we can rework the Edje Edit 
to
manage all items (parts, programs, items etc) by index for solve all issues.

---
Viacheslav Reutskiy (rimmed)

>> Please revert this patch.
>>
>>
>> On 06/09/16 01:07, Cedric BAIL wrote:
>>> cedric pushed a commit to branch master.
>>>
>>>
> http://git.enlightenment.org/core/efl.git/commit/?id=088043e77af158941d9240a14f3463387f8eabfb
>>> commit 088043e77af158941d9240a14f3463387f8eabfb
>>> Author: Cedric BAIL <ced...@osg.samsung.com>
>>> Date:   Wed Jun 8 15:00:30 2016 -0700
>>>
>>>       edje: unamed part are completely legit as long as no program try
> to interact with them.
>>> ---
>>>    src/bin/edje/edje_cc_handlers.c | 3 +--
>>>    1 file changed, 1 insertion(+), 2 deletions(-)
>>>
>>> diff --git a/src/bin/edje/edje_cc_handlers.c
> b/src/bin/edje/edje_cc_handlers.c
>>> index 0435713..0bbdb13 100644
>>> --- a/src/bin/edje/edje_cc_handlers.c
>>> +++ b/src/bin/edje/edje_cc_handlers.c
>>> @@ -15079,9 +15079,8 @@ edje_cc_handlers_hierarchy_pop(void)
>>>
>>>            if (!current_part->name)
>>>              {
>>> -             ERR("Parse error near %s:%i. Unnamed part exists in Group
> \"%s\" - Pausing for a bit so you notice and remember to fix this.",
>>> +             WRN("Parse error near %s:%i. Unnamed part exists in Group
> \"%s\".",
>>>                     file_in, line - 1, current_de->entry);
>>> -             sleep(10);
>>>              }
>>>
>>>            for (i = 0; i < current_part->other.desc_count; i++)
>>>
>> --
>> Viacheslav Reutskiy (rimmed)
>>
>>
> ------------------------------------------------------------------------------
>> What NetFlow Analyzer can do for you? Monitors network bandwidth and
> traffic
>> patterns at an interface-level. Reveals which users, apps, and protocols
> are
>> consuming the most bandwidth. Provides multi-vendor support for NetFlow,
>> J-Flow, sFlow and other flows. Make informed decisions using capacity
>> planning reports. https://ad.doubleclick.net/ddm/clk/305295220;132659582;e
>> _______________________________________________
>> enlightenment-devel mailing list
>> enlightenment-devel@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
>>
> ------------------------------------------------------------------------------
> What NetFlow Analyzer can do for you? Monitors network bandwidth and traffic
> patterns at an interface-level. Reveals which users, apps, and protocols are
> consuming the most bandwidth. Provides multi-vendor support for NetFlow,
> J-Flow, sFlow and other flows. Make informed decisions using capacity
> planning reports. https://ad.doubleclick.net/ddm/clk/305295220;132659582;e
> _______________________________________________
> enlightenment-devel mailing list
> enlightenment-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

-- 
Viacheslav Reutskiy (rimmed)

------------------------------------------------------------------------------
What NetFlow Analyzer can do for you? Monitors network bandwidth and traffic
patterns at an interface-level. Reveals which users, apps, and protocols are 
consuming the most bandwidth. Provides multi-vendor support for NetFlow, 
J-Flow, sFlow and other flows. Make informed decisions using capacity 
planning reports. https://ad.doubleclick.net/ddm/clk/305295220;132659582;e
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to