projects / platform / upstream / efl.git / commitdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (parent: d5d119b)
eina_cxx: use efl_new instead of efl_add
authorMarcel Hollerbach <mail@marcel-hollerbach.de>
Tue, 16 Oct 2018 11:07:26 +0000 (13:07 +0200)
committerYeongjong Lee <yj34.lee@samsung.com>
Thu, 8 Nov 2018 06:40:45 +0000 (15:40 +0900)
Summary:
parent is NULL, this results in a error. And a NULL object. This fixes
that!
Depends on D7163

Reviewers: segfaultxavi

Reviewed By: segfaultxavi

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

src/tests/eina_cxx/eina_cxx_test_accessor.cc patch | blob | history
src/tests/eina_cxx/eina_cxx_test_ptrarray.cc patch | blob | history
src/tests/eina_cxx/eina_cxx_test_ptrlist.cc patch | blob | history

diff --git a/src/tests/eina_cxx/eina_cxx_test_accessor.cc b/src/tests/eina_cxx/eina_cxx_test_accessor.cc
index 5978345..7e3d8cf 100644 (file)
--- a/src/tests/eina_cxx/eina_cxx_test_accessor.cc
+++ b/src/tests/eina_cxx/eina_cxx_test_accessor.cc
@@ -50,10 +50,10 @@ EFL_START_TEST(eina_cxx_eo_accessor_indexing)
 
   efl::eina::list<wrapper> list;
 
-  wrapper const w1(efl_add(SIMPLE_CLASS, NULL));
-  wrapper const w2(efl_add(SIMPLE_CLASS, NULL));
-  wrapper const w3(efl_add(SIMPLE_CLASS, NULL));
-  wrapper const w4(efl_add(SIMPLE_CLASS, NULL));
+  wrapper const w1(efl_new(SIMPLE_CLASS));
+  wrapper const w2(efl_new(SIMPLE_CLASS));
+  wrapper const w3(efl_new(SIMPLE_CLASS));
+  wrapper const w4(efl_new(SIMPLE_CLASS));
 
   list.push_back(w1);
   list.push_back(w2);
@@ -104,10 +104,10 @@ EFL_START_TEST(eina_cxx_eo_accessor_iterator)
 
   efl::eina::list<wrapper> list;
 
-  wrapper const w1(efl_add(SIMPLE_CLASS, NULL));
-  wrapper const w2(efl_add(SIMPLE_CLASS, NULL));
-  wrapper const w3(efl_add(SIMPLE_CLASS, NULL));
-  wrapper const w4(efl_add(SIMPLE_CLASS, NULL));
+  wrapper const w1(efl_new(SIMPLE_CLASS));
+  wrapper const w2(efl_new(SIMPLE_CLASS));
+  wrapper const w3(efl_new(SIMPLE_CLASS));
+  wrapper const w4(efl_new(SIMPLE_CLASS));
 
   list.push_back(w1);
   list.push_back(w2);
diff --git a/src/tests/eina_cxx/eina_cxx_test_ptrarray.cc b/src/tests/eina_cxx/eina_cxx_test_ptrarray.cc
index 2cc1d2f..564b13d 100644 (file)
--- a/src/tests/eina_cxx/eina_cxx_test_ptrarray.cc
+++ b/src/tests/eina_cxx/eina_cxx_test_ptrarray.cc
@@ -33,9 +33,9 @@ EFL_START_TEST(eina_cxx_ptrarray_push_back)
   int result[] = {5, 10, 15};
   int rresult[] = {15, 10, 5};
 
