- User Since
- Aug 24 2015, 1:00 PM (173 w, 1 d)
Fri, Dec 14
Update after Vitor comments.
About the <value> tag on properties. Should we create a new task or it would be a new item here?
Thu, Dec 6
Wed, Dec 5
Resolved with new API in commit 3623753c1d6b893955a1c31e8559a2312f674192.
Tue, Dec 4
Update generating documentation for properties.
Mon, Dec 3
A suggestion from today's discussion on IRC was to do something like:
Fri, Nov 30
Well, just renaming the interfaces wouldn't be enough, as implementing classes like Efl.Ui.Text would still conflict when they implement the interface property.
Update after Vitor comments. Changed properties.eo to property_holder.eo
Updated version supporting interface properties.
Another thing: Some properties would lead to invalid names in C#, with the same name as the enclosing class (e.g. Efl.Text.Text). I'm thinking about adding a Prop suffix for these, like:
Thu, Nov 29
Update removing dead code from a previous mockup for this patch.
Update to use any_of in event_is_unique after suggestion by Vitor.
Update after removing return "abc" from name_helpers
Tue, Nov 27
Update after Vitor comments.
Mon, Nov 26
Update removing remaining event cast and changing name_helpers.hh comparison to case-sensitive. It was case insensitive when namespaces were lower cased.
Update after Xavi's comments on events.
Nov 16 2018
Nov 14 2018
Nov 12 2018
Update revision after conflict with newer version of parent revision.
Update C# code after Xavi's comments.
Nov 6 2018
The regular param tags already use the name attribute in function parameters. It was missing from the constructors but I added it to the new API branch.
Nov 5 2018
The *_class_get() methods can be safely made internal. I've integrated this in the new API.
Nov 1 2018
Oct 31 2018
From the C part, I think so. It'll need some work on the eolian_cxx library (which the C# generator uses) but we already work on both the bindings and eolian_cxx at the same time regularly.
Oct 26 2018
Oct 24 2018
Oct 19 2018
There still seems to be something wrong even after removing the Invalidate() call, as dismissing the popup in the updated example from D7173 shows errors like ERR<14922>:eina_safety ../../src/lib/elementary/efl_ui_layout_object.c:933 _efl_ui_layout_object_efl_layout_signal_signal_emit() safety check failed: efl_invalidated_get(obj) is true and some safety check failure afterwards (caught by the changed event handler in the example).
Oct 16 2018
Oct 11 2018
Oct 4 2018
Update removing the comment. Somehow I forgot it when splitting the patches.
Oct 3 2018
I've rebased the current new API work on top of yesterday (Oct 3rd) master (still in devs/lauromoura/csharp-new-classes) and it compiled without requiring any changes to the Eo files. All tests pass and a quick test showed at least elm examples are working. The generator new code requires some cleanup before pushing, though.
Update with SetIcon change from D7133
Will merge this change into D7134, as it already fixes the example build system.
Sep 27 2018
About hiding them from the tree, these are the methods either returning (->) or receiving as a parameter (between ()) a Mixin type.
Sep 25 2018
Sep 13 2018
The Mono compiler already extracts the documentation from the generated code (eolian_mono adds them as C# comments) into a XML file that VS/MonoDevelop use. Maybe we could just add a script to transform the generated XML into the DokuWiki syntax.
Sep 12 2018
I've pushed the current code to devs/lauromoura/csharp-new-classes.
Sep 11 2018
This @ctor_param would behave just like a hint, right? So each language decides whether it should support them as explicit constructor parameters (C#) or keep it implicit through efl_added (C/C++).
Sep 10 2018
Sep 5 2018
Marked as resolved. Somehow phab didn't close it after pushing.
Sep 4 2018
Sep 3 2018
Aug 31 2018
With help of pyolian, I could find these instances where we have abstract classes inheriting from regular classes somewhere in their inheritance chain:
(Somehow I removed myself as the assignee...)
The problem seems to be eina.Value using malloc/free internally to create/delete Eina_Values while it should use eina_value_new/eina_value_free to alloc/free from the Value mempool.