[EGIT] [core/efl] master 02/13: eo_cxx: Fix do_eo_add after lifecycle branch.

[Lauro Moura]

cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=7d4c5f5d7c6de3359ed7887dbeaa150bf1423920

commit 7d4c5f5d7c6de3359ed7887dbeaa150bf1423920
Author: Lauro Moura <lauromo...@expertisesolutions.com.br>
Date:   Fri May 25 10:00:40 2018 -0700

    eo_cxx: Fix do_eo_add after lifecycle branch.
    
    Summary:
    Also changed the previously failing tests to use ck_assert_int_eqinstead
    of fail_if for better reporting.
    
    Test Plan: make check
    
    Reviewers: cedric, felipealmeida
    
    Reviewed By: cedric
    
    Subscribers: #committers, zmike
    
    Tags: #efl
    
    Differential Revision: https://phab.enlightenment.org/D6194
    
    Reviewed-by: Cedric BAIL <ced...@osg.samsung.com>
---
 src/bindings/cxx/eo_cxx/eo_cxx_interop.hh       |  2 +-
 src/tests/eolian_cxx/eolian_cxx_test_binding.cc | 12 ++++++------
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/bindings/cxx/eo_cxx/eo_cxx_interop.hh 
b/src/bindings/cxx/eo_cxx/eo_cxx_interop.hh
index fc00db830e..019a417d14 100644
--- a/src/bindings/cxx/eo_cxx/eo_cxx_interop.hh
+++ b/src/bindings/cxx/eo_cxx/eo_cxx_interop.hh
@@ -837,7 +837,7 @@ void do_eo_add(Eo*& object, P const& parent, Efl_Class 
const* klass
                , F&& f
                , typename std::enable_if< 
eo::is_eolian_object<P>::value>::type* = 0)
 {
-  bool is_ref = (parent._eo_ptr() != nullptr);
+  bool const is_ref = true;
   object = ::_efl_add_internal_start(__FILE__, __LINE__, klass, 
parent._eo_ptr(), is_ref, EINA_FALSE);
   ::efl::eolian::call_lambda(std::forward<F>(f), proxy);
   object = ::_efl_add_end(object, is_ref, EINA_FALSE);
diff --git a/src/tests/eolian_cxx/eolian_cxx_test_binding.cc 
b/src/tests/eolian_cxx/eolian_cxx_test_binding.cc
index 6cc6c6198e..0804cd7599 100644
--- a/src/tests/eolian_cxx/eolian_cxx_test_binding.cc
+++ b/src/tests/eolian_cxx/eolian_cxx_test_binding.cc
@@ -24,8 +24,8 @@ START_TEST(eolian_cxx_test_binding_constructor_only_required)
      }
     );
 
-  fail_if(1 != g.req_ctor_a_value_get());
-  fail_if(2 != g.req_ctor_b_value_get());
+  ck_assert_int_eq(1, g.req_ctor_a_value_get());
+  ck_assert_int_eq(2, g.req_ctor_b_value_get());
 }
 END_TEST
 
@@ -55,10 +55,10 @@ 
START_TEST(eolian_cxx_test_binding_constructor_all_optionals)
     }
     );
   
-  fail_if(2 != g.req_ctor_a_value_get());
-  fail_if(3 != g.opt_ctor_a_value_get());
-  fail_if(4 != g.req_ctor_b_value_get());
-  fail_if(5 != g.opt_ctor_b_value_get());
+  ck_assert_int_eq(2, g.req_ctor_a_value_get());
+  ck_assert_int_eq(3, g.opt_ctor_a_value_get());
+  ck_assert_int_eq(4, g.req_ctor_b_value_get());
+  ck_assert_int_eq(5, g.opt_ctor_b_value_get());
 }
 END_TEST
 

--