2016-03-08 5:47 GMT-03:00 Stefan Schmidt <ste...@osg.samsung.com>: > Hello. > > On 03/03/16 20:14, Vitor Sousa wrote: > >> vitorsousa pushed a commit to branch master. >> >> >> http://git.enlightenment.org/core/efl.git/commit/?id=9cbda57bf03bfdaa5c0379023a05868b2f0986aa >> >> commit 9cbda57bf03bfdaa5c0379023a05868b2f0986aa >> Author: Vitor Sousa <vitorsousasi...@gmail.com> >> Date: Thu Mar 3 15:10:13 2016 -0300 >> >> eolian-cxx: Fix Eolian C++ constructing methods with new eo_add >> Also fix Eolian C++ examples using the new "eo_super". >> --- >> src/bindings/eo_cxx/eo_cxx_interop.hh | 4 ++-- >> src/bindings/eo_cxx/eo_inherit.hh | 13 >> ++++++++++--- >> src/bindings/eo_cxx/eo_inherit_bindings.hh | 12 >> ------------ >> src/examples/eolian_cxx/colourable_cxx.cc | 6 +++--- >> src/examples/eolian_cxx/colourablesquare_cxx.cc | 2 +- >> src/examples/eolian_cxx/eolian_cxx_inherit_01.cc | 4 ++-- >> .../eolian_cxx/grammar/eo_class_constructors_generator.hh | 9 >> +++++---- >> 7 files changed, 23 insertions(+), 27 deletions(-) >> >> diff --git a/src/bindings/eo_cxx/eo_cxx_interop.hh >> b/src/bindings/eo_cxx/eo_cxx_interop.hh >> index 7cd7b35..b1978a8 100644 >> --- a/src/bindings/eo_cxx/eo_cxx_interop.hh >> +++ b/src/bindings/eo_cxx/eo_cxx_interop.hh >> @@ -472,9 +472,9 @@ F* alloc_static_callback(F&& f) >> template <typename... Fs> >> inline >> -void call_ctors(Fs&&... fs) >> +void call_ctors(Eo* _obj_eoid, Fs&&... fs) >> { >> - std::initializer_list<int const> const v {(fs(), 0)...}; >> + std::initializer_list<int const> const v {(fs(_obj_eoid), 0)...}; >> > > > After this change I get the following warning: > > CXXLD tests/evas_cxx/cxx_compile_test > In file included from ../src/bindings/eo_cxx/eo_inherit.hh:16:0, > from ../src/bindings/eo_cxx/Eo.hh:7, > from ../src/tests/eolian_cxx/generic.eo.hh:8, > from tests/eolian_cxx/eolian_cxx_test_binding.cc:5: > ../src/bindings/eo_cxx/eo_cxx_interop.hh: In instantiation of ‘void > efl::eolian::call_ctors(Eo*, Fs&& ...) [with Fs = {}; Eo = _Eo_Opaque]’: > ../src/tests/eolian_cxx/generic.eo.hh:368:208: required from ‘static Eo* > generic::_ctors_call(efl::eo::parent_type, generic::_c_required_ctor_a, > generic::_c_required_ctor_b<F0>, FOpts&& ...) [with F0 = > std::_Bind<eolian_cxx_test_binding_constructor_only_required(int)::<lambda()>()>; > FOpts = {}; Eo = _Eo_Opaque]’ > ../src/tests/eolian_cxx/generic.eo.hh:200:28: required from > ‘generic::generic(generic::_c_required_ctor_a, > generic::_c_required_ctor_b<F0>, FOpts&& ...) [with F0 = > std::_Bind<eolian_cxx_test_binding_constructor_only_required(int)::<lambda()>()>; > FOpts = {}]’ > tests/eolian_cxx/eolian_cxx_test_binding.cc:18:3: required from here > ../src/bindings/eo_cxx/eo_cxx_interop.hh:475:21: warning: parameter > ‘_obj_eoid’ set but not used [-Wunused-but-set-parameter] > void call_ctors(Eo* _obj_eoid, Fs&&... fs) > ^ > In file included from ../src/bindings/eo_cxx/eo_inherit.hh:16:0, > from ../src/bindings/eo_cxx/Eo.hh:7, > from ../src/tests/eolian_cxx/simple.eo.hh:8, > from tests/eolian_cxx/eolian_cxx_test_inheritance.cc:8: > ../src/bindings/eo_cxx/eo_cxx_interop.hh: In instantiation of ‘void > efl::eolian::call_ctors(Eo*, Fs&& ...) [with Fs = {}; Eo = _Eo_Opaque]’: > ../src/bindings/eo_cxx/eo_inherit.hh:88:209: required from > ‘efl::eo::inherit<D, E>::inherit(efl::eo::parent_type, Args&& ...) [with > Args = {}; D = bar; E = {simple}]’ > tests/eolian_cxx/eolian_cxx_test_inheritance.cc:16:45: required from here > ../src/bindings/eo_cxx/eo_cxx_interop.hh:475:21: warning: parameter > ‘_obj_eoid’ set but not used [-Wunused-but-set-parameter] > void call_ctors(Eo* _obj_eoid, Fs&&... fs) > ^ > CXXLD tests/eldbus_cxx/eldbus_cxx_suite >
Ops, I missed that. =) I created a diff on phab. > > regards > Stefan Schmidt > ------------------------------------------------------------------------------ Transform Data into Opportunity. Accelerate data analysis in your applications with Intel Data Analytics Acceleration Library. Click to learn more. http://makebettercode.com/inteldaal-eval _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
