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> --