Page MenuHomePhabricator

efl (efl-1.22)Milestone
ActivePublic

Details

Is Sprint
Yes

Recent Activity

Yesterday

Herald removed a project from T7499: solve compiler (gcc) warnings between __attribute__ ((__malloc__)) and __attribute__ ((__pure__)): Restricted Project.
Mon, Mar 18, 6:22 PM · efl (efl-1.22)
lauromoura moved T7752: C#: Get rid of all lower-cased methods from Backlog to InProgress on the efl: language bindings board.
Mon, Mar 18, 3:42 PM · Active Work Proposal, Restricted Project, efl: language bindings, efl (efl-1.22)
lauromoura added a revision to T7751: C#: Add back the I prefix to interfaces: Restricted Differential Revision.
Mon, Mar 18, 3:41 PM · Active Work Proposal, Restricted Project, efl: language bindings, efl (efl-1.22)
lauromoura added a revision to T7751: C#: Add back the I prefix to interfaces: D8397: efl-csharp: Add back I prefix for interfaces..
Mon, Mar 18, 3:40 PM · Active Work Proposal, Restricted Project, efl: language bindings, efl (efl-1.22)
lauromoura added a revision to T7752: C#: Get rid of all lower-cased methods: D8396: csharp: rename add/remove event handler.
Mon, Mar 18, 3:38 PM · Active Work Proposal, Restricted Project, efl: language bindings, efl (efl-1.22)
lauromoura triaged T7757: C#: Remove static_cast method as High priority.
Mon, Mar 18, 2:37 PM · Active Work Proposal, Restricted Project, efl: language bindings, efl (efl-1.22)
lauromoura added a comment to T7751: C#: Add back the I prefix to interfaces.

In order to integrate better with the C# ecosystem, we have do adapt. This should not be a big change in current C# code, as we moved on from that old all-interface API scheme.

Mon, Mar 18, 1:53 PM · Active Work Proposal, Restricted Project, efl: language bindings, efl (efl-1.22)
q66 added a comment to T7738: Calling convention for events in eolian.

That works too

Mon, Mar 18, 1:45 PM · Restricted Project, efl (efl-1.22), efl: api
bu5hm4n added a comment to T7738: Calling convention for events in eolian.

Don't get me wrong, but this needs to be decided / done for this release, we cannot wait for new syntax, lets just go now with the syntax that is everywhere, and change it afterwards.

Mon, Mar 18, 1:25 PM · Restricted Project, efl (efl-1.22), efl: api
q66 added a comment to T7738: Calling convention for events in eolian.

I intend to do away with const() as well as ptr() in the current form entirely, so having a new syntax is the better choice for it, besides, immutable by default is a better choice than mutable by default

Mon, Mar 18, 1:15 PM · Restricted Project, efl (efl-1.22), efl: api
bu5hm4n added a comment to T7738: Calling convention for events in eolian.

Maybe we should just be able to have klass-types const'able... I would not create another new syntax for it tbh.

Mon, Mar 18, 12:59 PM · Restricted Project, efl (efl-1.22), efl: api
q66 added a comment to T7738: Calling convention for events in eolian.

Or maybe, make them const by default and some kind of event tag to make it mutable where needed?

Mon, Mar 18, 12:13 PM · Restricted Project, efl (efl-1.22), efl: api
q66 added a comment to T7738: Calling convention for events in eolian.

Understood. Do we need a way to make objects const when possible?

Mon, Mar 18, 12:13 PM · Restricted Project, efl (efl-1.22), efl: api
bu5hm4n added a comment to T7738: Calling convention for events in eolian.

We need to have objects mutable as this is used in our API, see Efl.Input.Hold / Efl.Input.Event. Properties like processed are required to be set.

Mon, Mar 18, 12:10 PM · Restricted Project, efl (efl-1.22), efl: api
q66 added a comment to T7738: Calling convention for events in eolian.

That sounds good to me. So basically the only types allowed are value types, objects and container types, minus list/iterator; everything is annotated without any qualifier, and everything is immutable and passed by reference, except objects. Maybe we should pass objects by const too? So that only @const methods can be called on it. That would make all event data completely immutable.