-  wrapper const w1(efl_add(SIMPLE_CLASS, NULL));
-  wrapper const w2(efl_add(SIMPLE_CLASS, NULL));
-  wrapper const w3(efl_add(SIMPLE_CLASS, NULL));
+  wrapper const w1(efl_new(SIMPLE_CLASS));
+  wrapper const w2(efl_new(SIMPLE_CLASS));
+  wrapper const w3(efl_new(SIMPLE_CLASS));
 
   {
     efl::eina::ptr_array<int> array;
@@ -109,9 +109,9 @@ EFL_START_TEST(eina_cxx_ptrarray_pop_back)
     ck_assert(std::equal(array.rbegin(), array.rend(), rresult));
   }
   {
-    wrapper const w1(efl_add(SIMPLE_CLASS, NULL));
-    wrapper const w2(efl_add(SIMPLE_CLASS, NULL));
-    wrapper const w3(efl_add(SIMPLE_CLASS, NULL));
+    wrapper const w1(efl_new(SIMPLE_CLASS));
+    wrapper const w2(efl_new(SIMPLE_CLASS));
+    wrapper const w3(efl_new(SIMPLE_CLASS));
 
     efl::eina::array<wrapper> array;
 
@@ -204,11 +204,11 @@ EFL_START_TEST(eina_cxx_ptrarray_insert)
     ck_assert(std::equal(array.begin(), array.end(), array4.begin()));
   }
   {
-    wrapper const w0(efl_add(SIMPLE_CLASS, NULL));
-    wrapper const w1(efl_add(SIMPLE_CLASS, NULL));
-    wrapper const w2(efl_add(SIMPLE_CLASS, NULL));
-    wrapper const w3(efl_add(SIMPLE_CLASS, NULL));
-    wrapper const w4(efl_add(SIMPLE_CLASS, NULL));
+    wrapper const w0(efl_new(SIMPLE_CLASS));
+    wrapper const w1(efl_new(SIMPLE_CLASS));
+    wrapper const w2(efl_new(SIMPLE_CLASS));
+    wrapper const w3(efl_new(SIMPLE_CLASS));
+    wrapper const w4(efl_new(SIMPLE_CLASS));
 
     efl::eina::array<wrapper> array;
     ck_assert(std::distance(array.begin(), array.end()) == 0u);
@@ -284,7 +284,7 @@ EFL_START_TEST(eina_cxx_ptrarray_constructors)
   efl::eina::eina_init eina_init;
   efl::eo::eo_init eo_init;
 
-  wrapper const w1(efl_add(SIMPLE_CLASS, NULL));
+  wrapper const w1(efl_new(SIMPLE_CLASS));
 
   {
     efl::eina::ptr_array<int> array1;
@@ -363,12 +363,12 @@ EFL_START_TEST(eina_cxx_ptrarray_erase)
     ck_assert(array1.back() == 25);
   }
   {
-    wrapper const w1(efl_add(SIMPLE_CLASS, NULL));
-    wrapper const w2(efl_add(SIMPLE_CLASS, NULL));
-    wrapper const w3(efl_add(SIMPLE_CLASS, NULL));
-    wrapper const w4(efl_add(SIMPLE_CLASS, NULL));
-    wrapper const w5(efl_add(SIMPLE_CLASS, NULL));
-    wrapper const w6(efl_add(SIMPLE_CLASS, NULL));
+    wrapper const w1(efl_new(SIMPLE_CLASS));
+    wrapper const w2(efl_new(SIMPLE_CLASS));
+    wrapper const w3(efl_new(SIMPLE_CLASS));
+    wrapper const w4(efl_new(SIMPLE_CLASS));
+    wrapper const w5(efl_new(SIMPLE_CLASS));
+    wrapper const w6(efl_new(SIMPLE_CLASS));
 
     efl::eina::array<wrapper> array1;
 
@@ -444,12 +444,12 @@ EFL_START_TEST(eina_cxx_ptrarray_range)
   }
 
   {
-    wrapper const w1(efl_add(SIMPLE_CLASS, NULL));
-    wrapper const w2(efl_add(SIMPLE_CLASS, NULL));
-    wrapper const w3(efl_add(SIMPLE_CLASS, NULL));
-    wrapper const w4(efl_add(SIMPLE_CLASS, NULL));
-    wrapper const w5(efl_add(SIMPLE_CLASS, NULL));
-    wrapper const w6(efl_add(SIMPLE_CLASS, NULL));
+    wrapper const w1(efl_new(SIMPLE_CLASS));
+    wrapper const w2(efl_new(SIMPLE_CLASS));
+    wrapper const w3(efl_new(SIMPLE_CLASS));
+    wrapper const w4(efl_new(SIMPLE_CLASS));
+    wrapper const w5(efl_new(SIMPLE_CLASS));
+    wrapper const w6(efl_new(SIMPLE_CLASS));
 
     efl::eina::array<wrapper> array;
     array.push_back(w1);
diff --git a/src/tests/eina_cxx/eina_cxx_test_ptrlist.cc b/src/tests/eina_cxx/eina_cxx_test_ptrlist.cc
index 9635c0e..5c89d95 100644 (file)
--- a/src/tests/eina_cxx/eina_cxx_test_ptrlist.cc
+++ b/src/tests/eina_cxx/eina_cxx_test_ptrlist.cc
@@ -35,9 +35,9 @@ EFL_START_TEST(eina_cxx_ptrlist_push_back)
   int result[] = {5, 10, 15};
   int rresult[] = {15, 10, 5};
 
-  wrapper const w1(efl_add(SIMPLE_CLASS, NULL));
-  wrapper const w2(efl_add(SIMPLE_CLASS, NULL));
-  wrapper const w3(efl_add(SIMPLE_CLASS, NULL));
+  wrapper const w1(efl_new(SIMPLE_CLASS));
+  wrapper const w2(efl_new(SIMPLE_CLASS));
+  wrapper const w3(efl_new(SIMPLE_CLASS));
 
   {
     efl::eina::ptr_list<int> list;
@@ -84,9 +84,9 @@ EFL_START_TEST(eina_cxx_ptrlist_pop_back)
   int result[] = {5, 10};
   int rresult[] = {10, 5};
 
-  wrapper const w1(efl_add(SIMPLE_CLASS, NULL));
-  wrapper const w2(efl_add(SIMPLE_CLASS, NULL));
-  wrapper const w3(efl_add(SIMPLE_CLASS, NULL));
+  wrapper const w1(efl_new(SIMPLE_CLASS));
+  wrapper const w2(efl_new(SIMPLE_CLASS));
+  wrapper const w3(efl_new(SIMPLE_CLASS));
 
   {
     efl::eina::ptr_list<int> list;
@@ -150,9 +150,9 @@ EFL_START_TEST(eina_cxx_ptrlist_push_front)
     ck_assert(std::equal(list.rbegin(), list.rend(), rresult));
   }
   {
-    wrapper const w1(efl_add(SIMPLE_CLASS, NULL));
-    wrapper const w2(efl_add(SIMPLE_CLASS, NULL));
-    wrapper const w3(efl_add(SIMPLE_CLASS, NULL));
+    wrapper const w1(efl_new(SIMPLE_CLASS));
+    wrapper const w2(efl_new(SIMPLE_CLASS));
+    wrapper const w3(efl_new(SIMPLE_CLASS));
 
     efl::eina::list<wrapper> list;
 
@@ -178,9 +178,9 @@ EFL_START_TEST(eina_cxx_ptrlist_pop_front)
   int result[] = {10, 5};
   int rresult[] = {5, 10};
 
-  wrapper const w1(efl_add(SIMPLE_CLASS, NULL));
-  wrapper const w2(efl_add(SIMPLE_CLASS, NULL));
-  wrapper const w3(efl_add(SIMPLE_CLASS, NULL));
+  wrapper const w1(efl_new(SIMPLE_CLASS));
+  wrapper const w2(efl_new(SIMPLE_CLASS));
+  wrapper const w3(efl_new(SIMPLE_CLASS));
 
   {
     efl::eina::ptr_list<int> list;
@@ -280,11 +280,11 @@ EFL_START_TEST(eina_cxx_ptrlist_insert)
     ck_assert(std::equal(list.begin(), list.end(), list4.begin()));
   }
   {
-    wrapper const w0(efl_add(SIMPLE_CLASS, NULL));
-    wrapper const w1(efl_add(SIMPLE_CLASS, NULL));
-    wrapper const w2(efl_add(SIMPLE_CLASS, NULL));
-    wrapper const w3(efl_add(SIMPLE_CLASS, NULL));
-    wrapper const w4(efl_add(SIMPLE_CLASS, NULL));
+    wrapper const w0(efl_new(SIMPLE_CLASS));
+    wrapper const w1(efl_new(SIMPLE_CLASS));
+    wrapper const w2(efl_new(SIMPLE_CLASS));
+    wrapper const w3(efl_new(SIMPLE_CLASS));
+    wrapper const w4(efl_new(SIMPLE_CLASS));
 
     efl::eina::list<wrapper> list;
 
@@ -373,11 +373,11 @@ EFL_START_TEST(eina_cxx_ptrlist_constructors)
     ck_assert(list2 == list4);
   }
   {
-    wrapper const w0(efl_add(SIMPLE_CLASS, NULL));
-    wrapper const w1(efl_add(SIMPLE_CLASS, NULL));
-    wrapper const w2(efl_add(SIMPLE_CLASS, NULL));
-    wrapper const w3(efl_add(SIMPLE_CLASS, NULL));
-    wrapper const w4(efl_add(SIMPLE_CLASS, NULL));
+    wrapper const w0(efl_new(SIMPLE_CLASS));
+    wrapper const w1(efl_new(SIMPLE_CLASS));
+    wrapper const w2(efl_new(SIMPLE_CLASS));
+    wrapper const w3(efl_new(SIMPLE_CLASS));
+    wrapper const w4(efl_new(SIMPLE_CLASS));
 
     efl::eina::list<wrapper> list1;
     ck_assert(list1.empty());
Domain: UI Framework / EFL;