cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=9e364d6f4b1718e07efa0b4105c96259ef4826b9

commit 9e364d6f4b1718e07efa0b4105c96259ef4826b9
Author: Felipe Magno de Almeida <fel...@expertisesolutions.com.br>
Date:   Tue Mar 11 10:07:17 2014 +0900

    eina-cxx: Fixes compilation errors and warnings in clang
    
    Summary: @fix compilation errors with defining variable and type on the 
same statement on clang without a default-constructor. Also removed warnings 
with inconsistent uses of struct/class for forward declaration and unused 
parameters.
    
    Reviewers: cedric, stefan_schmidt
    
    CC: savio, cedric
    
    Differential Revision: https://phab.enlightenment.org/D622
---
 src/Makefile_Eina_Cxx.am               |  2 ++
 src/bindings/eina_cxx/eina_inarray.hh  |  2 +-
 src/bindings/eina_cxx/eina_inlist.hh   |  2 +-
 src/bindings/eina_cxx/eina_log.hh      | 12 ++++++++----
 src/bindings/eina_cxx/eina_ptrarray.hh |  6 +++---
 src/bindings/eina_cxx/eina_ptrlist.hh  |  4 ++--
 6 files changed, 17 insertions(+), 11 deletions(-)

diff --git a/src/Makefile_Eina_Cxx.am b/src/Makefile_Eina_Cxx.am
index 69ae679..d82493a 100644
--- a/src/Makefile_Eina_Cxx.am
+++ b/src/Makefile_Eina_Cxx.am
@@ -14,8 +14,10 @@ bindings/eina_cxx/eina_inarray.hh  \
 bindings/eina_cxx/eina_inlist.hh   \
 bindings/eina_cxx/eina_iterator.hh \
 bindings/eina_cxx/eina_lists_auxiliary.hh \
+bindings/eina_cxx/eina_log.hh \
 bindings/eina_cxx/eina_ptrarray.hh \
 bindings/eina_cxx/eina_ptrlist.hh \
+bindings/eina_cxx/eina_range_types.hh \
 bindings/eina_cxx/eina_ref.hh \
 bindings/eina_cxx/eina_stringshare.hh \
 bindings/eina_cxx/eina_thread.hh \
diff --git a/src/bindings/eina_cxx/eina_inarray.hh 
b/src/bindings/eina_cxx/eina_inarray.hh
index 7f6f9d2..a82eaa7 100644
--- a/src/bindings/eina_cxx/eina_inarray.hh
+++ b/src/bindings/eina_cxx/eina_inarray.hh
@@ -146,7 +146,7 @@ static T& index(Eina_Inarray* raw, std::size_t i)
 };
 
 template <typename T>
-struct inarray;
+class inarray;
 
 template <typename T>
 struct range_inarray : _range_template<T, _inarray_access_traits>
diff --git a/src/bindings/eina_cxx/eina_inlist.hh 
b/src/bindings/eina_cxx/eina_inlist.hh
index 9144735..a051209 100644
--- a/src/bindings/eina_cxx/eina_inlist.hh
+++ b/src/bindings/eina_cxx/eina_inlist.hh
@@ -240,7 +240,7 @@ struct _inlist_access_traits {
 };
 
 template <typename T, typename Allocator>
-struct inlist;
+class inlist;
 
 template <typename T>
 struct range_inlist : _range_template<T, _inlist_access_traits>
diff --git a/src/bindings/eina_cxx/eina_log.hh 
b/src/bindings/eina_cxx/eina_log.hh
index aec9b2e..d6b8f29 100644
--- a/src/bindings/eina_cxx/eina_log.hh
+++ b/src/bindings/eina_cxx/eina_log.hh
@@ -43,15 +43,19 @@ struct _domain_base
   }
 };
 
-const struct global_domain : _domain_base<global_domain>
+struct global_domain : _domain_base<global_domain>
 {
   int domain_raw() const { return EINA_LOG_DOMAIN_GLOBAL; }
-} global_domain;
+};
+
+struct global_domain const global_domain = {};
 
-const struct default_domain : _domain_base<default_domain>
+struct default_domain : _domain_base<default_domain>
 {
   int domain_raw() const { return EINA_LOG_DOMAIN_DEFAULT; }
-} default_domain;
+};
+
+struct default_domain const default_domain = {};
 
 struct log_domain : _domain_base<log_domain>
 {
diff --git a/src/bindings/eina_cxx/eina_ptrarray.hh 
b/src/bindings/eina_cxx/eina_ptrarray.hh
index b08d258..0587610 100644
--- a/src/bindings/eina_cxx/eina_ptrarray.hh
+++ b/src/bindings/eina_cxx/eina_ptrarray.hh
@@ -221,7 +221,7 @@ static eina::iterator<T> ibegin(Eina_Array* array)
   return eina::iterator<T>( ::eina_array_iterator_new(array) );
 }
 template <typename T>
-static eina::iterator<T> iend(Eina_Array* array)
+static eina::iterator<T> iend(Eina_Array*)
 {
   return eina::iterator<T>();
 }
@@ -231,7 +231,7 @@ static eina::iterator<T const> ibegin(Eina_Array const* 
array)
   return eina::iterator<T const>( ::eina_array_iterator_new(array) );
 }
 template <typename T>
-static eina::iterator<T const> iend(Eina_Array const* array)
+static eina::iterator<T const> iend(Eina_Array const*)
 {
   return eina::iterator<T const>();
 }
@@ -259,7 +259,7 @@ static bool empty(Eina_Array const* array)
 };
 
 template <typename T, typename Allocator>
-struct ptr_array;
+class ptr_array;
 
 template <typename T>
 struct range_ptr_array : _range_template<T, _ptr_array_access_traits>
diff --git a/src/bindings/eina_cxx/eina_ptrlist.hh 
b/src/bindings/eina_cxx/eina_ptrlist.hh
index 435077d..f81c0ba 100644
--- a/src/bindings/eina_cxx/eina_ptrlist.hh
+++ b/src/bindings/eina_cxx/eina_ptrlist.hh
@@ -215,7 +215,7 @@ static eina::iterator<T const> ibegin(Eina_List const* list)
   return eina::iterator<T const>( ::eina_list_iterator_new(list) );
 }
 template <typename T>
-static eina::iterator<T const> iend(Eina_List const* list)
+static eina::iterator<T const> iend(Eina_List const*)
 {
   return eina::iterator<T const>();
 }
@@ -379,7 +379,7 @@ struct _mutable_range_ptr_list : _const_range_ptr_list<T>
 };
 
 template <typename T, typename Allocator>
-struct ptr_list;
+class ptr_list;
 
 template <typename T>
 struct range_ptr_list : _range_template<T, _ptr_list_access_traits>

-- 


Reply via email to