efl: major rewrite of efl_part.

The interface efl_part_get should not be directly called from C, but the efl_part
wrapper should. It rely on efl_noref to properly destroy the object. Binding can
control the lifecycle of the reference the way they want by either calling the
wrapper or efl_part_get directly. It also means that the ugly ___efl_auto_unref_set
doesn't need to be exposed outside of EFL anymore.

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

1 files changed, 2 insertions, 3 deletions

diff --git a/src/lib/eolian_cxx/grammar/part_implementation.hpp b/src/lib/eolian_cxx/grammar/part_implementation.hpp
index d88ae05b92..ec4da28913 100644
--- a/src/lib/eolian_cxx/grammar/part_implementation.hpp
+++ b/src/lib/eolian_cxx/grammar/part_implementation.hpp
@@ -29,9 +29,8 @@ struct part_implementation_generator
         return false;
       // FIXME: part_def can't depend on klass_def so C type is not known :(
       if(!as_generator(string << ">::type "<< string << "::" << string << "() const\n{\n"
-                       << scope_tab << "::Eo *__return_value = ::efl_part"
-                       << "(this->_eo_ptr(), \"" << string << "\");\n"
-                       << scope_tab << "::___efl_auto_unref_set(__return_value, false);\n")
+                       << scope_tab << "::Eo *__return_value = ::efl_part_get"
+                       << "(this->_eo_ptr(), \"" << string << "\");\n")
             .generate(sink, std::make_tuple(part.klass.eolian_name, klass_name, part.name, part.name), ctx))
         return false;
       if(!as_generator(scope_tab << "return ::" << *(string << "::"))