On 9/4/06, Matthias Baas <ba...@ir...> wrote:
> Roman Yakovenko wrote:
> > Hi Matthias. I just added function transformation unit test. Can you
> > add the code that uses FT
> > feature to it. Thus I will be able to work on code and to commit it
> > and to be sure I did not break
> > something.
>
> I've just updated and committed the test, but currently it won't pass
> anyway. There's still the problem of obtaining the result of an override
> that returns several values (see one of my previous mails).
> And apart from that I noticed that the test takes another code path in
> mem_fun_v_t.create_function_ref_code() and that path isn't updated to
> the new feature yet. By the way, Roman, could you please explain the
> differences between those three paths? I didn't fully grasp when each
> one has to be used.
>
Now I can complain about it too :-)
1. Why do you start to count arguments from 1 and not from zero?
2. Why Output is not according to coding convention?
3. I suggest to leave class output_t, but to add convenience function
output_only_arg( name or index ). I suggest to change output name, because
the word "output" is over abused in programming and it is not clear
from reading
the source code what you are trying to achieve.
--
Roman Yakovenko
C++ Python language binding
http://www.language-binding.net/
|