Mon, Mar 18, 12:02 PM · Restricted Project, efl (efl-1.22), efl: api
bu5hm4n added a comment to T7738: Calling convention for events in eolian.

When something gets *passed* the type of the input and output is always the same. (where input is a call to efl_event_callback_call, output is the API user that subscribed to a event)
Objects are passed as a normal Eo*, this means event subscribers can call functions on that object.
Iterators & lists should not be allowed as event types.
Structs / buildin-types / containers are *always* passed as const pointer, with exactly ONE level of indirection. The types of hashes and accessors *must* be annotated as const.

Mon, Mar 18, 11:50 AM · Restricted Project, efl (efl-1.22), efl: api
zmike updated subscribers of T7713: some elementary_test cases no longer run.
Mon, Mar 18, 11:44 AM · efl (efl-1.22), efl: widgets
zmike triaged T7756: 'part background' in elementary_test spews lots of errors and doesn't display correctly as High priority.
Mon, Mar 18, 11:42 AM · efl (efl-1.22), efl: widgets
q66 added a comment to T7738: Calling convention for events in eolian.

I'd say that allowing less is always a good thing, as it allows for fewer errors/mistakes, so if we can do away with it, let's do it

Mon, Mar 18, 11:39 AM · Restricted Project, efl (efl-1.22), efl: api
zmike added a comment to T7753: efl.ui.panel example in elementary_test hangs infinitely.

Same with Efl.Ui.Popup.Anchor.

Mon, Mar 18, 11:37 AM · efl (efl-1.22), efl: widgets
segfaultxavi added a comment to T7751: C#: Add back the I prefix to interfaces.

What? Are we going to change the API now?

Mon, Mar 18, 11:35 AM · Active Work Proposal, Restricted Project, efl: language bindings, efl (efl-1.22)
zmike triaged T7755: infinite loop in focus manager on destroy as High priority.
Mon, Mar 18, 11:34 AM · Restricted Project, efl (efl-1.22)
zmike triaged T7754: efl.ui.tags in elm_test triggers CRI error as High priority.
Mon, Mar 18, 11:27 AM · efl (efl-1.22), efl: widgets
zmike triaged T7753: efl.ui.panel example in elementary_test hangs infinitely as High priority.
Mon, Mar 18, 11:22 AM · efl (efl-1.22), efl: widgets
zmike added a revision to T7713: some elementary_test cases no longer run: D8393: elm_code: add theme alias for layout group.
Mon, Mar 18, 11:15 AM · efl (efl-1.22), efl: widgets
lauromoura moved T7751: C#: Add back the I prefix to interfaces from Backlog to InProgress on the efl: language bindings board.
Mon, Mar 18, 11:04 AM · Active Work Proposal, Restricted Project, efl: language bindings, efl (efl-1.22)
lauromoura claimed T7751: C#: Add back the I prefix to interfaces.
Mon, Mar 18, 11:03 AM · Active Work Proposal, Restricted Project, efl: language bindings, efl (efl-1.22)
lauromoura triaged T7752: C#: Get rid of all lower-cased methods as High priority.
Mon, Mar 18, 11:02 AM · Active Work Proposal, Restricted Project, efl: language bindings, efl (efl-1.22)
lauromoura triaged T7751: C#: Add back the I prefix to interfaces as High priority.
Mon, Mar 18, 11:01 AM · Active Work Proposal, Restricted Project, efl: language bindings, efl (efl-1.22)
bu5hm4n added a comment to T7738: Calling convention for events in eolian.

Okay this brings up another questions, are we supposed to allow changing of struct fields in event callbacks ?

Mon, Mar 18, 10:40 AM · Restricted Project, efl (efl-1.22), efl: api
zmike added a revision to T7713: some elementary_test cases no longer run: D8388: elm_test: use legacy api for glview gears test.
Mon, Mar 18, 10:34 AM · efl (efl-1.22), efl: widgets
cedric added a comment to T7738: Calling convention for events in eolian.

Hum, there is a side effect to being passed by values vs references. In one case events can not alter the value as it pass there handler. In the other, they can. Maybe we should have a "const" marker to specify explicitly this intent. I think this also make a stronger case to mark efl_event_callback_call a protected function and generate function that will have the proper type checking in their prototype.

