HomePhabricator

csharp: Change to new class API.

Authored by lauromoura on Thu, Nov 29, 3:04 PM.

Description

csharp: Change to new class API.

Summary:
As discussed in T7204:

  • Eo Interfaces/mixins -> C# Interfaces with concrete class implementations
  • Eo Regular/Abstracts -> Proper C# classes
  • Added some new generators and helper methods.
  • Refactored the class generator, splitting into helper methods

Eo handles now are stored only in the "root" class in any given
inheritance tree (generally, Efl.Object), and accessible to each child.
Methods also are defined in a single place instead of repeatedly
generated in everyfile, reducing the size of the generated .dll from
30MB to around 4.5MB.

Mixins are generated as C# interfaces but any regular class it inherits
from is lost, as we can't have interfaces inheriting from regular
classes. This will be dealt with in a later commit.

Summary of API Changes:

  • Merged Inherit/Concrete classes. (These suffixes disappear from regular classes).
  • Interface still have implementations with 'Concrete' suffix for when they are returned from methods.
  • Removed 'I' from interface names.
  • Removed interfaces for regular/abstract Eo classes.
  • Concrete classes for interfaces/mixins hold the event argument struct.
  • Removed '_' from classes, enums, structs, etc, as indicated in C# naming conventions.
  • Namespaces are now Camel.Cased.
  • Renamed IWrapper's raw_handle/raw_klass to NativeHandle/NativeClass

Also renamed the test classes as after the namespace change, the
test namespace Test can conflict with the helper Test namespace.
(And use more meaningful names than Test.Testing...)

Also Fixes T7336 by removing a deprecated example and adding
efl_loop_timer_example to build system.

Fixes T7451 by hiding the class_get DllImports and renaming the IWrapper
fields. The native handlers are used in the manual binding.

Still need to work:

  • As there are still some events names clashing (e.g. Efl.Ui.Bg with "resize" from Efl.Gfx.Entity and Efl.Gfx.Image), Events are currently declared on the interface and implemented "namespaced" in the classes, requiring the cast to the interface to access the event.
  • The Mixin Conundrum. Mixin inheritance will be dealt in a future commit.

Depends on D7260

Reviewers: segfaultxavi, vitor.sousa, felipealmeida, Jaehyun_Cho

Reviewed By: vitor.sousa

Subscribers: cedric, reviewers, committers

Tags: efl

Maniphest Tasks: T7451, T7336

Differential Revision: https://phab.enlightenment.org/D7262