Mon, Mar 18, 9:40 AM · Restricted Project, efl (efl-1.22), efl: api
q66 added a comment to T7738: Calling convention for events in eolian.

I don't think we currently specify how things are *meant* to be; we need to come up with something.

Mon, Mar 18, 9:13 AM · Restricted Project, efl (efl-1.22), efl: api
q66 added a comment to T7738: Calling convention for events in eolian.

Any proposals on this?

Mon, Mar 18, 9:13 AM · Restricted Project, efl (efl-1.22), efl: api
zmike raised the priority of T7738: Calling convention for events in eolian from TODO to Showstopper Issues.

This is currently the most urgent ticket for the 1.22 release.

Mon, Mar 18, 8:56 AM · Restricted Project, efl (efl-1.22), efl: api
zmike added a comment to T7311: fileselector unit test errors.

Still broken with that patchset.

Mon, Mar 18, 5:56 AM · Restricted Project, efl (efl-1.22), efl: data types
zmike merged T7666: elm_suite crashes intermittently into T7311: fileselector unit test errors.
Mon, Mar 18, 5:56 AM · Restricted Project, efl (efl-1.22), efl: data types
zmike merged task T7666: elm_suite crashes intermittently into T7311: fileselector unit test errors.
Mon, Mar 18, 5:56 AM · efl (efl-1.22), Restricted Project
zmike closed T7747: E segvs open emixer popup as Resolved.
Mon, Mar 18, 5:53 AM · efl (efl-1.22), enlightenment-git
zmike added a commit to T7747: E segvs open emixer popup: rEFLcced5487c83c: efl_ui_layout: call efl_del instead of efl_unref when text part does not exist.
Mon, Mar 18, 5:53 AM · efl (efl-1.22), enlightenment-git
zmike updated the task description for T7747: E segvs open emixer popup.
Mon, Mar 18, 5:37 AM · efl (efl-1.22), enlightenment-git

Sun, Mar 17

bu5hm4n closed T7224: eina safety check failed, obj == NULL in evas_object_event_callback_del_full in lib/evas/canvas/evas_callbacks.c:539, a subtask of T7030: efl internals pass null objects to functions which should not be passed null, as Resolved.
Sun, Mar 17, 8:38 AM · efl (efl-1.22), efl: data types, Restricted Project
bu5hm4n added a comment to T7666: elm_suite crashes intermittently.

@zmike Still happening ? I cannot reproduce this here. The test freezes from time to time (1/20 runs). But it never crashes ...

Sun, Mar 17, 8:31 AM · efl (efl-1.22), Restricted Project

Sat, Mar 16

bu5hm4n triaged T7747: E segvs open emixer popup as Showstopper Issues priority.
Sat, Mar 16, 9:27 AM · efl (efl-1.22), enlightenment-git

Fri, Mar 15

zmike closed T7745: elementary_config does not work anymore as Resolved.
Fri, Mar 15, 1:34 PM · efl (efl-1.22)
zmike added a commit to T7745: elementary_config does not work anymore: rEFL721f1776db5d: evas_events: prevent double event_freeze in evas_object_freeze_events_set.
Fri, Mar 15, 1:34 PM · efl (efl-1.22)
bu5hm4n added a comment to T7745: elementary_config does not work anymore.

Okay a few breadcrumps: there is a object overlaying everything.
The object is the frame object added by _elm_win_frame_add. I don't know why it overlaps everything, and i don't know why its not below.

Fri, Mar 15, 8:51 AM · efl (efl-1.22)
bu5hm4n claimed T7745: elementary_config does not work anymore.
Fri, Mar 15, 8:17 AM · efl (efl-1.22)
bu5hm4n added a comment to T7384: elm_test gengrid update speed test locks up.

Sooo - I manaed to bring down the times. However, its still lagging, the scrolling experience its less than perfect. However, on every group_calculate we walk a list of 5K items just by the design of gengrid, and that is slowing things down *a lot*. Not sure what else we can do here ....

Fri, Mar 15, 6:40 AM · efl (efl-1.22), Restricted Project
bu5hm4n added a revision to T7384: elm_test gengrid update speed test locks up: D8367: elm_gengrid: speed up update order calls.
Fri, Mar 15, 6:36 AM · efl (efl-1.22), Restricted Project